-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
A11y[Op#51022] Replace New Storage "magic form" with more accessible ActionMenu #14251
Conversation
46b4ad5
to
e384eb9
Compare
fa0baf4
to
4714982
Compare
@akabiru telling from your videos it looks great! |
4714982
to
47de181
Compare
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.
Hi @akabiru
Great job! It feels so much more intuitive than before! 🥳 I only have one minor remark, other than that, it is good to go 👍
) { t("storages.instructions.new_storage_link_text") } | ||
).html_safe | ||
t("storages.instructions.new_storage", | ||
provider_link: ::OpenProject::Static::Links[:storage_docs][:"#{@storage.short_provider_type}_setup"][:href], |
Check notice
Code scanning / Brakeman
Unescaped model attribute. Note
Co-authored-by: Henriette Darge <[email protected]>
ae97545
to
cbac21c
Compare
https://community.openproject.org/work_packages/51022
+ Storage
buttonno-magic-demo.mp4
select-provider-edge-case.mp4
edit-page-provider-label.mp4