Yan Or Assignee
	Description • 10 years ago

      No description provided.

	Yan Or Assignee
	Comment 1 • 10 years ago

Kevin,

Please redirect r? as needed.  Thanks.

	Kevin Grandon :kgrandon
	Comment 2 • 10 years ago

Comment on attachment 8429838 [details]
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!

	iliu@mozilla.com, ianliu.moz@gmail.com
	Comment 3 • 10 years ago

Comment on attachment 8429838 [details]
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.

	Yan Or Assignee
	Comment 4 • 10 years ago

Comment on attachment 8429838 [details]
Pull request

Hi Ian,

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

	iliu@mozilla.com, ianliu.moz@gmail.com
	Comment 5 • 10 years ago

Comment on attachment 8429838 [details]
Pull request

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

	iliu@mozilla.com, ianliu.moz@gmail.com
	Comment 6 • 10 years ago

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

	Wilson Page [:wilsonpage]
	Comment 7 • 10 years ago

Comment on attachment 8429838 [details]
Pull request

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