The default bug view has changed. See this FAQ.

Correct margins in history preference pane

VERIFIED FIXED in Camino1.5

Status

Camino Graveyard
Preferences
--
minor
VERIFIED FIXED
11 years ago
11 years ago

People

(Reporter: froodian (Ian Leue), Assigned: froodian (Ian Leue))

Tracking

({verified1.8.1})

unspecified
Camino1.5
PowerPC
Mac OS X
verified1.8.1

Details

Attachments

(2 attachments, 6 obsolete attachments)

6.58 KB, application/zip
Mike Pinkerton (not reading bugmail)
: superreview+
Details
58.61 KB, image/png
Details
(Assignee)

Description

11 years ago
This is part of preference pane cleanup. (see bug 325880)
(Assignee)

Comment 1

11 years ago
Created attachment 227496 [details]
New History.nib
Attachment #227496 - Flags: review?(alqahira)
(Assignee)

Comment 2

11 years ago
Created attachment 227497 [details]
Screenshot
Comment on attachment 227496 [details]
New History.nib

There's extra space on the right, due to the extra length of the text boxes again.
Attachment #227496 - Flags: review?(alqahira) → review-
(Assignee)

Comment 4

11 years ago
Comment on attachment 227496 [details]
New History.nib

Are you sure?  Both hint text boxes are pulled exactly to 20px from the end...
Attachment #227496 - Flags: review- → review?(alqahira)
Yes, but both have 15-20px of blank space between the last character and the edge of the box, so it looks like 35-40px.
(Assignee)

Comment 6

11 years ago
Created attachment 227523 [details]
Fixes centering

yeah, you're right.  sorry about that. :p
Attachment #227496 - Attachment is obsolete: true
Attachment #227523 - Flags: review?(alqahira)
Attachment #227496 - Flags: review?(alqahira)
(Assignee)

Comment 7

11 years ago
Created attachment 227524 [details]
Screenshot
Attachment #227497 - Attachment is obsolete: true
Comment on attachment 227523 [details]
Fixes centering

The bottom rectangle still is longer than its text, which makes the right side look bigger.  It's certainly better.

Shouldn't we be setting first responder on the text field for when the "force focus highlight on first item" bug is fixed?
Attachment #227523 - Flags: review?(alqahira) → review-
(Assignee)

Comment 9

11 years ago
Created attachment 227645 [details]
Centers more betterer

Also fixes initialFirstResponder.  Thanks for catching that.
Attachment #227523 - Attachment is obsolete: true
(Assignee)

Updated

11 years ago
Attachment #227645 - Flags: review?(alqahira)
(Assignee)

Comment 10

11 years ago
Created attachment 227646 [details]
Screenshot
Attachment #227524 - Attachment is obsolete: true
Attachment #227645 - Flags: review?(alqahira) → review+
(Assignee)

Updated

11 years ago
Attachment #227645 - Flags: superreview?(mikepinkerton)
i think the vertical compression makes things too busy and hard to separate the differnt prefs visually. Agreement?
(Assignee)

Comment 12

11 years ago
Created attachment 230123 [details]
Expands space slightly
Attachment #227645 - Attachment is obsolete: true
Attachment #227645 - Flags: superreview?(mikepinkerton)
(Assignee)

Comment 13

11 years ago
Created attachment 230124 [details]
Screenshot
Attachment #227646 - Attachment is obsolete: true
(Assignee)

Updated

11 years ago
Attachment #230123 - Flags: superreview?(mikepinkerton)
Comment on attachment 230123 [details]
Expands space slightly

sr=pink
Attachment #230123 - Flags: superreview?(mikepinkerton) → superreview+
Whiteboard: [needs checkin]

Comment 15

11 years ago
Fixed trunk and branch.
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Keywords: fixed1.8.1
Resolution: --- → FIXED
Whiteboard: [needs checkin]
(Assignee)

Updated

11 years ago
Status: RESOLVED → VERIFIED
Keywords: verified1.8.1
(Assignee)

Updated

11 years ago
Keywords: fixed1.8.1
You need to log in before you can comment on or make changes to this bug.