-
Notifications
You must be signed in to change notification settings - Fork 23
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 misplaced themes details menu navigation buttons #563
Conversation
This pull request is deployed at test.admin-interface.opencast.org/563/2024-06-04_11-02-23/ . |
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. |
@dennis531 would that be also good to nowrap the text in the nav items? In this way it looks even better. What do you think? |
|
There is still a continuous line below the "arrow" that points to the current tab that doesn't exist on the old admin-ui. |
4de3c1e
to
de9f287
Compare
This pull request is deployed at test.admin-interface.opencast.org/563/2024-06-04_12-37-48/ . |
de9f287
to
8a98bc2
Compare
This pull request is deployed at test.admin-interface.opencast.org/563/2024-06-04_12-39-15/ . |
I like the |
Looks like people are happy with this, merging. |
Closes #551