Comments

fubhy’s picture

Yep, that is indeed a problem (noticed that a few times already too) but its not a Omega Issue, its a problem with Rubik and how it handles nested Fieldsets :(

himerus’s picture

Status: Active » Closed (works as designed)

Yes, it is sad.... I was a HUGE fan of Rubik... However, I've recently switched to using the core provided Seven theme for administrative pages.

It does look like there was an update for Rubik in March, but historically, they've been VERY slow to update anything in the D7 version.

anniegreens’s picture

Project: Omega » Rubik
Version: 7.x-3.x-dev » 7.x-4.0-beta6
Status: Closed (works as designed) » Active

I'm reassigning the project on this issue to Rubik, as it appears to be caused by Rubik's handling of nested/collapsible fieldsets.

Attached are screenshots of two vertical tab fieldsets from Omega's theme configuration page. The screenshots from Rubik show broken handling and the screenshots from Seven show correct handling.

This may be a simple CSS fix. I will dig into it a little deeper this afternoon.

EDIT: d.o seems to be having issues. I cannot successfully upload all attachments at this time. Please find the images here: http://imgur.com/a/ZPGVY#9ObAB

coffeymachine’s picture

Status: Active » Needs review
StatusFileSize
new1.13 KB

Here is a patch for this, based on the dev branch. If someone could sanity check my work and make sure I didn't break anything else, that would be awesome. :)

himerus’s picture

I will check this next time in front of a Rubik/Omega install... (tomorrow) :)

I did some playing myself on some CSS that "sorta" got it to work in Rubik, but broke the nested fieldsets outside of any Rubik admin, so I scrapped my effort.

coffeymachine’s picture

StatusFileSize
new1.71 KB

After some further testing this morning, I discovered some issues with non-collapsible fieldsets that the patch created. I fixed those, here is a new patch.

coffeymachine’s picture

StatusFileSize
new2.19 KB

...aaand I found some new issues with this patch and the Views admin UI. I fixed them here in this version.

wavesailor’s picture

FYI: It only seems to break in Firefox and not Chrome. Unfortunately it's the only browsers I use.

himerus’s picture

Status: Needs review » Reviewed & tested by the community

I can verify that the patch from #7 fixes the layout issues w/nested fieldsets in an omega theme settings page.

Sorry it took me so long to getting around to test it.

markhalliwell’s picture

I can attest to this problem as well, is there any time frame on when this might get pushed?

Thanks!

Tezza’s picture

#7 works for me too. Thanks c-machine, that saved me from applying my own targeted but sloppy negative top margin fix.

coffeymachine’s picture

Status: Reviewed & tested by the community » Fixed

Thanks for testing everyone. The patch is now committed.
Beta7 release also coming shortly...

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.