Closed Bug 811611 Opened 12 years ago Closed 10 years ago

[PopupManager][polish] Window.open progress indicator visual update.

Categories

(Firefox OS Graveyard :: Gaia::System::Window Mgmt, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED INVALID

People

(Reporter: alive, Assigned: alive)

Details

Attachments

(1 file)

Currently the progress is stolen from browser app, and Visual has a new design on the progress in https://bugzilla.mozilla.org/show_bug.cgi?id=796741
Window.open should use the white-background version as https://bugzilla.mozilla.org/show_bug.cgi?id=796741#c14 stated.
Component: Gaia → Gaia::System
Assignee: nobody → alive
Attached file PR 6389
NOTE: If blocking-basecamp+ is set, just land it for now.

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 
User impact if declined: 
Testing completed: 
Risk to taking this patch (and alternatives if risky):
Attachment #681374 - Flags: review?(timdream+bugs)
Attachment #681374 - Flags: approval-gaia-master?(21)
Comment on attachment 681374 [details]
PR 6389

r=me. The CSS only change is little and shouldn't break anything if visually checked.
Attachment #681374 - Flags: review?(timdream+bugs) → review+
Comment on attachment 681374 [details]
PR 6389

Hey can you land that in Gordon UX branch? https://github.com/gordonbrander/gaia/tree/ux

Any css changes that touch height/position should land in this branch and QA will do one pass on the UX branch on Thursday. Then it will be only one approval/review for all those changes. 

CC'ing Gordon.


Also you don't have to worry about such things right now. We have a bunch of people that are going to focus on polish/consistency issues, your skills will be much more useful at fixing blockers ;)
Attachment #681374 - Flags: approval-gaia-master?(21) → approval-gaia-master-
Component: Gaia::System → Gaia::System::Window Mgmt
Flags: approval-gaia-v1-
Closed as no PopupManager anymore.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: