Closed
Bug 627166
Opened 14 years ago
Closed 14 years ago
Sync setup wizard: captcha dialog in some locales doesn't show Next button
Categories
(Firefox :: Sync, defect)
Firefox
Sync
Tracking
()
VERIFIED
FIXED
Tracking | Status | |
---|---|---|
blocking2.0 | --- | final+ |
People
(Reporter: tracy, Assigned: philikon)
References
Details
(Whiteboard: [hardblocker][has patch in bug 609160])
Attachments
(4 files)
seen with Firefox Sync 1.6.2 on Firefox 3.6.13 on Mac.
Buttons in lower right not visible. Particularly the Next button.
Hitting enter here does move through the dialog. So not entirely stuck.
Reporter | ||
Comment 1•14 years ago
|
||
fr on XP is partially affected by this bug. just a bit of the Next button shows.
OS: Mac OS X → All
Hardware: x86 → All
Reporter | ||
Updated•14 years ago
|
Summary: captcha dialog to small with ja-JP → captcha dialog in some locales doesn't show Next button
Assignee | ||
Comment 2•14 years ago
|
||
This is addon (3.x) only or does this affect Firefox 4.0 as well?
Reporter | ||
Comment 3•14 years ago
|
||
Checked ja-JPmac on beta11, this bug is present there as well.
Assignee | ||
Comment 4•14 years ago
|
||
Thanks. I think there's two things to this:
1. In the addon UI we made the setup title text wrap (bug 608807) but we never ported that to the Firefox 4.0 UI (bug 609160). We should fix that. I believe that will fix the fr locale here. Tracy, could you check whether the fr locale has the same problem in the addon?
2. As far as I can tell, the Japanese string doesn't contain any spaces so even if we fixed bug 609160 (which we should!), it probably wouldn't wrap. I'm not entirely sure what the best course of action is here. CCing Axel for some input.
blocking2.0: --- → ?
Depends on: 609160
Summary: captcha dialog in some locales doesn't show Next button → Sync setup wizard: captcha dialog in some locales doesn't show Next button
Whiteboard: [softblocker?]
Reporter | ||
Comment 5•14 years ago
|
||
fr on XP was with the add-on. I'd guess there are other locales affected. I can't check them all. 2 of 4 tier 1 locales affected isn't good.
Assignee | ||
Comment 6•14 years ago
|
||
(In reply to comment #5)
> fr on XP was with the add-on.
Oh right. Never mind :)
Comment 7•14 years ago
|
||
Can we re-use the concepts in the preferences dialog crop mozmill test to create a similar automated test for this UI? http://hg.mozilla.org/qa/mozmill-tests/file/default/firefox/l10nTests/testCropped has sources.
Also, do we know what the problem is right now? I haven't done any sync setup things in a while, so I'd need STR. Also, I'm doing FOSDEM this weekend/travel on monday. (Same for Henrik, if you're wondering.)
Comment 8•14 years ago
|
||
Basically, extra-long wizard title. Not sure what the right fix is. (Just crop over-long labels? Provide a different screen?)
Comment 9•14 years ago
|
||
Blocks Sync in Japan, blocks the product!
blocking2.0: ? → final+
Whiteboard: [softblocker?] → [hardblocker]
Comment 10•14 years ago
|
||
FWIW, line wraps don't require latin spaces, there's more to scripts that i18n line wrapping can handle. We should definitely try to enable the line to wrap. I assume it doesn't, that is.
Comment 11•14 years ago
|
||
Axel, is there a best practice for wizards like this? We're just using a straight wizard element, albeit with some extra styling.
Do other wizards wrap/crop?
Comment 12•14 years ago
|
||
No idea, sadly.
Assignee | ||
Comment 13•14 years ago
|
||
(In reply to comment #11)
> Do other wizards wrap/crop?
They don't. This is a problem with wizard.xml. Fortunately I have a fix which I will upload in bug 609160.
Assignee | ||
Comment 14•14 years ago
|
||
Here's the sync setup wizard running without syncSetup.css and syncCommon.css. Proof that this isn't our wizard styling messing things up.
Assignee | ||
Comment 15•14 years ago
|
||
Patch uploaded to bug 609160, pending review from Enn. It fixes the problem at its core, the wizard widget binding. Once we agree that this is the right way to solve this problem, we should probably move the blocking flag over to that bug.
Whiteboard: [hardblocker] → [hardblocker][has patch in bug 609160]
Assignee | ||
Comment 16•14 years ago
|
||
Assignee | ||
Comment 17•14 years ago
|
||
I just landed bug 609160. Do we do localized nightlies or how can we verify that this issue is solved in Japanese builds?
Comment 18•14 years ago
|
||
Localized fx4 nightlies are on http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/latest-mozilla-central-l10n/.
Updated•14 years ago
|
Assignee: nobody → philipp
Comment 19•14 years ago
|
||
Yep, this is fixed, tested on mac/win nightlies.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Reporter | ||
Updated•14 years ago
|
Status: RESOLVED → VERIFIED
Reporter | ||
Comment 20•14 years ago
|
||
Philipp, this bug is still in the extension. Will bug 609160 not be ported to the branches?
Assignee | ||
Comment 21•14 years ago
|
||
(In reply to comment #20)
> Philipp, this bug is still in the extension. Will bug 609160 not be ported to
> the branches?
Probably not. I've reopened bug 608807.
Comment 22•13 years ago
|
||
Still in Firefox 13
Comment 23•13 years ago
|
||
Let's just kill the captcha already...
Updated•6 years ago
|
Component: Firefox Sync: UI → Sync
Product: Cloud Services → Firefox
You need to log in
before you can comment on or make changes to this bug.
Description
•