larry UI in portrait mode is cut off

RESOLVED FIXED

Status

Fennec Graveyard
General
RESOLVED FIXED
9 years ago
9 years ago

People

(Reporter: jmaher, Assigned: mfinkle)

Tracking

({uiwanted})

Trunk
ARM
Windows Mobile 6 Professional
uiwanted

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

9 years ago
In portrait mode on wince builds, I load up any website (mozilla.org), and click on the larry button (to the left of the url/title) and I see the larry UI pop up.

In landscape mode, I can view the whole UI, but in portrait mode I lose some of it.  Attempting to pan does not help my situation as only the web page pans.
(Reporter)

Updated

9 years ago
tracking-fennec: --- → ?

Updated

9 years ago
tracking-fennec: ? → 1.0b1-wm+
Keywords: uiwanted
(Assignee)

Updated

9 years ago
Assignee: nobody → mark.finkle
Created attachment 378782 [details] [diff] [review]
patch

this patch has everything but updated images for larry (48px and transparent background) and the lock/unlock images (24px and transparent).

Text has been updated and re-arranged based on Madhava's mockups:
http://www.grabup.com/uploads/f93b892110889832bf9901e8f892324e.png
http://www.grabup.com/uploads/e8d7be9ede80ccb0240870daf1fcdf3c.png

Screenshots:
http://people.mozilla.com/~mfinkle/fennec/screenshots/fennec-larry-unknown.png
http://people.mozilla.com/~mfinkle/fennec/screenshots/fennec-larry-verified.png
Comment on attachment 378782 [details] [diff] [review]
patch

I was waiting for new larry and lock/unlock images before requesting review. Those images are now part fo bug 494551, so I'm going to ask for review now, then update then images in bug 494551.
Attachment #378782 - Flags: review?(gavin.sharp)
(Assignee)

Updated

9 years ago
Blocks: 477620
Created attachment 381203 [details] [diff] [review]
patch 2 (no bitrot and using new theme images)

This patch merges to trunk and uses the new theme where appropriate (images and 'mm' padding)

It also reverts all but one ID to "identity-popup-*" (it actually hurt, but I did it)
Attachment #378782 - Attachment is obsolete: true
Attachment #381203 - Flags: review?(gavin.sharp)
Attachment #378782 - Flags: review?(gavin.sharp)
Attachment #381203 - Flags: review?(gavin.sharp) → review+
Comment on attachment 381203 [details] [diff] [review]
patch 2 (no bitrot and using new theme images)

>diff --git a/chrome/content/browser.js b/chrome/content/browser.js

>   setMode: function(newMode) {

>     // Update the popup too, if it's open
>-    if (this._identityPopup.state == "open")
>+    if (this._identityPopup.hidden == false)

!hidden rather than hidden == false?

>   handleIdentityButtonEvent: function(event) {

>+    if (this._identityPopup.hidden == true)

ditch the "== true"?

>+  handleEvent: function(event) {
>+    this.hide();

Add a comment saying this only handles blurs?

>diff --git a/themes/hildon/browser.css b/themes/hildon/browser.css

>-#identity-popup-content-box > description {

>-  white-space: pre-wrap;

>-#identity-popup-content-box > label {

>-  white-space: pre-wrap;

Looks like you forgot to move this to the common label/description rule? You did keep it in the wince version.
pushed with those changes:
http://hg.mozilla.org/mobile-browser/rev/21896505976e
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.