"Manage fields" tab not styled correctly on sub-pages [#702788]

Comment	File	Size	Author
#8	seven_tab.patch	573 bytes	james.elliott
	manage fields.png	28.5 KB	tstoeckler

Comment #1

yched commented 2 February 2010 at 21:25

The paths were garbled in the OP ?

Off hand, I'm not sure I see how that's a bug in field_ui.

Log in or register to post comments

Comment #2

tstoeckler

he/him

German

Essen, Germany

commented 2 February 2010 at 21:47

Oh yeah, that's correct. Anyways, when you're on the
/admin/structure/types/manage/article/fields
page or the same page for any other content type, and click on any link. You'll see that the tab is messed up.
That covers field settings, field instance settings, widget select, deletion.

field_ui.module was just a guess. Maybe it's even Seven theme. Don't know.

Log in or register to post comments

Comment #3

yched commented 2 February 2010 at 22:38

I cannot reproduce, actually.
Win FF 3.6 here.

Although there *is* a difference in the markup for the tab
On /admin/structure/types/manage/article/fields, the <li> and the <a> tags both have the 'active' class
On the subpages, only the <li> is 'active', not the <a> - which is expected behavior.

So I'd call that a Seven bug in [fill in your browser] :-)

Log in or register to post comments

Comment #4

yched commented 2 February 2010 at 22:41

Component:

field_ui.module

» Seven theme

Oddly enough, I could reproduce the bug on my 1st visit to admin/appearance/settings/seven
but everything back to normal if I reload the page...

Log in or register to post comments

Comment #5

tstoeckler

he/him

German

Essen, Germany

commented 3 February 2010 at 00:58

Yeah, you're right. I get it there too, so nothing to do with Field UI.
It doesn't disappear for me on a reload, though.
Are the ones on the Field UI pages gone then for you too?
If yes, maybe it's just something with my setup. (Weird though, it's a pretty fresh install, and clearing caches doesn't help either)

Log in or register to post comments

Comment #7

cosmicdreams commented 5 March 2010 at 21:59

Hmm........ I can't seem to be able to reproduce this bug. Maybe another patch as resolved it?

Log in or register to post comments

Comment #8

james.elliott commented 1 April 2010 at 19:45

Status	File	Size
new	seven_tab.patch	573 bytes

There seems to be a problem with the reset.css file being more specific than style.css

reset.css selects the following on line 122
ul.primary li.active a,

And then style.css adds padding to the active tab with the following selectors starting on line 292

ul.primary li a,
ul.primary li a.active,
ul.primary li a:active,
ul.primary li a:visited,
ul.primary li a:hover

The reset.css selector is more specific than the ul.primary li a in style.css which is the one that is applicable in this circumstance, therefore the padding gets set to 0.

This patch adds ul.primary li.active a to the list of selectors starting on line 292 so that the padding is applied correctly.