Closed Bug 482232 Opened 11 years ago Closed 11 years ago

Using search field in saved password window changes width of search field

Categories

(Toolkit :: Password Manager, defect, minor)

1.9.1 Branch
defect
Not set
minor

Tracking

()

VERIFIED FIXED
mozilla1.9.2a1

People

(Reporter: whimboo, Assigned: MattN)

Details

(Keywords: polish, verified1.9.1, Whiteboard: [polish-easy] [polish-interactive][polish-p2])

Attachments

(2 files, 1 obsolete file)

Attached image screenshot
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1b3) Gecko/20090305 Firefox/3.1b3

With a fresh profile the show password window is too narrow. So the whole textbox is cut-off and the magnifying glass is not visible. But when you search for a host the textbox width is changed and it will fit into the window with. Even the brush is visible.

Steps:
1. Create a fresh profile
2. Open the page info window of any web page
3. Go to the security tab and press "view saved passwords"
4. Hit ESC to clear the host name from the search textbox

With step 3 the search textbox fits exactly into the window. But hitting ESC stretches the textbox which let it flow outside of the window bounds and cuts-off the magnifying glass.
Summary: Using search field in saved password manager changes window width → Using search field in saved password window changes width of search field
Don't think this is a theme bug.
Component: Theme → Password Manager
Product: Firefox → Toolkit
QA Contact: theme → password.manager
Version: 3.1 Branch → 1.9.1 Branch
Whiteboard: [polish-easy] [polish-interactive]
Same happens with Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1b4pre) Gecko/20090305 Shiretoko/3.1b4pre ID:20090305020312
OS: Linux → All
Hardware: x86 → All
Assignee: nobody → mmn100+mozilla
The longest string (from current localizations) that will be shown in the <description> of this window is from es-AR:
   Las contrase\u00F1as para los siguientes sitios est\u00E1n guardadas en su computadora:

This fits snug into 450px so 500px leaves room for growth.
Attachment #376143 - Flags: review?(dolske)
Comment on attachment 376143 [details] [diff] [review]
Set min-width of 500px on description element

This isn't future-proof, depends on the system font face and size, and is probably the wrong approach anyway.

I think what you want is make the text wrap when it's too long. I.e. set .textContent rather than .value.
Attachment #376143 - Flags: review?(dolske) → review-
new patch based on suggestion in comment #4
Attachment #376143 - Attachment is obsolete: true
Attachment #376310 - Flags: review?(dolske)
Attachment #376310 - Flags: review?(dolske) → review+
Keywords: checkin-needed
(I've got this in my checkin queue with other things, just waiting for a green tree)
Keywords: checkin-needed
Pushed http://hg.mozilla.org/mozilla-central/rev/dee2ce1fce39
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Comment on attachment 376310 [details] [diff] [review]
wrap long text with textContent of descr.

a191=beltzner
Attachment #376310 - Flags: approval1.9.1+
Target Milestone: --- → mozilla1.9.2a1
Verified fixed on trunk and 1.9.1 on all platforms with builds like

Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.2a1pre) Gecko/20090517 Minefield/3.6a1pre ID:20090517031115

Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1b5pre) Gecko/20090519 Shiretoko/3.5b5pre ID:20090519031312
Status: RESOLVED → VERIFIED
This bug's priority relative to the set of other polish bugs is:
P2 - Polish issue that is in a secondary interface, occasionally encountered, and is easily identifiable.
Whiteboard: [polish-easy] [polish-interactive] → [polish-easy] [polish-interactive][polish-p2]
You need to log in before you can comment on or make changes to this bug.