-
Notifications
You must be signed in to change notification settings - Fork 37
Fix tooltip text #1200
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
Fix tooltip text #1200
Conversation
Hi @priya-tik |
Use Run test server using develop.opencast.org as backend:
Specify a different backend like stable.opencast.org:
It may take a few seconds for the interface to spin up. |
This pull request is deployed at test.admin-interface.opencast.org/1200/2025-05-22_09-40-37/ . |
@Arnei Could you please add the label "type:visual-clarity" or "type:usability" so the PR passes the label check? |
Note: #1184 seems to address the same issue. |
#1184 is not closed, as can be seen by the green "Open" label under the title. Furthermore, a PR being open for a couple weeks also does not imply that the PR is closed or unwanted. |
ok |
Please make sure your git history is clean and that each PR only contains the commits that are relevant to the issue you are solving. For example, this PR also contains commits that are used to solve #1197. This PR should not contain those commits, unless there is a good reason for it. |
This pull request has conflicts ☹ |
This pull request has conflicts ☹ |
45622eb
to
f3e5c7a
Compare
This pull request has conflicts ☹ |
c2fcc05
to
1231a27
Compare
This pull request has conflicts ☹ |
Closing this PR as the changes for this issue has been already merged... Thank you @priya-tik... |
This PR fixes incorrect tooltips on delete buttons across the Users, Groups, and Access Policies tabs.
The tooltip previously said "Open details", which was misleading. Now it shows "Delete user", "Delete group", etc., as appropriate.