1016818 - [Bluetooth] Update to use gaia-header

Status: RESOLVED FIXED

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

Comment 1

[Yan Or]

Attachment: Pull request

Kevin,

Please redirect r? as needed.  Thanks.

Comment 2

[Kevin Grandon :kgrandon]

Comment on attachment 8429838 [details] [review]
Pull request

Hey, you should probably use the modules page (https://wiki.mozilla.org/Modules/FirefoxOS) to find the right reviewer, or use the history in github of the repo. In this case, I think it would make sense for Ian to review this perhaps.

Ian - could you please take a look at this? If you don't have time, feel free to delegate to me. Thanks!

Comment 3

[[e-mail], [e-mail]]

Comment on attachment 8429838 [details] [review]
Pull request

Hi Yan, 

I leave some comment on GitHub. Since we set the header property 'hidden' dynamically, the tag 'gaia-subheader' is not working for the original property operation. So we always see the showing header 'Paired devices' when no device are paired. Shell the component plan to have the ability? Thanks.

Comment 4

[Yan Or]

Comment on attachment 8429838 [details] [review]
Pull request

Hi Ian,

Resync the patch with the latest base and added support for 'hidden' in gaia-subheader.  Please review.  Thanks.

Comment 5

[[e-mail], [e-mail]]

Comment on attachment 8429838 [details] [review]
Pull request

Yan, the patch looks good for me without a nit on Github. Please check the comment on Github. Thanks. r=me

Comment 6

[[e-mail], [e-mail]]

Please check all tests on try-server are passed before land the patch.

Comment 7

[Wilson Page [:wilsonpage]]

Comment on attachment 8429838 [details] [review]
Pull request

LANDED (master) https://github.com/mozilla-b2g/gaia/commit/859f927016382b445e3b0005a60f5529dd4e5bd5