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

useNotify multiline does not work #7187

Closed
ghost opened this issue Feb 4, 2022 · 3 comments · Fixed by #7188
Closed

useNotify multiline does not work #7187

ghost opened this issue Feb 4, 2022 · 3 comments · Fixed by #7188
Labels

Comments

@ghost
Copy link

ghost commented Feb 4, 2022

What you were expecting:
notify(`Hello mate. It time to go.`,{multiLine:true,type:"info",autoHideDuration:50000,undoable:true})js to print

Hello mate.
It time to go.

What happened instead:
Hello mate. It time to go.

Steps to reproduce:
notify(`Hello mate. It time to go.`,{multiLine:true,type:"info",autoHideDuration:50000,undoable:true})

Environment

  • React-admin version: latest
  • Last version that did not exhibit the issue (if applicable):
  • React version: latest
  • Browser: chrome
  • Stack trace (in case of a JS error): none
@WiXSL
Copy link
Contributor

WiXSL commented Feb 4, 2022

@evergreen-trading-systems, thanks for reporting this.
But this is not a bug. New lines imply having \n character in your message to identify new lines, not dots (.).
So I'm closing this issue. Feel free to re-open it if you still find a way to reproduce it.

@WiXSL WiXSL closed this as completed Feb 4, 2022
@ghost
Copy link
Author

ghost commented Feb 4, 2022

I am reopening. You say use \n but I did. I opened this issue expecting you to at least test it or point to a working example. \n does not work. Nothing sensible or intuitive works.

@WiXSL
Copy link
Contributor

WiXSL commented Feb 4, 2022

Please, I'm just trying to help you. There is no need for that tone.

@WiXSL WiXSL reopened this Feb 4, 2022
@djhi djhi closed this as completed in #7188 Feb 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant