Closed Bug 353438 Opened 19 years ago Closed 19 years ago

Preview of Location Bar in "Customize" dialog flawed

Categories

(Firefox :: Toolbars and Customization, defect)

x86
Windows XP
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 3 alpha2

People

(Reporter: me, Assigned: asaf)

References

Details

Attachments

(1 file, 1 obsolete file)

User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1) Gecko/20060918 BonEcho/2.0 Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1) Gecko/20060918 BonEcho/2.0 The preview of the Location bar in the "Customize" dialog is flawed. What we should do with this really depends on Bug 347754. If the Go button turns out to be removable (as Bug 347754 calls for), we should center the picture (this is what I would much prefer). If it turns out that the "Go" button is to be connected to the Location bar, than a Go button should added to the preview in "Customize". Reproducible: Always Steps to Reproduce: 1. Right click on the top toolbars. Click "Customize..." to open the Customize dialog. 2. Remove the location bar by dragging it to the dialog. Actual Results: Preview of Location bar in "Customize" dialog is flawed. Expected Results: Fix depending on Bug 347754.
Depends on: 347754
*** Bug 353605 has been marked as a duplicate of this bug. ***
Blocks: NewTheme
Flags: blocking-firefox2?
edgecase, still recognizable, not going to block.
Flags: blocking-firefox2? → blocking-firefox2-
Assignee: nobody → mano
Target Milestone: --- → Firefox 3 alpha2
Attached patch winstripe fix (obsolete) — Splinter Review
Attachment #246080 - Flags: review?(gavin.sharp)
Attached patch winstripe fixSplinter Review
oops, wrong file. Also note this only addresses the go-button part.
Attachment #246080 - Attachment is obsolete: true
Attachment #246082 - Flags: review?(gavin.sharp)
Attachment #246080 - Flags: review?(gavin.sharp)
Attachment #246082 - Flags: review?(gavin.sharp) → review+
note to self: not-an-issue in pinstripe.
mozilla/browser/themes/winstripe/browser/browser.css 1.52
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: