Closed Bug 508757 Opened 16 years ago Closed 11 years ago

About Mozilla Firefox window: image offset

Categories

(Firefox :: General, defect)

x86
macOS
defect
Not set
trivial

Tracking

()

RESOLVED INVALID

People

(Reporter: gelosi, Assigned: markus)

Details

(Whiteboard: [polish-easy] [polish-visual][polish-p2])

Attachments

(3 files, 1 obsolete file)

User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1.2) Gecko/20090729 Firefox/3.5.2 Build Identifier: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1.2) Gecko/20090729 Firefox/3.5.2 Open "About Mozilla Firefox" window, look at the right edge of window - you see white line upside down. Guess, this gap should be removed. Reproducible: Always Steps to Reproduce: 1. Open "About Mozilla Firefox" Actual Results: see gap between firefox image and window edge Expected Results: All is nice You can see screenshot at: http://yfrog.com/74picture1ccyp
Attached image About window in Firefox 3.5.3 Mac (obsolete) —
Not present on Firefox 3.5.3 Mac. However, I remember this being present previously, and it is also present in trunk.
Here's the about window from mac Firefox trunk, which obviously has the annoying 1px line.
Strange… when trying a freshly downloaded 3.5.3 Mac version, the line is there, I don't know how I managed to produce the screenshot in comment 1. Anyway, here's a simple patch to solve this minor glitch. By evaluating a few builds, I found out that the bug was not present in 2.0 but was there in 3.0. None of the changes in browser/base/aboutDialog.css/js/xul between those two versions could explain this bug. Loading chrome://browser/content/aboutDialog.xul in a regular browser window resizes the window to the proper size, without the white line. Something seems to be happening that obstructs the about window from doing the same. So the fix for now is to lower the window width by one pixel, as in the attached patch.
Assignee: nobody → markus.magnuson
Attachment #403555 - Attachment is obsolete: true
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attachment #413397 - Flags: ui-review?(faaborg)
Attachment #413397 - Flags: review?(mano)
Attachment #413397 - Flags: ui-review?(faaborg) → ui-review+
Whiteboard: [polish-easy] [polish-visual][polish-p2]
Hmm, I just realized this has not been tested on Linux/Win which may not suffer from the same glitch. Who could I ask to check this out? Also, is it even possible to conditionally branch CSS files according to platform?
This is the about window from 3.5.5 on Ubuntu. I am not really sure how window borders are supposed to work on Linux, could someone with a bit more knowledge of interface guidelines and such comment on this? Should we change the width on Linux too?
The bug is still reproduced in Firefox 3.5.5 on my Mac OS X 10.5.8
Comment on attachment 413397 [details] [diff] [review] Remove white line by reducing window width by one pixel. In this case, the width should be set for individual elements and not for the entire dialog.
Attachment #413397 - Flags: review?(mano) → review-
Asaf, how then should the width of the background image be related to window width?
I figure you mean that the window should not have a width set at all, is that correct?
window get sized by their content. If the content is well sized, there should not be a need to set the window's width.
This bug is now invalid and should be closed.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: