Verdana is running the show in overlay titles and tabs. Posting pics in first comment

.ui-widget {
font-family:Verdana,Arial,sans-serif;
font-size:1.1em;
}
ui.theme.css?D (line 13)
CommentFileSizeAuthor
#9 goawayverdana.patch522 bytesyoroy

Comments

yoroy’s picture

Issue tags: +CSS, +Needs design review

Currently:
Only local images are allowed.

Desired:
Only local images are allowed.

jacine’s picture

This is not the only problem by far with ui.theme.css.

ui.theme.css is the jQuery Smoothness theme. It is not a reasonable default. Any site/theme that uses jQuery UI will have these styles imposed on it.

The patch I posted here fixes this problem for Seven, but we are running out of time on it: #614494: Give Seven a nice looking default jQuery UI theme. Demo here: http://bit.ly/7ftjcX

As far as other themes are concerned, the file will most likely need to be overwritten by the themer. There is an issue related to this here #583400: jQuery UI libraries are loading too much CSS that needs to be reverted / Allow jQuery UI themes to be swappable. I've started on a stripped down version for themers to start with. It's shown here: http://bit.ly/5pyd25

David_Rothstein’s picture

Component: Seven theme » overlay.module

@yoroy, it looks like your screenshots were made with the Seven theme underneath the overlay as well (on the Content Types page, right?) If you had made them with the main site theme underneath (e.g., Garland) you'd get a different result.

What's going on here is explained in more detail at #700832: Parts of the overlay inherit CSS from the underlying site theme, leading to a font mismatch in the overlay page title - in the general case, the overlay header and tabs are actually not even using the same theme as the rest of the overlay!!! (They are actually using the main site theme, and the jQuery UI stuff apparently was trying to paper it over a bit - but not really that well.)

So although #614494: Give Seven a nice looking default jQuery UI theme is in theory the right idea, that patch by itself doesn't fix this particular problem. We'd need to find a way to make the default jQuery UI theme for Seven be applied to all of the overlay, including the parts that aren't even using Seven - or else, rip this apart entirely. Ugh :(

aspilicious’s picture

Verdana is gone, Helvitica it is...
Still a problem?

David_Rothstein’s picture

Again, this isn't an issue with the specific font - the problem is that the wrong theme is being used entirely. All you have to do is change the main site theme to something unusual and you can make these fonts bleed through from "underneath", into the overlay title and tabs. Seven can't fix it if Seven isn't even controlling it.

For more details, see here - I think this is the underlying cause:
#782656: The overlay title and tabs should be inside the child window, not the parent window

aspilicious’s picture

Status: Active » Fixed

Fixed by #668640: Overlay shouldn't be based on jQuery UI Dialog.

Change themes and fontsize stays the same :)

Status: Fixed » Closed (fixed)

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

yoroy’s picture

Status: Closed (fixed) » Active

Still defines Verdana:
Only local images are allowed.

yoroy’s picture

Status: Active » Needs review
StatusFileSize
new522 bytes

Lets just remove that line

aspilicious’s picture

Status: Needs review » Reviewed & tested by the community

Played with this some more. You're right! Nice catch!
Love to RTBC this.

casey’s picture

Still applies.

ksenzee’s picture

Still applies and makes perfect sense.

jacine’s picture

How the heck did that get there? :P

Yes, please go away Verdana. RTBC +1.

dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)
Issue tags: -CSS, -Needs design review

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