Skip to content

[docs] Add notistack demo to the snackbar page - #13685

Merged
oliviertassinari merged 4 commits into
mui:masterfrom
iamhosseindhv:master
Nov 27, 2018
Merged

oliviertassinari merged 4 commits into
mui:masterfrom
iamhosseindhv:master

Conversation

@iamhosseindhv

Copy link
Copy Markdown
Contributor

Following up this issue, I have added a simple example of notistack to Snackbar documentation page.

However, this PR is not fully functional since notistack needs @material-ui/core as a peer dependency. So I wanted to make sure if I can add @material-ui/core as a devDependency to package.json ?

Thanks.

@iamhosseindhv iamhosseindhv changed the title [docs] Add notistack live deme to Snackbar documentation page [docs] Add notistack live demo to Snackbar documentation page Nov 25, 2018
@joshwooding

joshwooding commented Nov 25, 2018 •

Copy link
Copy Markdown
Collaborator

There is some guidance here on solving the peer dependency issue: #13659 (comment) :)

@iamhosseindhv

Copy link
Copy Markdown
Contributor Author

Thanks @joshwooding. The issue is fixed now and live demo is working. ❤️

@oliviertassinari oliviertassinari added docs Improvements or additions to the documentation. scope: tooltip Changes related to the tooltip. labels Nov 27, 2018
@oliviertassinari oliviertassinari changed the title [docs] Add notistack live demo to Snackbar documentation page [docs] Add notistack demo to the snackbar page Nov 27, 2018
@oliviertassinari
oliviertassinari merged commit 8f31edc into mui:master Nov 27, 2018
@oliviertassinari

oliviertassinari commented Nov 27, 2018 •

Copy link
Copy Markdown
Member

@iamhosseindhv Thank you, keep up the good work with this project :).

@iamhosseindhv

Copy link
Copy Markdown
Contributor Author

Thanks @oliviertassinari. So great to have a demo on the official website.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Improvements or additions to the documentation. scope: tooltip Changes related to the tooltip.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants