1177432 - [Settings] Introduce gaia-checkbox in developer panel

Status: RESOLVED FIXED

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

Description

[Kevin Grandon :kgrandon]

In preparation of moving away from building blocks, and onto web components. Also fixes a few issues with the gaia-checkbox component that will make it easier to use everywhere.

Comment 1

[Kevin Grandon :kgrandon]

Attachment: [Pull Request] Convert developer panel checkboxes to use gaia-checkbox

Comment 2

[Kevin Grandon :kgrandon]

Comment on attachment 8626192 [details] [review]
[Pull Request] Convert developer panel checkboxes to use gaia-checkbox

Hi Arthur - next up I would like to start porting checkboxes to use gaia-checkbox. Could you give this a review if you have a chance? Thanks!

Comment 3

[Arthur Chen [:arthurcc][inactive after 6/30]]

Comment on attachment 8626192 [details] [review]
[Pull Request] Convert developer panel checkboxes to use gaia-checkbox

Thanks for taking the effort, Kevin. Overall the patch is looking good, but the text color should be overrided to black.

And I noticed that the highlight was gone after using gaia components, just want to make sure this is by design. I'm fine with that if we are going to improve it from the component side.

Comment 4

[Kevin Grandon :kgrandon]

Comment on attachment 8626192 [details] [review]
[Pull Request] Convert developer panel checkboxes to use gaia-checkbox

Hi Arthur, thank you very much for the review. I've addressed your nit and added another commit with the color fix which I will squash. Regarding the :active highlight, we are currently doing this initial work and will swap out the components with the new versions which have a different hover state. (We would like to land this initial work for that, but our next release will definitely have an :active state)

Please take another look when you get a chance. Thanks!

Comment 5

[Arthur Chen [:arthurcc][inactive after 6/30]]

Comment on attachment 8626192 [details] [review]
[Pull Request] Convert developer panel checkboxes to use gaia-checkbox

r=me, thanks!

Comment 6

[Kevin Grandon :kgrandon]

In master: https://github.com/mozilla-b2g/gaia/commit/5997b406e77ea726fbd9047057a1c3504f6cd6d4