-
Notifications
You must be signed in to change notification settings - Fork 823
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Leo]: Enable Leo in Polymer, and convert settings page #18156
Conversation
5c769da
to
120cfc0
Compare
A Storybook has been deployed to preview UI for the latest push |
080bff7
to
d369eb5
Compare
d369eb5
to
f8d3333
Compare
A Storybook has been deployed to preview UI for the latest push |
A Storybook has been deployed to preview UI for the latest push |
889ae03
to
ec1e074
Compare
A Storybook has been deployed to preview UI for the latest push |
ec1e074
to
23e39dc
Compare
A Storybook has been deployed to preview UI for the latest push |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm with few nits.
23e39dc
to
bfce596
Compare
A Storybook has been deployed to preview UI for the latest push |
bfce596
to
8f679b8
Compare
3576baa
to
66b0953
Compare
66b0953
to
16d2e5c
Compare
A Storybook has been deployed to preview UI for the latest push |
16d2e5c
to
fad603a
Compare
A Storybook has been deployed to preview UI for the latest push |
fad603a
to
f30ef99
Compare
A Storybook has been deployed to preview UI for the latest push |
A Storybook has been deployed to preview UI for the latest push |
Resolves brave/brave-browser#30051
This unblocks #16388, where our polymer overriding of
iron-icons
gets broken.WAITING ON brave/leo#285Submitter Checklist:
QA/Yes
orQA/No
;release-notes/include
orrelease-notes/exclude
;OS/...
) to the associated issuenpm run test -- brave_browser_tests
,npm run test -- brave_unit_tests
wikinpm run lint
,npm run presubmit
wiki,npm run gn_check
,npm run tslint
git rebase master
(if needed)Reviewer Checklist:
gn
After-merge Checklist:
changes has landed on
Test Plan: