Closed
Bug 171454
Opened 22 years ago
Closed 5 years ago
[cust] Customize toolbar windows is incorrectly sized when using larger fonts
Categories
(Toolkit :: Toolbars and Toolbar Customization, defect)
Toolkit
Toolbars and Toolbar Customization
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: andre.bugs2, Unassigned)
References
Details
(Keywords: access, fixed-aviary1.0, helpwanted)
Attachments
(13 files, 6 obsolete files)
1.59 KB,
patch
|
hewitt
:
review+
|
Details | Diff | Splinter Review |
22.07 KB,
image/png
|
Details | |
17.76 KB,
image/jpeg
|
Details | |
18.79 KB,
image/png
|
Details | |
12.98 KB,
image/png
|
Details | |
61.24 KB,
image/png
|
Details | |
41.13 KB,
image/jpeg
|
Details | |
18.99 KB,
image/png
|
Details | |
39.90 KB,
image/jpeg
|
Details | |
20.67 KB,
image/png
|
Details | |
34.65 KB,
image/png
|
Details | |
37.40 KB,
image/png
|
Details | |
1.03 KB,
patch
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.0.2) Gecko/20020922 Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.0.2) Gecko/20020922 The new customize toolbar window is incorrectly sized on Linux - Asa told me it looks better on Windows. I will attach a screenshot below. Reproducible: Always Steps to Reproduce:
Reporter | ||
Comment 1•22 years ago
|
||
Updated•22 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
Target Milestone: --- → Phoenix0.2
Comment 3•22 years ago
|
||
I don't see this. I suspect the larger fontsize makes the difference.
Summary: Customize toolbar windows is incorrectly sized → [cust] Customize toolbar windows is incorrectly sized
Comment 5•22 years ago
|
||
I see this! Running on WinNT, I have "Large Fonts" installed.
Reporter | ||
Comment 6•22 years ago
|
||
OS->All since this seams to be related to large fonts and not limited to Linux.
OS: Linux → All
Comment 7•22 years ago
|
||
I use Debian 3.0 sid with fvwm winmgr. Indeed the cust window shows only partially without scrolling bars. I get around it by defining (or having) a resize key and then resizing the window so all the buttons show up. "done" button then rolls up the window after use.
Reporter | ||
Updated•22 years ago
|
Summary: [cust] Customize toolbar windows is incorrectly sized → [cust] Customize toolbar windows is incorrectly sized when using larger fonts
Comment 9•22 years ago
|
||
Wanted to add I see this on both Win2k and Linux (RH8). It is only a problem with the larger fonts as far as I can tell.
Comment 10•22 years ago
|
||
Try a very large screen font, e.g. -microsoft-Verdana-medium-r-normal--*-120-*-*-p-*-iso8859-1 (Linux) and you have no way to close the dialog. The OK and Cancel buttons are completely outside the dialog, and the dialog remains open even after exiting Phoenix.
Comment 11•22 years ago
|
||
*** Bug 174343 has been marked as a duplicate of this bug. ***
Comment 12•22 years ago
|
||
*** Bug 176633 has been marked as a duplicate of this bug. ***
Comment 13•22 years ago
|
||
This patch resizes the customization window if the content needs more space.
Updated•22 years ago
|
Target Milestone: Phoenix0.4 → Phoenix0.5
Comment 14•22 years ago
|
||
*** Bug 177678 has been marked as a duplicate of this bug. ***
Comment 15•22 years ago
|
||
I see this too. I suppose that's because I have only VGA display (shame on me...) Phoenix 0.4, Win 98. Artem Vakhitov
Comment 16•22 years ago
|
||
Comment on attachment 104322 [details] [diff] [review] patch Hey Joe, can you take a look at this customize toolbar patch? Thanks.
Attachment #104322 -
Flags: review?(hewitt)
Updated•22 years ago
|
Attachment #104322 -
Flags: review?(hewitt) → review+
Comment 17•22 years ago
|
||
Comment on attachment 104322 [details] [diff] [review] patch r=me
Comment 18•22 years ago
|
||
With this patch, isn't it possible that the window can be wider than the screen, thus hiding the Done button?
Comment 19•22 years ago
|
||
Dean, do you think that's a common case? I don't think that should block our landing this fix for people who have fonts that are slightly or even moderately larger. We need an "escape" key to dismiss the dialog in the case that the window position (or I suppose, really large fonts or a really small screen) forces the dialog off screen. Is that reported?
Comment 20•22 years ago
|
||
I agree with Asa's most recent comment. I just looked at Phoenix under 1600x1200. Although i don't normally use that screen resolution, I know some people do. Those people would need to use a larger font size to keep toolbars readable. We really need an elastic box. Failing that, and if scrollbars are not available, the standard Windows command to accept and close a settings window is the Enter key. I'd recommend that.
Comment 21•22 years ago
|
||
Asa: You're probably right. And the patch doesn't add the problem, since the done button is barely visible in the screenshot, it just doesn't resolve it.
Comment 22•22 years ago
|
||
> We need an "escape" key to dismiss the dialog ...
That's a terrible solution. If we can't get this working properly on all
platforms and all resolutions then we need to ditch the whiz-bang sheet effect
and stick with a standard dialog.
Updated•22 years ago
|
Target Milestone: Phoenix0.5 → Phoenix0.6
Comment 23•22 years ago
|
||
You could temporarily replace the content area with the customisation tools. That way it's still attached to the toolbar (by being part of the same window!) and it can be resized by the user. Of course, it wouldn't really be a dialog/sheet any more, so I suspect the implementation would be nasty, but I don't think it's entirely Crazy Talk.
Comment 24•22 years ago
|
||
Is this going to be merged into the main source tree that will become the nightly builds of Phoenix anytime soon? Using the 11/19 nightly I am still encountering this. Maybe the window should be just flagged as resizeable by default... ie so that the WM puts resize bars on the corners?
Comment 25•22 years ago
|
||
Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.3a) Gecko/20021202 Phoenix/0.4 It also affects the Help | About Phoenix dialog, which is non resizeable.
Comment 26•22 years ago
|
||
Comment 27•22 years ago
|
||
I atteched the screenshot above. Phoenix often shows the dialog with wrong size. This window really has 4 buttons but 3 of them are not visible. Steps to reproduce: 1.Launch Phoenix milestone 0.4. or 0.5 undex Windows XP (easier to reproduce) or Windows 2000. 2.Navigate to www.sunhill.ru. You should see the page in Russian. 3.Choose the link in left frame with location https://62.16.74.2/cgi-bin/clients/enter. (It is called "statistics" in Russian, the word of 10 cyrillic leters.) 3.Observe the results. The bug is not always reproducable, but it happens very often under WinXP.
Comment 28•22 years ago
|
||
*** Bug 186786 has been marked as a duplicate of this bug. ***
Comment 29•22 years ago
|
||
I have noticed that the problem occurs because the icons are in a line, left to right, and the rightmost ones fall outside of the box. OK, this is obvious. Would it be easier to make this customize box if the icons were stacked vertically, with scrollbars, etc.?
Comment 30•22 years ago
|
||
*** Bug 189150 has been marked as a duplicate of this bug. ***
Comment 31•22 years ago
|
||
*** Bug 187413 has been marked as a duplicate of this bug. ***
Comment 32•21 years ago
|
||
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.3b) Gecko/20030204 Phoenix/0.5 with the new Nautilus 1.0 theme. The Customize window has a vertical scrollbar. The Qute 1.0 beta theme has the same old problem.
Comment 33•21 years ago
|
||
well, from my testing, it seems that the problem is not on the palette box, but is on the row of buttons below the palette box which is too large when the font size is too large, so, my proposed fix for this bug is to break the `buttons bar' apart.
Comment 34•21 years ago
|
||
Comment 35•21 years ago
|
||
! first saw this on WinME, so it applies to all.
It looks like patches are already made... but I think I found something
relaited to this bug.
(on winME)
Help-->about-->Copyright and contributer-->right click on menubar-->cutomize--
>Oh my god, can't close it! so alt-F4 (customization window goes away) --> Menu
is all greyed out (on the about: page), it's closable... but it's frustrating
and its a bug.
Mozilla/5.0 (Windows; U; Win 9x 4.90; en-US; rv:1.4a) Gecko/20030321 Phoenix/0.5
It happened the few times that I tried to do it.
Comment 36•21 years ago
|
||
> [...] but it's frustrating and its a bug. may be related to bug 170499. BTW anybody can try to see if that patch can solve the problem? i have tried on gtk2, and it seems that the customize windows does not overflow if a set to ~18pt application font size. Furthermore, this patch does _NOT_ address all font size, if user uses insanely HUGE size (say, 26pt), the customize window may still overflow.
Comment 37•21 years ago
|
||
how do you apply these patches anyway? I have no idea where to start even!
Updated•21 years ago
|
Attachment #117594 -
Attachment is obsolete: true
Comment 38•21 years ago
|
||
there are three patches concerning customize dialog in total. (bug 178078, bug 171454, bug 171106)
Comment 39•21 years ago
|
||
Comment on attachment 118195 [details] [diff] [review] customize dialog patch, part 2 of 3 sorry...
Attachment #118195 -
Attachment is obsolete: true
Comment 40•21 years ago
|
||
there are three patches concerning customize dialog in total. (bug 178078, bug 171454, bug 171106)
Updated•21 years ago
|
Attachment #118196 -
Flags: review?(hewitt)
Comment 41•21 years ago
|
||
My fonts are set to a custom size (84% -- makes a logical inch on my monitor equivilant to an actual inch), and this causes similar issues, see attachment. I could attach many more examples of this upon request.
Comment 43•21 years ago
|
||
anybody can take a look at my patch? it is very simple and it seems to fix the problem.
Comment 44•21 years ago
|
||
*** Bug 211874 has been marked as a duplicate of this bug. ***
Comment 45•21 years ago
|
||
I've looked for ways to implement patches into the Firebird... I think I remember finding something, but couldn't get it to work.. So, How do we try your patch?
Updated•21 years ago
|
Target Milestone: Firebird0.7 → Firebird1.0
Comment 48•21 years ago
|
||
Noririty, your checkin probably belongs to this bug, not bug 171451. 08/10/2003 04:11 mozilla/ toolkit/ content/ customizeToolbar.css tweak css. b=171451
Comment 49•21 years ago
|
||
Steffen: Right. It's still not completely but expected works well.
Comment 50•21 years ago
|
||
*** Bug 216092 has been marked as a duplicate of this bug. ***
Comment 51•21 years ago
|
||
Using the latest nightly (Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.5b) Gecko/20030813 Mozilla Firebird/0.6.1+), I now see a scrollbar which allows me to access the elements that do not fit. However, I can not use the mouse wheel to scroll through it. Is this worth a separate bug?
Comment 52•21 years ago
|
||
This bug causes usability issues to people with bad sight bacause they set a larger font size by default. Adding the Access keyword.
Keywords: access
Comment 53•21 years ago
|
||
Why wasn't this marked FIXED? The Customize window now works well with larger fonts.
Comment 54•21 years ago
|
||
There is an issue remaining: The button "Restore Default Set" is cut off.
Attachment #101023 -
Attachment is obsolete: true
Comment 55•21 years ago
|
||
This it pathetic. This bug is almost one year old, and configuring the latest Firebird release is still a pain in the ass because of the messed up toolbar customize dialog. If you are unable to fix this, couldn't you just make that window a normal, resizable one, instead of this frameless thingie?
Comment 56•21 years ago
|
||
This patch removes the sheet effect (as suggested by Blake in comment 22) and turns the dialog into a proper, self-sizing dialog. I've tested it at various font sizes (up to 26px) and it always lays out correctly.
Updated•21 years ago
|
Attachment #134408 -
Flags: review?(bugs)
Updated•21 years ago
|
Attachment #134408 -
Flags: superreview?(blake)
Comment 57•21 years ago
|
||
Comment on attachment 134408 [details] [diff] [review] Whiz-bang sheet effect removal patch Superreview is currently not needed for mozilla firebird. Since Ben is pretty loaded, moving review request over to Dean.
Attachment #134408 -
Flags: superreview?(blake)
Attachment #134408 -
Flags: review?(dean_tessman)
Attachment #134408 -
Flags: review?(bugs)
Updated•21 years ago
|
Attachment #134408 -
Flags: review?(dean_tessman) → review?(blake)
Comment 58•21 years ago
|
||
I run into this problem with the Dutch translation, i circumvent it by editing the width in the file content/global/customizeToolbar.js from within the toolkit.jar archive. While you work out how to programmatically solve this problem, i'd like to see this be set like the "prefWindow.size" in browser/pref/pref.dtd ...
Comment 59•20 years ago
|
||
*** Bug 236359 has been marked as a duplicate of this bug. ***
Comment 60•20 years ago
|
||
In WinXP opening the customize toolbar window. There is no title bar, you can not move the window. The 'done' button can not be clicked. The only solution is Alt+F4.
Comment 61•20 years ago
|
||
Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.6) Gecko/20040206 Firefox/0.8 with 9pt verdana as my 'message box' font, the text at the bottom "show [icons] []use small icons [add new toolbar][restore default set]" gets a horizontal scroll bar, which partially hides the text. 9pt verdana is not much larger than the default font on win2k, but is much easier for me to read, except here! i can upload a screenshot of this if it will help
Updated•20 years ago
|
Flags: blocking1.0?
Comment 62•20 years ago
|
||
I'm not convinced we can't do better here. -ing. I'm pretty sure that if someone sat down and thought about it they could come up with a patch that sized correctly.
Flags: blocking1.0? → blocking1.0-
Comment 63•20 years ago
|
||
*** Bug 240669 has been marked as a duplicate of this bug. ***
Comment 64•20 years ago
|
||
Has no progress been made for this bug? I'm using the latest (0.9.1) under Linux/Gnome/Metacity and have had this issue for ages. The Customize dialog is in a random size and position pretty much every time I open it. People are going on about font sizes etc.. but I don't appear to have done anything from the default and I have this issue.. Is this a window manager issue or are no size considerations being done at all? Anyhow, just wanted to register the fact that this is not limited to Win or Fonts or whatever. I'll have a look at this stylesheet stuff, but if it works, why hasn't this been put into the main releases given that this has been floating around since firefox was called PHOENIX?
Comment 65•20 years ago
|
||
can we please review (and possibly land) the patch at comment 53? I've been able to reproduce a variet of wrongly sized "sheet" on windows and especially linux. Currently with default settings on all of the platforms, we get a vertical scrollbar.
Flags: blocking-aviary1.0- → blocking-aviary1.0?
Updated•20 years ago
|
Flags: blocking-aviary1.0? → blocking-aviary1.0PR+
Whiteboard: [have patch]
Updated•20 years ago
|
Whiteboard: [have patch] → [have patch] -blake
Comment 66•20 years ago
|
||
blake or ben, can you have a look?
Whiteboard: [have patch] -blake → [have patch] -blake ben
Comment 67•20 years ago
|
||
sometimes it looks as horribly as the screenshot in the attachment. (ff0.9.3)
Comment 68•20 years ago
|
||
Text string is hidden behind "Choose Folder ..." button.
Comment 69•20 years ago
|
||
I just installed Firefox 0.9.3, added the Noia 2.0 theme, and saw this bug when browsing Firefox' features. The first time I opened the Customize Toolbar dialog, it was positioned so that the right side was slightly off-screen, completely hiding the vertical scrollbar and half-hiding the Done button. (I'm just lucky it didn't hide the -whole- Done button!) Without any window decoration, I am unable to properly resize or move the dialog. What I want to know is, what goofball decided to create a dialog box without any window manager support (title, move, resize controls)? And why, given the long history of complaints in this bug, has nothing been done about it?
Comment 70•20 years ago
|
||
I just noticed that in Tools > Options > Enable Javascript > Advanced, the window size is fixed. Yet the companion windows Load Images > Exceptions and Block Popup Windows > Allowed Sites, the windows are resizable. The Extension Manager and Theme Manager windows are resizable. I guess I don't understand why the Customize window is not resizable. It has been a problem since I began using Phoenix in October 2002.
Comment 71•20 years ago
|
||
because its implemented as a sheet, not a dialog/window.
Comment 72•20 years ago
|
||
I don't think this should block 1.0 PR. There won't be a l10n impact, since worst-case is that we don't use the Done string and use the built-in OK string. That's not a stopper at this point. The sheet is a much better interface than a dialog is going to be, IMO, and I think its worth holding off and trying to make it work right, instead of killing it. As for the patch, it won't do anythign about the vertical scrollbar, since that will appear depending on the number of items in the toolbar palette. (The patch fixes the palette box to 300px in height, which virtually assures the scrollbar with our current palette.) Fixing that will be dependent on the same careful thought as fixing the rest of this bug, so why make a negative change now? If we have to do something like this for 1.0, then lets wait on that drastic step and try to find a better solution in the meantime. The change is quite low-risk as it stands, so there's no need to rush a decision here.
Comment 73•20 years ago
|
||
ok, lets try and get this fixed by 1.0
Flags: blocking-aviary1.0PR-
Flags: blocking-aviary1.0PR+
Flags: blocking-aviary1.0+
Comment 74•20 years ago
|
||
hu... I was just working on it. I have a patch ready for linux with the animation. I need to test it on windows, but I agree it shouldn't block PR1.0.
Comment 75•20 years ago
|
||
*** Bug 258576 has been marked as a duplicate of this bug. ***
Comment 76•20 years ago
|
||
Wrong dialog size still present on Linux with 1.0PR, fresh install (new profile, default theme) Same behavior as the 0.9.3 demo attachment.
Comment 77•20 years ago
|
||
I'm attaching a screenshot of the customize-toolbar window, as rendered using a fresh profile on my SuSE/KDE desktop. As you can see, the window is rendered in a ridiculous size. I don't know if this is the same bug, or a different one. My screen resolution is 1600x1200. I'm using a medium font size. Mozilla/5.0 (X11; U; Linux i686; rv:1.7.3) Gecko/20040910 Firefox/0.10
Comment 78•20 years ago
|
||
Fixed by the combination of bug 245088 and bug 262256?
Updated•20 years ago
|
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Updated•20 years ago
|
Attachment #118196 -
Attachment is obsolete: true
Attachment #118196 -
Flags: review?(hewitt)
Updated•20 years ago
|
Attachment #134408 -
Attachment is obsolete: true
Attachment #134408 -
Flags: review?(firefox)
Updated•20 years ago
|
Keywords: fixed-aviary1.0
Comment 79•20 years ago
|
||
This bug is still present in Firefox/0.10; the screenshot was taken on windows xp with display set to 120dpi.
Comment 80•20 years ago
|
||
Indeed, even on Fx 1.0RC1 the selectable options toolbar hangs over the right end when large fonts are selected. You can get at the overhang with a scrollbar in SphereGnome 0.9.8.2 and 0.9.9, but the trouble is that when you use ever larger fonts, the fixed size of the window means that the palette portion gets smaller and smaller. This will be an issue on notebook computers that have high resolutions and small screens. Is there any way to specify the customize window with ex units rather than px units? If so, the box would expand as the fontsize grows larger.
Comment 81•20 years ago
|
||
Because 'overflow: auto' is set to the box containing the form elements in the toolbar, the scrollbar runs over those form elements. This is very annoying, because you have to make the toolbar wider to make the scrollbar disappear, instead of higher what you expect you must do.
Comment 82•20 years ago
|
||
Reopening, as the bug has not been completely fixed in Firefox 1.0.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 83•20 years ago
|
||
Attachment #159343 -
Attachment is obsolete: true
Comment 84•19 years ago
|
||
If this was fixed in aviary-1.0 is it really still a problem? Marking P2 since a fix was devies once already so it should be low hanging fruit.
Keywords: helpwanted
Priority: -- → P2
Comment 85•19 years ago
|
||
The fix is a kludge, not a proper fix. On my system - Win XP, Large Font option (125%/120px), the Restore Default Button is only half in the window. The problem is exacerbated when larger fonts are chosen. Why would larger fonts be chosen? Well, one user of the SphereGnome Jumbo theme uses a screen resolution of 3820x2400. You would really need large fonts for that. Even with a screen resolution of 1900x1200, if you have a 15-inch laptop display, you'd want larger fonts. I note the window size is in px units. Perhaps if it were sized in ex units it could grow as system font size increases.
Updated•19 years ago
|
Priority: P2 → P3
Comment 86•19 years ago
|
||
I have just looked at the new Netscape browser pre-beta. The Customize window is resizable. This solves every one of my problems with the Customize window.
Updated•19 years ago
|
QA Contact: bugzilla → toolbars
Comment 87•19 years ago
|
||
By moving the "Restore Default Set" button to the bottom left of the Customize Toolbar window, there is much less chance that some part will fall outside the window because of large font sizes. This also solves an issue mentioned in bug #171106 comment #4 https://bugzilla.mozilla.org/show_bug.cgi?id=171106#c4 (point 11). Clicking this button does have major consequences that can not be undone (not in one step at least), so I think it is reasonable that this button should be placed where it is (in my experience) less accessible: bottom left of the window. I know this will not fix issues with really large font sizes, but it will definitely make those issues less worse.
Attachment #204142 -
Flags: review?(mconnor)
Comment 88•18 years ago
|
||
With recent builds on Linux and Windows, this dialog now appears resizable and a vertical scroll bar appears. That resolves my issue, since I can resize the dialog to accomodate larger fonts. The only potential problem I see is if the dialog width is wider than the screen. But perhaps this bug should be closed, and a new one opened to track that issue. Here are the versions I tried that seem fixed: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.13) Gecko/20060414 CentOS/1.0.8-1.4.1.centos4 Firefox/1.0.8 Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.0.4) Gecko/20060508 Firefox/1.5.0.4
Updated•17 years ago
|
Attachment #204142 -
Flags: review?(mconnor)
Comment 89•17 years ago
|
||
Seems like the patch Teune van Steeg posted was never reviewed? The problem still exists here on Linux, and I don't use overly huge fonts. 10pt I think. Can we fix this for Firefox 3? ;)
Comment 90•16 years ago
|
||
Just discovered that there was a "stop" button available to add to Thunderbird 2.0.0.9 toolbar in WinXP, SP2. It was not seen for years because it was located on the left side of the customize window and was blocked. Only discovered it by increasing the window to full screen, but then couldn't drag it off due to it is in fact full screen! Then discovered that the right side of the window could be stretched and the button became visible. However, the sizing must be hard coded and does not stick on the new size. Very annoying to say the least! If additional info is needed regarding font size, etc., please let me know.
Comment 91•16 years ago
|
||
Realized that this bug is for Firefox, not Thunderbird which is my concern. Perhaps it should be refiled as a new bug for TB, or will that be taken care of on your end?
Updated•16 years ago
|
Assignee: noririty → nobody
Status: REOPENED → NEW
Priority: P3 → --
Hardware: PC → All
Whiteboard: [have patch] -blake ben
Target Milestone: Firefox1.0 → ---
Updated•16 years ago
|
Flags: blocking-firefox3?
Comment 94•16 years ago
|
||
This will not block the final release of Firefox 3. Any patch will need unit tests in order to be approved.
Flags: blocking-firefox3? → blocking-firefox3-
Comment 96•16 years ago
|
||
patch needs new owner
Component: Toolbars → Toolbars and Toolbar Customization
Flags: blocking-firefox3-
Product: Firefox → Toolkit
QA Contact: toolbars → toolbars
Comment 97•5 years ago
|
||
We no longer use a sheet/dialog for customize and instead use a full-page tab. I've tested the new Customize... tab with very large fonts and everything still works as expected. Can this bug be resolved as WFM given the new implementation?
Comment 98•5 years ago
|
||
Makes sense to me. As the reporter of one of the dupes (bug 414435), I'll go ahead and close as WFM. :D
Status: NEW → RESOLVED
Closed: 20 years ago → 5 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•