-
Notifications
You must be signed in to change notification settings - Fork 6
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
LINK-2051 | hide instruction notifications from admin users #411
LINK-2051 | hide instruction notifications from admin users #411
Conversation
LINKEDCOMPONENTS-UI branch is deployed to platta: https://linkedcomponents-pr411-ui.dev.hel.ninja 🚀🚀🚀 |
e2e tests result is failed for https://linkedcomponents-pr411-ui.dev.hel.ninja 😿💢💥💥 |
f7ff0d9
to
e6dfd4a
Compare
LINKEDCOMPONENTS-UI branch is deployed to platta: https://linkedcomponents-pr411-ui.dev.hel.ninja 🚀🚀🚀 |
e2e tests result is success for https://linkedcomponents-pr411-ui.dev.hel.ninja 😆🎉🎉🎉 |
e6dfd4a
to
13a33b1
Compare
LINKEDCOMPONENTS-UI branch is deployed to platta: https://linkedcomponents-pr411-ui.dev.hel.ninja 🚀🚀🚀 |
e2e tests result is success for https://linkedcomponents-pr411-ui.dev.hel.ninja 😆🎉🎉🎉 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, though should be tested with a screenreader to ensure it knows about the popup being opened.
13a33b1
to
2b2ddfa
Compare
LINKEDCOMPONENTS-UI branch is deployed to platta: https://linkedcomponents-pr411-ui.dev.hel.ninja 🚀🚀🚀 |
e2e tests result is success for https://linkedcomponents-pr411-ui.dev.hel.ninja 😆🎉🎉🎉 |
2b2ddfa
to
79e36c0
Compare
LINKEDCOMPONENTS-UI branch is deployed to platta: https://linkedcomponents-pr411-ui.dev.hel.ninja 🚀🚀🚀 |
e2e tests result is success for https://linkedcomponents-pr411-ui.dev.hel.ninja 😆🎉🎉🎉 |
|
Tested Tooltip component with NVDA and I works as expected |
Description
Hide instruction notifications from admin users. Instead show tooltip instructions for them.
Note that it's intentional to display "Julkaisu" instruction for all the users.
Closes
LINK-2051
Screenshot