Closed Bug 1172349 Opened 9 years ago Closed 9 years ago

[Settings] Convert remaining switches to use gaia-switch component

Categories

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

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: kgrandon, Assigned: kgrandon)

References

Details

Attachments

(1 file, 6 obsolete files)

      No description provided.
Attachment #8635013 - Attachment is obsolete: true
Summary: [Settings] Convert building block switches to use gaia-switch in root panel → [Settings] Convert remaining switches to use gaia-switch component
Attachment #8648502 - Attachment is obsolete: true
Comment on attachment 8651426 [details] [review]
[gaia] KevinGrandon:bug_1172349_settings_convert_network_operator_switch > mozilla-b2g:master

Fred - Going to start working my way through the last switch controls in the settings app and porting them over. Some of the implementations were pretty tricky, so I think maybe one panel at a time might be best for implementation/review purposes, but I'll try to not overload you in reviews.

Thanks for taking a look!
Attachment #8651426 - Flags: review?(gasolin)
Comment on attachment 8651426 [details] [review]
[gaia] KevinGrandon:bug_1172349_settings_convert_network_operator_switch > mozilla-b2g:master

Yeah there are some tricky part to support switch. Thanks for make it happen.

The patch works well on device.
Please make sure the related test are all green before landing.
Attachment #8651426 - Flags: review?(gasolin) → review+
Thanks. Network operator switch update checked in: https://github.com/mozilla-b2g/gaia/commit/0894b1d9d40a11ce74e77971bbfdb3a5674d3da7
please neglect USB storage switch at root panel, since that switch (and the special split case) will be removed and moved into its sub panel. we'll refresh the UI at bug 1194045
Depends on: 1194045
(In reply to Fred Lin [:gasolin] from comment #8)
> please neglect USB storage switch at root panel, since that switch (and the
> special split case) will be removed and moved into its sub panel. we'll
> refresh the UI at bug 1194045

Great, thank you for the info!
Comment on attachment 8652153 [details] [review]
[gaia] KevinGrandon:bug_1172349_settings_simpin_panel_switch > mozilla-b2g:master

Fred - please take a look at this when you get a chance. Thanks!
Attachment #8652153 - Flags: review?(gasolin)
Comment on attachment 8652153 [details] [review]
[gaia] KevinGrandon:bug_1172349_settings_simpin_panel_switch > mozilla-b2g:master

Thanks kevin for fixing these hard work. 

I leave some comment in github, let's discuss if there's some room to improve the implementation.

BTW, I suggest put these PR to separate issues so QA can test them as soon as possible when we resolve the issue.
Attachment #8652153 - Flags: review?(gasolin)
(In reply to Fred Lin [:gasolin] from comment #12)
> Comment on attachment 8652153 [details] [review]
> [gaia] KevinGrandon:bug_1172349_settings_simpin_panel_switch >
> mozilla-b2g:master
> 
> Thanks kevin for fixing these hard work. 
> 
> I leave some comment in github, let's discuss if there's some room to
> improve the implementation.

Thanks, I will address these.

> BTW, I suggest put these PR to separate issues so QA can test them as soon
> as possible when we resolve the issue.

Sure, let's track future work in different bugs if that's easier.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Attachment #8652153 - Attachment is obsolete: true
Attachment #8656384 - Attachment is obsolete: true
Attachment #8657485 - Attachment is obsolete: true
Attachment #8651426 - Attachment is obsolete: true
Comment on attachment 8678533 [details] [review]
[gaia] KevinGrandon:bug_1172349_settings_remaining_switches > mozilla-b2g:master

Fred - can you take a look when you get a chance? I think we are done with switches after this. Thanks for working with me on this!
Attachment #8678533 - Flags: review?(gasolin)
(Oops - I already had a PR open for this, but it was closed. Just going to re-open it for now as the bug better suits the current work).
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment on attachment 8678533 [details] [review]
[gaia] KevinGrandon:bug_1172349_settings_remaining_switches > mozilla-b2g:master

Looks good, just need fix some gij issues.

Thanks kevin for help improving the overall settings CSS maintainance!
Attachment #8678533 - Flags: review?(gasolin) → review+
Thanks for the review! Glad to be done with this one. In master: https://github.com/mozilla-b2g/gaia/commit/1efeb738fe3abe385c7ff8f9724df3a0fa706f6d
Status: REOPENED → RESOLVED
Closed: 9 years ago9 years ago
Resolution: --- → FIXED
Blocks: 1181023
Depends on: 1219619
Depends on: 1219378
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: