Admin panel: unclear naming and buttons

[sgaist]

I find the admin panel is a bit confusing when it comes to modifying items shown there.

I am rather used to have an Edit button that indicates that something can be changed.
In the case of the admin panel Update is used.
While technically, both indicates some form of changes, using Update is rather not intuitive especially since we are dealing with a system running on Kubernetes.
Most of the time, when I am using an Update button, it triggers a change in a live system such as a firmware upgrade or similar operations. Therefore, I would suggest modifying the text to use Edit rather than Update.

Another related issue is the Update buttons themselves. The Session thresholds part has a dark grey button while the rest (quota and classes) seem to have a white button. I am writing seem because just looking at the UI, only the text is visible but when clicking on it, one can see the button activation color.
I would suggest to make them all similar to the delete button: have a pen icon to hint on the action and a maybe a grey outline. That would make things a bit clearer.

[lorenzo-cavazzi]

Hey @sgaist , thanks for sharing that!

For the colors/styling, we are aware of the problem. The admin page was using the old color scheme, and we should update that (see this issue #3720). We will try to get there in the next couple of weeks.

For the Update vs Edit issue: I assume you are referring to the Compute Resource section, where Update is used instead of Edit. Thanks for reporting that! The rest of the Admin page, as well as the rest of the UI, seems to be using "Edit" for opening a modal (or enabling inputs) to allow editing, and then an "Update" button to confirm the change. I'll add that fix to the above issue

[lorenzo-cavazzi]

@sgaist Issue #3720 on https://github.com/SwissDataScienceCenter/renku-ui has been updated to include the changes to the button text.

Feel free to close this discussion if the issue now include everything, or further comment if I left anything out. Thanks again for reporting these problems 🙂

[sgaist]

#3720 Looks good !

Thanks :-)