improve toaster #6543
improve toaster #6543
Conversation
✅ Deploy Preview for reliable-cocada-166884 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Position of toaster should be improved |
There was a problem hiding this comment.
Looks almost fine to me. @ninja-fire Can you check if we can make the toaster color visibility more user friendly?
yann300
force-pushed
the
improve_toaster
branch
from
63a53a2 to
b1e4955
Compare
yann300
force-pushed
the
improve_toaster
branch
from
6783d80 to
48432e2
Compare
Aniket-Engg
left a comment
Aniket-Engg
left a comment
There was a problem hiding this comment.
Please fix the UI for cancel icon.
I think we can keep the text color to white and update background color as per the toaster type.
| timeOutId, | ||
| message: shortTooltipText | ||
| // Show toast using Sonner | ||
| const duration = props.timeOut || 120000 |
There was a problem hiding this comment.
default value should be 1-2 seconds only
yann300
force-pushed
the
improve_toaster
branch
from
b616ab2 to
03172bb
Compare
|
View your CI Pipeline Execution ↗ for commit 7fffa2f
☁️ Nx Cloud last updated this comment at |
|
✅ E2E tests passed (workflow: web) Last run: Mon, Dec 8, 2025, 04:44 PM UTC All tests are now passing! Previous failures have been resolved. |
yann300
force-pushed
the
improve_toaster
branch
4 times, most recently
from
d2c01cc to
a6c7fe4
Compare
yann300
force-pushed
the
improve_toaster
branch
2 times, most recently
from
db20e22 to
6c4ff77
Compare
yann300
force-pushed
the
improve_toaster
branch
from
6c4ff77 to
b056bc7
Compare
| }) | ||
| }, | ||
|
|
||
| /* |
There was a problem hiding this comment.
why have you commented this e2e?
Removed commented-out test case for loading compiler from URL.
No description provided.