Skip to content
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

UXC: Tables #3011

Closed
5 tasks done
mrCherry97 opened this issue Jun 25, 2024 · 1 comment · Fixed by #3118
Closed
5 tasks done

UXC: Tables #3011

mrCherry97 opened this issue Jun 25, 2024 · 1 comment · Fixed by #3118
Assignees
Labels
kind/feature Categorizes issue or PR as related to a new feature.

Comments

@mrCherry97
Copy link
Contributor

mrCherry97 commented Jun 25, 2024

Description

  • Inactive Tables should have a normal cursor. fix: cursor pointer on GenericList #3085
  • The names of table items shouldn’t be a link, because the whole rows are navigateable. (To discuss)
  • The orange lock icon should be clickable and a popover should appear, instead of having this hover interaction.
  • The locker of the protected resource should be next to the name instead of the special column.
  • When the Table is interactive, the cursor should be settled on the pointer fix: cursor pointer on GenericList #3085

Reasons
UX review.

Attachments
Figma

@mrCherry97 mrCherry97 added the kind/feature Categorizes issue or PR as related to a new feature. label Jun 25, 2024
@chriskari chriskari self-assigned this Aug 8, 2024
@chriskari chriskari linked a pull request Aug 9, 2024 that will close this issue
4 tasks
@vicslavova
Copy link

The Name/identifier of table items shouldn’t be a link if clicking on the row opens flexible column layout. Use link styling when the link sends to an external page or is navigating the user to a different area inside the Kyma dashboard.
If there is no flexible column layout and the name of the row is a link that opens another page, don't make the whole row clickable, only the link.

In all Kyma tables the names of the items are still links, we would suggest changing them to just black bold text since in this case the whole row is clickable and opens a second column. The only exception is the Namespaces table where we suggest to keep the link, remove the hover of the whole row and make only the link clickable.

Another example of this is the Events table - there the name should be changed to normal text styling, but we also have the "Involved Object" as a link and this is okay since it navigates to a different page.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes issue or PR as related to a new feature.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants