Closed Bug 67410 Opened 24 years ago Closed 24 years ago

focus effect on widgets is ugly

Categories

(SeaMonkey :: Themes, defect, P2)

x86
Windows 98
defect

Tracking

(Not tracked)

VERIFIED FIXED
mozilla0.9

People

(Reporter: jruderman, Assigned: hewitt)

References

()

Details

(Keywords: modern, polish)

In prefs under modern, most controls get blue borders when they're focused. The borders seem to be broken/transparent at the top right of textboxes and buttons (but not most other controls).
This has to do with the crappy implementation of -moz-border-radius, which draws really ugly "rounded" corners when the radius is any greater than 3px. It would probably make sense for us to thin out the focus border on textfields, buttons, and menulists (especially textfields) to make the rounded borders look a little cleaner.
Status: NEW → ASSIGNED
Priority: -- → P3
I don't know exactly how the widgets are rendered, but possibly the method I used for my own custom controls would work (for the same reason). Render the focus border as 1 pixel thicker than you need it, so that roughly 1 pixel of it is *inside* the area where the actual text field/button will be. Then render the text field or button over this. This way, you don't have to worry about the slight errors when doing rounded corners.
The current focus looks awful. Hewitt, please fix this for 0.9!
I agree whole-heartedly. I don't know what we were thinking when we made widgets use such a wide -moz-border-radius on focus. We'll definitely clean it up for mozilla0.9
Keywords: mozilla0.9, uipolish
Priority: P3 → P2
Summary: [modern] no blue border at upper-right corner of focused textbox, button → [modern] focus effect on widgets is ugly
Target Milestone: --- → mozilla0.9
Keywords: modern
Summary: [modern] focus effect on widgets is ugly → focus effect on widgets is ugly
Depends on: 65745
This is now partially fixed. I just checked in polish for the focus effects on textfield. Other widgets will follow when I land the modern cleanup bug (bug 65745).
Much prettier now :) fixed.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Marking verified on windows (2001-03-19-12-Mtrunk).
Status: RESOLVED → VERIFIED
Product: Core → SeaMonkey
You need to log in before you can comment on or make changes to this bug.