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

Support for Multiline? #26

Closed
araneta opened this issue Feb 28, 2017 · 4 comments
Closed

Support for Multiline? #26

araneta opened this issue Feb 28, 2017 · 4 comments

Comments

@araneta
Copy link

araneta commented Feb 28, 2017

I tried to show a notification notify.show(errors.join('
'),'error'); but it show weird notification. is this module support multiline
?

@jesusoterogomez
Copy link
Owner

I have to test if <br/> is compatible with this module. But my initial thought is that it's not. There was a PR created for supporting multiline text (at least 2 lines of it). but it wasn't merged due to the approach taken, since it was going to affect the single line text rendering.

@araneta
Copy link
Author

araneta commented Mar 5, 2017

so what is the solutions for this? thanks

@felipebritor
Copy link

@araneta, I've managed to do it by changing the toast span css:

#notification-wrapper span {
  display: inline-block !important;
}

@stephenfeather
Copy link

stephenfeather commented Oct 15, 2017

The css hack works as long as I don't have 3 lines of text. If we stretch into 3 lines, then when the toast clears, it leaves about half a line exposed. May explain part of the question I had in #44. Targeting mobile, so a lot of stuff will probably stretch to 3 lines :)

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

No branches or pull requests

4 participants