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)
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.
Assignee | ||
Updated•12 years ago
|
Component: Gaia → Gaia::System
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → alive
Assignee | ||
Comment 1•12 years ago
|
||
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 2•12 years ago
|
||
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 3•12 years ago
|
||
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-
Updated•11 years ago
|
Component: Gaia::System → Gaia::System::Window Mgmt
Flags: approval-gaia-v1-
Assignee | ||
Comment 4•10 years ago
|
||
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.
Description
•