Skip to content
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

Web SDK 109115 changes: #13

Merged
merged 1 commit into from Feb 10, 2016
Merged

Web SDK 109115 changes: #13

merged 1 commit into from Feb 10, 2016

Conversation

jasonpang
Copy link
Contributor

  • Allow customization of notify button's main button colors
    • Can customize badge colors
    • Cannot customize text hover popup colors
    • Cannot customize dialog popup colors
  • Add detection when manifest.json is served from a different origin (e.g. a CDN)
    • Throws a friendly error message on the console

Review on Reviewable

- Allow customization of notify button's main button colors
    - Can customize badge colors
    - Cannot customize text hover popup colors
    - Cannot customize dialog popup colors
- Add detection when manifest.json is served from a different origin (e.g. a CDN)
    - Throws a friendly error message on the console
@gdeglin
Copy link
Member

gdeglin commented Feb 10, 2016

Reviewed 5 of 5 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.


Comments from the review on Reviewable.io

gdeglin added a commit that referenced this pull request Feb 10, 2016
@gdeglin gdeglin merged commit 7267887 into master Feb 10, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants