Closed
Bug 364262
Opened 18 years ago
Closed 18 years ago
white rect appears behind the "gear" action pop-up menu when it is clicked.
Categories
(Camino Graveyard :: Preferences, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Camino1.5
People
(Reporter: murph, Assigned: froodian)
Details
(Keywords: fixed1.8.1.2, polish, regression)
Attachments
(2 files, 1 obsolete file)
39.83 KB,
image/png
|
Details | |
20.08 KB,
application/zip
|
alqahira
:
review+
stuart.morgan+bugzilla
:
superreview+
|
Details |
Inside either of the two sheets that are displayed from within the privacy preference pane (Show Cookies… or Edit Exceptions List…) there is a white artifact drawn around the shortcut actions pop-up button (bottom left corner).
Steps to Reproduce:
1) Open Preferences.
2) Switch to the Privacy pane.
3) Click the Show Cookies or Edit Exceptions List buttons.
4) Press the gear pop-up in the bottom left and leave it selected.
Actual Results: A white rect is drawn behind the button. (10.4.8+: control+scroll to zoom in and see it better). A screenshot of this is attached.
Expected Results: Behave like the pop-up buttons inside the bookmark manager and not display anything around the button when pressed.
Reporter | ||
Comment 1•18 years ago
|
||
I wonder if it's always done that?
Keywords: polish
Assignee | ||
Comment 3•18 years ago
|
||
No. 1.1a1 is the first release to have this problem... which means it was probably something I did. :(
Assignee | ||
Comment 4•18 years ago
|
||
This is because the border for those elements extends slightly beyond the button. I changed it for ease in positioning, but looks like the border should shrink down again.
Assignee | ||
Comment 5•18 years ago
|
||
I resized the bounding boxes, then manually repositioned them until the final result had the same placement as the originals (by checking the build results with a ruler).
Comment on attachment 249065 [details]
New Privacy.nib
There's still one white pixel along the entire left side of the button in each sheet :( (There is no such line in 1.0.x, so unless there is clipping in 1.0.x on 10.4, this is not a 10.3-10.4 discrepancy....) Other than that, the layout is fine.
Also, how difficult is it to port the grey-button-hack from Edit Permissions to Edit Cookies, so that Esc works in that sheet, too.
Attachment #249065 -
Flags: review?(alqahira) → review-
Assignee | ||
Comment 7•18 years ago
|
||
This fixes the remaining white pixel.
The grey escape button is cool, but unfortunately causes an ugly white flash for me. I've taken it out for now - we should discuss a good way to keep this functionality in another bug.
Attachment #249065 -
Attachment is obsolete: true
Attachment #249084 -
Flags: review?(alqahira)
Comment on attachment 249084 [details]
New Privacy.nib
r=ardissone
Sucks about the hidden button :(
Attachment #249084 -
Flags: superreview?(stuart.morgan)
Attachment #249084 -
Flags: review?(alqahira)
Attachment #249084 -
Flags: review+
> The grey escape button is cool, but unfortunately causes an ugly white flash
> for me. I've taken it out for now - we should discuss a good way to keep this
> functionality in another bug.
Now bug 364320.
Comment 10•18 years ago
|
||
Comment on attachment 249084 [details]
New Privacy.nib
sr=smorgan, but someone should double-check that it doesn't stomp bug 330534 (it seemed to still work for me).
Attachment #249084 -
Flags: superreview?(stuart.morgan) → superreview+
Whiteboard: [needs checkin with comment 10 addressed]
Assignee | ||
Comment 11•18 years ago
|
||
Checked in on trunk and MOZILLA_1_8_BRANCH (and yeah, that other fix still worked for me too)
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Keywords: fixed1.8.1.2
Resolution: --- → FIXED
Whiteboard: [needs checkin with comment 10 addressed]
You need to log in
before you can comment on or make changes to this bug.
Description
•