Closed Bug 916660 (app-modal-dialog) Opened 7 years ago Closed 7 years ago
[Window Management] refactor modal dialog, attach to app
Just as current appError class, attach the modal dialog to appWindow and then we won't need to maintain the origin info anymore. And we could remove the DOM of modal dialog from index.html to reduce load time.
No longer blocks: task-manager
This would be done in bug 907013. Need finetune here
Assignee: gduan → alive
The weird thing is I referenced index.html and change class from 'confirm' back to 'affirmative', it make crash....
Assignee: alive → gasolin
Summary: [Window Management] Rewrite modal dialog, attach to appWindow → [Window Management] refactor modal dialog, attach to appWindow
1. move elementClasses to similar structure of app_authentication_dialog, 2. simplify html render tags
Comment on attachment 8347117 [details] [review] pull request redirect to github Please do the opposite: Move element-fetch into _fetchElements and remove render.
Comment on attachment 8347117 [details] [review] pull request redirect to github updated, please kindly review it again
Comment on attachment 8347117 [details] [review] pull request redirect to github Pay attention to travis
Attachment #8347117 - Flags: review?(alive) → review+
rebase and check result ok merged to gaia-master https://github.com/mozilla-b2g/gaia/commit/2459080fde148b241ce46378a7009676a244ed47
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.