If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Updater dialog should size itself appropriately on mobile devices

RESOLVED FIXED in mozilla1.9.2b1

Status

()

Toolkit
Application Update
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: rstrong, Assigned: rstrong)

Tracking

Trunk
mozilla1.9.2b1
ARM
Windows Mobile 6 Professional
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(status1.9.2 beta1-fixed)

Details

Attachments

(6 attachments, 6 obsolete attachments)

63.09 KB, image/png
Details
338.08 KB, image/jpeg
Details
58.75 KB, image/png
Details
59.86 KB, image/png
Details
310.21 KB, image/png
Details
10.27 KB, patch
vlad
: review+
Details | Diff | Splinter Review
Spinoff of bug 507338.

The updater dialog info text is always one line and will cause the dialog to be too wide on some devices. Since the nv WinCE devices have had a few idiosyncrasies I'll take this bug and make sure it doesn't cause any problems on it.
Assignee: nobody → robert.bugzilla
Status: NEW → ASSIGNED
Created attachment 394229 [details]
screenshot of bug
Created attachment 394237 [details]
screenshot of proposed ui (margins are off)
Madhava, can I get your thoughts on what the updater ui should look like on Fennec? Thanks
OS: Windows CE → Windows Mobile 6 Standard
Created attachment 394440 [details]
WinMo screenshot example
We don't support winmo standard yet
OS: Windows Mobile 6 Standard → Windows Mobile 6 Professional
Created attachment 394452 [details]
screenshot - without dynamically moving progress bar

It should be easy enough to move the progress bar to the correct position. *crosses fingers*
Created attachment 394453 [details] [diff] [review]
patch in progress
Created attachment 394461 [details]
screenshot with dynamic resizing and placement

I've got it working... just need to clean it up a bit.
Attachment #394237 - Attachment is obsolete: true
Attachment #394452 - Attachment is obsolete: true
Created attachment 394463 [details]
 screenshot with dynamic resizing / placement and long text
Created attachment 394470 [details] [diff] [review]
patch in progress rev2
Attachment #394453 - Attachment is obsolete: true
Created attachment 394574 [details]
screenshot

Madhava, this is what I've come up with... I'd like to get a ui review before submitting the patch. Thanks
Attachment #394574 - Flags: ui-review?(madhava)
note to self... still need to remove the ok button.
Created attachment 395406 [details]
screenshots
Attachment #394470 - Attachment is obsolete: true
Attachment #394574 - Attachment is obsolete: true
Attachment #394574 - Flags: ui-review?(madhava)
Created attachment 395407 [details] [diff] [review]
patch in progress rev3
Created attachment 395430 [details] [diff] [review]
patch rev1
Attachment #395407 - Attachment is obsolete: true
Attachment #395430 - Flags: review?(vladimir)
Attachment #395430 - Flags: review?(vladimir) → review+
Attachment #395430 - Flags: approval1.9.2+
Pushed to mozilla-central
http://hg.mozilla.org/mozilla-central/rev/cdf69678b702
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Pushed to mozilla-1.9.2
http://hg.mozilla.org/releases/mozilla-1.9.2/rev/ae041c2c14a2
Keywords: fixed1.9.2
Target Milestone: --- → mozilla1.9.2b1
No reasonable way to test this so minusing in test suite
Flags: in-testsuite-
status1.9.2: --- → beta1-fixed
Keywords: fixed1.9.2
You need to log in before you can comment on or make changes to this bug.