Closed Bug 783267 Opened 12 years ago Closed 12 years ago

Box.com auth window spinner is misaligned

Categories

(Thunderbird :: FileLink, defect)

15 Branch
x86
Windows 7
defect
Not set
normal

Tracking

(thunderbird16 fixed)

RESOLVED FIXED
Thunderbird 17.0
Tracking Status
thunderbird16 --- fixed

People

(Reporter: mconley, Assigned: Paenglab)

References

Details

Attachments

(3 files, 1 obsolete file)

When the Box.com auth window first comes up, and token negotiation begins, the spinner appears to be misaligned (at least on Windows).

See screenshots.
Attached image Misaligned spinner (obsolete) —
Attached patch patchSplinter Review
Instead of playing with margins it's better to use background-position. Then also the URL doesn't jump.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #652461 - Flags: review?(mconley)
Comment on attachment 652461 [details] [diff] [review]
patch

Review of attachment 652461 [details] [diff] [review]:
-----------------------------------------------------------------

I haven't built with this to see what these look like, but I trust your eye. r=me! Thanks Richard!
Attachment #652461 - Flags: review?(mconley) → review+
Keywords: checkin-needed
FYI the root of this bug is that the spinner icon is 16x16 which is smaller than the 22x22 size of the security level icon(s), hence the need for it to be positioned so that the transition between icons does not cause too much of a visual jump.
https://hg.mozilla.org/comm-central/rev/caf3e0f5733f
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Flags: in-testsuite-
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 17.0
Hrm - I probably should have tried the patch before giving it my r+. The spinner is still misaligned in today's Daily.

See upcoming screenshot.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Attached image Still misaligned
Attachment #652445 - Attachment is obsolete: true
Hmm, I checked with my updated Daily and the change isn't in it :/ Still using the margin.

Could this be because of the DONTBUILD on check-in?
Turns out I didn't see the fix because this patch didn't land until *after* today's Daily was built. *sigh*.

Sorry about that. Local testing confirms that this patch is indeed awesome.
Status: REOPENED → RESOLVED
Closed: 12 years ago12 years ago
Resolution: --- → FIXED
Comment on attachment 652461 [details] [diff] [review]
patch

[Approval Request Comment]
Regression caused by (bug #): 767327
User impact if declined: UX snafu
Testing completed (on c-c, etc.): ? 
Risk to taking this patch (and alternatives if risky): Small. Dependent on approval‑comm‑aurora of bug 767327
Attachment #652461 - Flags: approval-comm-aurora?
Comment on attachment 652461 [details] [diff] [review]
patch

a=me, since it seems small and innocuous, and lining things up is one of my crazy-making things…  ;)
Attachment #652461 - Flags: approval-comm-aurora? → approval-comm-aurora+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: