626099 - Sync setup wizard: No space between links on manual sign in page

Status: RESOLVED FIXED

(Firefox :: Sync, defect)

Description

[u60234]

Attachment: screenshot

In bug 601019, two text links, "Show me how." and "I have lost my other device.", were placed together at the bottom of the "I don't have the device with me" Setup pane. However, there is no space between the two links. So unless the line with the two links are long enough to wrap and push the second link to a new line, they are joined together.

Steps to reproduce:
1. In the Sync Setup dialog, click "Connect" under "I already have a Firefox Sync account".
2. Click "I don't have the device with me".

Result:
Depending on platform and font size, I guess, the two links at the bottom can be joined together.

Comment 1

[Philipp von Weitershausen [:philikon]]

That teaches me not testing bug 601019 on Windows.

Comment 2

[Philipp von Weitershausen [:philikon]]

Attachment: v1

Add a non-breaking space between the two labels. Because, of course, a regular space doesn't work. Or the copious amounts of whitespace that's around the labels already.

XUL, I swear, one of these days...

Comment 3

[Philipp von Weitershausen [:philikon]]

(In reply to comment #1)
> That teaches me not testing bug 601019 on Windows.

Btw, I remember now that I did test this on Windows 7... but like on OSX the label appears on the next line there.

Comment 4

[Mike Connor [:mconnor]]

Comment on attachment 504222 [details] [diff] [review]
v1

<spacer/> ? <separator/> ?

No strings for whitespace, especially now!

Comment 5

[Philipp von Weitershausen [:philikon]]

(In reply to comment #4)
> <spacer/> ? <separator/> ?

Hmm, for some reason I thought the kind of space would have to be l10n'able. Maybe I was overthinking this ;)

Comment 6

[Philipp von Weitershausen [:philikon]]

Attachment: v2

Now with 100% less strings and 100% more <spacer/>s

Comment 7

[Philipp von Weitershausen [:philikon]]

http://hg.mozilla.org/mozilla-central/rev/696096dc3c8c