10.5.2 statusbar text has weird display

VERIFIED FIXED in Camino1.6

Status

Camino Graveyard
General
VERIFIED FIXED
10 years ago
10 years ago

People

(Reporter: philippe (part-time), Assigned: Stuart Morgan)

Tracking

({fixed1.8.1.13})

unspecified
Camino1.6
x86
Mac OS X
fixed1.8.1.13

Details

Attachments

(2 attachments, 1 obsolete attachment)

21.24 KB, image/png
Details
v2
3.58 KB, patch
Mike Pinkerton (not reading bugmail)
: superreview+
Details | Diff | Splinter Review
(Reporter)

Description

10 years ago
Created attachment 302740 [details]
screenshot

the white 'shadow' is offset more than it used to.
(Assignee)

Comment 1

10 years ago
Created attachment 302744 [details] [diff] [review]
fix

I guess that answers the question of why I had to use an offset of 2 to get what appeared to be a single pixel of offset.

This doesn't seem worth special-casing; 10.5.[01] users can just get less etching.
Assignee: nobody → stuart.morgan
Status: NEW → ASSIGNED
Attachment #302744 - Flags: superreview?(mikepinkerton)
(Assignee)

Updated

10 years ago
Flags: camino1.5.6?
(Reporter)

Comment 2

10 years ago
(In reply to comment #1)
> Created an attachment (id=302744) [details]
> fix
> 
> I guess that answers the question of why I had to use an offset of 2 to get
> what appeared to be a single pixel of offset.

This might provide the answer:
http://islayer.com/blog/?p=255
see first comment

(the bug in WebKit/webcontent has long annoyed me; didn't know it affected chrome parts of apps).
(Assignee)

Comment 3

10 years ago
Created attachment 302747 [details] [diff] [review]
v2

Or, even better, I can finally find documentation about the right way to do this that I was sure they had mentioned at WWDC but couldn't find when I was looking for it.

This version should be right on all versions of 10.5, past and present.
Attachment #302744 - Attachment is obsolete: true
Attachment #302747 - Flags: superreview?(mikepinkerton)
Attachment #302744 - Flags: superreview?(mikepinkerton)
Comment on attachment 302747 [details] [diff] [review]
v2

sr=pink. me likey ;)
Attachment #302747 - Flags: superreview?(mikepinkerton) → superreview+
(Assignee)

Comment 5

10 years ago
Landed on trunk and MOZILLA_1_8_BRANCH.

I also switched MAC_OS_X_VERSION_MIN_REQUIRED to MAC_OS_X_VERSION_MAX_ALLOWED, since it's actually the SDK that we want that block to be conditional on.
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Keywords: fixed1.8.1.13
Resolution: --- → FIXED
Target Milestone: --- → Camino1.6

Comment 6

10 years ago
Thank you for fixing this so quickly.  And for a great Mac based browser.  Keep up the good work, Dev's.
(Reporter)

Comment 7

10 years ago
v. with 2.0a1pre (1.9b4pre 2008021420)
Status: RESOLVED → VERIFIED
(Assignee)

Updated

10 years ago
Flags: camino1.5.6?
You need to log in before you can comment on or make changes to this bug.