Closed Bug 16729 Opened 25 years ago Closed 24 years ago

Widgets in Modern skin should use CSS2 system fonts

Categories

(SeaMonkey :: Themes, defect, P2)

defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: ian, Assigned: hewitt)

References

(Depends on 1 open bug)

Details

(Keywords: access, Whiteboard: [rtm-])

Attachments

(1 file)

Since moving the XUL things like the size of the menus and the folder listing in mail/news ignore the windows settings and just use the default ones. These should be linked to the windows settings (ie the windows settings should be checked at Moz startup) Lemming
Depends on: 3371
Summary: Font sizes not respected → Widgets should use CSS2 system fonts
OS: other → All
Hardware: PC → All
Assignee: trudelle → shuang
Component: XUL → UE/UI
UI detail dependent upon bug #3371
Assignee: shuang → pierre
Target Milestone: M15
reassign it to pierre since I don't really know what to do. Also set ( a much later)same milestone as the dependency bug #3371.
Summary: Widgets should use CSS2 system fonts → Font sizes not respected
Target Milestone: M15
Something doesn't sound right here. I'm not sure Ian was refering to some settings in the Windows-OS Control Panels. Ian's comments, as well as his original summary line ("Font sizes not respected"), make me think that the bug was about some settings from the AppRunner Preferences panel that are currently ignored by the application. Ian, could you confirm? Or if I'm wrong, could you rephrase your bug report according to the guidelines? http://www.mozilla.org/quality/bug-writing- guidelines.html Changing the summary line back to "Font sizes not respected" from "Widgets should use CSS2 system fonts".
Whiteboard: waiting for more info from reporter
Sorry, I was unclear in my original report. What I mean is that by default, UI componants such as the name of menus, text under icons on toolbars etc are set to around size 10. This size can be modified in the appearance tab of the windows display properties (right click desktop > Properties). By using XUL instead of the standard windows hooks, the size of the names on menus etc _do not_ change according to the settings in the windows display properties dialog. It should.
Summary: Font sizes not respected → Widgets should use CSS2 system fonts
Whiteboard: waiting for more info from reporter
Target Milestone: M15
Thanks Ian. I'm keeping this bug for now. I'll reassign it to the XPToolkit team when the CSS2 fonts are implemented (see #3371).
Pushing my M15 bugs to M16
Maybe this bug should be re-assigned to the UI team when you have fixed 3371? I think the UI team is writing the CSS "skins" for the UI.
Sherry Wang (Browser Development - IBM, Austin TX - xiaotong@us.ibm.com) volunteered to do the work when I'm done with bug 3371.
Moving all UE/UI bugs to new component: User Interface: Design Feedback UE/UI component will be deleted.
Component: UE/UI → User Interface: Design Feedback
Blocks: 18135
Depends on: 33312, 33313
No longer depends on: 3371
Reassigned to ben. The Mac implementation works. GFX on Windows (bug 33312) and Unix (33313) needs to be completed. Sherry: if you want to help on Windows or Unix (the 2 bugs above) or for the chromes (this bug), please contact the respective assignees: rods, pavlov and ben. Thanks!
Assignee: pierre → ben
Status: ASSIGNED → NEW
Blocks: 31417
I still would like to volunteer for this bug - modifying the chromes to use CSS2 system fonts and colors. Sherry Wang xiaotong@us.ibm.com
This bug is similar to bug 1021.
No longer blocks: 31417
non-essential skin work. shifting off.
Status: NEW → ASSIGNED
Target Milestone: M16 → M30
Perhaps it is not essential to use CSS system fonts in the UI skins, but it does seem quite important that the product adhere to the user's choice of font face and size. This is an accessibility issue. Some people cannot read the default font because it is too small, or not the right font face for them, etc. Ben, are you saying that it is not important to consider these needs, or just that we can achieve that goal via some means other than CSS system fonts?
*** Bug 18135 has been marked as a duplicate of this bug. ***
I'm surprised this has been pushed past M20, because based on my reading of the chrome files, all it requires (once the dependencies have been fixed) is a quick blat through one CSS file (xul.css, I think it was), adding relevant `font:' lines to the properties for each widget. And the improvement in appearance of the UI would be quite substantial. Ben, if you really don't have time for this before M20, *please* reassign it to xiaotong@us.ibm.com who has volunteered to do it.
Ben, I will send you the changes to skin using CSS2 system fonts and colors shortly.
*** Bug 31417 has been marked as a duplicate of this bug. ***
Classic has been using system fonts on windows since the start. Unfortunately there are problems. Supposedly the system font support doesn't understand bitmap fonts like MS Sans Serif on Windows 95/98, so all fonts appear to be Times New Roman. Windows 2000 supplies MS Sans Serif in a different format however and it is understood and used by Mozilla correctly. cc'ing Michael Lowe, in case he has any ideas (?) Pierre informs me that system font support works on mac, but I've never seen it myself (running OS8.6 on my G3-450 all system font CSS is ignored.. now I *know* that it is possible on 8.6 because IE5 does it ;) I keep meaning to ask Pierre about this...
This bug really applies to the modern skin, since it will be the default. If Netscape are going to include other skins then they should also use system fonts (and the same when fonts are unspecified). I have changed the summary to reflect this, change it back if you disagree.
Severity: minor → blocker
Summary: Widgets should use CSS2 system fonts → Widgets in all skins shipped with Mozilla/Netscape should use CSS2 system fonts
Ben: Just FYI, system fonts don't appear to be working at all under Linux. In fact, in a build I compiled last night, changing the document font from "sans serif" to "serif" in the prefs changed the UI font. (Maybe that's a separate bug?)
*** Bug 44806 has been marked as a duplicate of this bug. ***
hm. Closing this bug as WONTFIX as I don't believe there was any intent to use system * in Modern. If you want a skin that tries to match your system settings, use classic ;)
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → WONTFIX
This would handily solve the problems in bug 5236, and as Erik pointed out, it's an accessibility issue, especially since the fonts in Modern are hardwired to be so small, and since a patch has apparently already been supplied. Sherry, do you still have your patch? Could you attach it to this bug and add the keyword "patch"? Will reassign to Don since Ben doesn't want it. Braden: I reopened bug 33313 since I agree with you that gtk system fonts don't seem to be working (at least, Classic skin comes up with miniscule fonts whereas normal gtk apps come up with nice readable fonts).
Status: RESOLVED → REOPENED
Keywords: access
Resolution: WONTFIX → ---
Someone suggested reassigning this to don, but I'll wait and let Ben comment on it.
adding "loose" dependency to 5236.
Blocks: 5236
Blocks: 51748
Turns out Pav's bug 33313 is a dup (but not marked as such) of 48907; adding that dependency as well.
Depends on: 48907
To Paul, who will probably mark as WONTFIX again...
Assignee: ben → hangas
Status: REOPENED → NEW
*** Bug 55441 has been marked as a duplicate of this bug. ***
*** Bug 55463 has been marked as a duplicate of this bug. ***
adding dependancy - we can't do this till the modern skin is totally scalable.
Depends on: 55464
Changing summary slightly: * Remove "Netscape". They might feel differently about this, and they can track this issue in the Netscape bug database if they need a separate resolution. * Changing "all skins" to "Modern2". That seems to be what this bug is tracking at this point, and the notion that all skins that ship with Mozilla *must* use system fonts seems a bit heavy-handed.
Summary: Widgets in all skins shipped with Mozilla/Netscape should use CSS2 system fonts → Widgets in Modern2 skin should use CSS2 system fonts
I don't understand. Is there such a thing as "Modern2"? Or are you saying that we should create a new theme called Modern2, and make it use system fonts?
Alec, I found the place where the font-size is being set to 8.5pt (it's set to 3mm, which is the same): mozilla/xpfe/global/resources/locale/en-US/intl.css In a commercial debug build on Linux, it's installed in: ns/dist/bin/chrome/en-US/locale/en-US/global/intl.css
For some reason I was under the impression that the Modern skin was called Modern2.
Summary: Widgets in Modern2 skin should use CSS2 system fonts → Widgets in Modern skin should use CSS2 system fonts
erik: gack! We should see how things look without that... but I'll bet we still need to fix this.
nominate for rtm
Keywords: rtm
jar and/or hamerly to check this out on akkana's machine, since alecf sez the bug can be demonstrated there.
This is my favorite bug. I really want this one fixed, it is huge for everyone that wants to change their system fonts. If we don't get this in rtm we will checkin a fix on the trunk for Modern as soon we can. We may even create a version of Modern with this fix to be made available for those folks running N6.
Status: NEW → ASSIGNED
Priority: P3 → P2
Whiteboard: [rtm need into]
sending to hewitt
Assignee: hangas → hewitt
Status: ASSIGNED → NEW
Sadly marking rtm-
Whiteboard: [rtm need into] → [rtm-]
--> Themes. (Out of my component! Shoo! Shoo!)
Component: User Interface: Design Feedback → Themes
Status: NEW → ASSIGNED
Attached patch big long patch (deleted) — Splinter Review
changes look OK. andreww has provided review, a=ben@netscape.com
Fixed. System fonts are now honored -- all widgets that contain text can now stretch in both dimensions.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago24 years ago
Resolution: --- → FIXED
Argh! I don't like this fix! On XFree86 3.3.6 under Linux with 100 dpi fonts, the widgets become HUGE! Even at 75 dpi they're bigger than they were. I thought that modern was beautiful before this, but now I'm just frustrated. Can anybody tell me how to put this back to the way that it was?????
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
This bug _IS FIXED_. The problem with large Linux fonts is a separate bug. For those of you Linux users who can't stand the enormity of the new fonts, and don't want to wait for it to be fixed, you can put this in your userChrome.css file to restore sanity: button, menubutton, menulist, menu, menuitem, tab, .button-toolbar, window { font-size: 12px !important; } You may want to tweak that font size to fit your preference.
Status: REOPENED → RESOLVED
Closed: 24 years ago24 years ago
Resolution: --- → FIXED
actually, things look really good now. especially the mailnews thread pane. thanks for the fix, hewitt.
Why do I have this? This should never have been assigned to me... qacontact->pmac
QA Contact: ckritzer → pmac
Marking verified (2001-01-24-09-Mtrunk).
Status: RESOLVED → VERIFIED
Product: Core → SeaMonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: