Closed Bug 801532 Opened 12 years ago Closed 7 years ago

[system] Refactor/Cleanup building block in system app

Categories

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

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(blocking-basecamp:-, b2g18?)

RESOLVED WONTFIX
blocking-basecamp -
Tracking Status
b2g18 ? ---

People

(Reporter: alive, Unassigned)

References

Details

Building block is used a lot in system app, but is in a mess now. The work need to do: 1. Add these to apps/system/index.html: /shared/style/action_menu.css /shared/style/confirm.css /shared/style/status.css 2. Remove/Delete these outdates css: style/modal_dialog/prompt.css style/themes/default/core.css style/themes/default/menus.css style/themes/default/buttons.css style/themes/default/banner.css 3. Adjust the DOM and then make sure modal dialog works. 4. Adjust the DOM and then make sure authentication dialog works. 5. Adjust the DOM and then make sure system banner works. 6. Adjust the DOM and then make sure permission dialog works. 7. Adjust the DOM and then make sure list menu works. 7.1 Make sure contextmenu works 7.2 Make sure activitymenu works 8. Make sure value selector still works. Attention: 1. per @kaze, the value selector itself would not be container in the shared dir. 1.1 But it still conflicts with current shared bb styles! 2. add `onsubmit="return:false"` to all `form` element.
blocking-basecamp: --- → ?
The policy for building block work is now that it's set basecamp-blocking- and requires driver approval to get landed. With regards to this particular issue QA have asked if they can spend some time testing any patches before they land because they're in the system app.
blocking-basecamp: ? → -
Component: Gaia → Gaia::System
I think we should try to add in shared/style/ value_selector.css , date_picker and time_picker as it was the inital approach. Even is is only used in the system application. This will give us more freedom/control to update it and will allow easiest/unified customizations for OEM as other similar bb belongs to shared folder.
I recommend to open a bug for "Move value selector-liked bb out of system app", and mark the dependency on this bug.
Alive, make sense moving them out to shared/style, as it will be easier to customise the skin, or add app specific needs.
(In reply to Alive Kuo [:alive] from comment #3) > I recommend to open a bug for "Move value selector-liked bb out of system > app", and mark the dependency on this bug. Alive, I just filed bug 870747 according to your suggestion.
No longer blocks: 846595
Firefox OS is not being worked on
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.