orchestracities / auth-management-ui

An UI for Anubis and FIWARE-based platforms
https://master--62bcedd6cc1b1300eb9e1607.chromatic.com/
Apache License 2.0
0 stars 1 forks source link

row check box not displayed in resourceType table #139

Closed chicco785 closed 1 year ago

chicco785 commented 1 year ago

Describe the bug

The check box is not visible for each resource listed.

To Reproduce

Steps to reproduce the behavior:

  1. Access resource type page
  2. See error

Expected behavior

Check box are visible

Environment (please complete the following information):

Additional context

image

Valecant commented 1 year ago

They weren't rendered for design choice, still undecided if is necessary to remove them in the other tables (a part from the select all) for design reasons, I think that looks cleaner without them.

chicco785 commented 1 year ago

They weren't rendered for design choice, still undecided if is necessary to remove them in the other tables (a part from the select all) for design reasons, I think that looks cleaner without them.

  1. The weird thing is that they are not visible, but you can actually check them. So if the idea was not to have them, there's a bug.
  2. Functionally speaking, how is a user supposed to delete only a single entry?
Valecant commented 1 year ago

They weren't rendered for design choice, still undecided if is necessary to remove them in the other tables (a part from the select all) for design reasons, I think that looks cleaner without them.

  1. The weird thing is that they are not visible, but you can actually check them. So if the idea was not to have them, there's a bug.
  2. Functionally speaking, how is a user supposed to delete only a single entry?

You can select the row just clicking on it, and then deleting them using the delete button on the up right. On the tables with a checkbox implementation you can also select the row clicking on it and not necessarily on the checkbox. I don't really like the checkboxes repeated especially on mobile when you have a lot of data

chicco785 commented 1 year ago

They weren't rendered for design choice, still undecided if is necessary to remove them in the other tables (a part from the select all) for design reasons, I think that looks cleaner without them.

  1. The weird thing is that they are not visible, but you can actually check them. So if the idea was not to have them, there's a bug.
  2. Functionally speaking, how is a user supposed to delete only a single entry?

You can select the row just clicking on it, and then deleting them using the delete button on the up right. On the tables with a checkbox implementation you can also select the row clicking on it and not necessarily on the checkbox. I don't really like the checkboxes repeated especially on mobile when you have a lot of data

I got it know -> selection is made but the check box is not available, you can guess what is selected by the color difference. On my screen the difference between selected and not selected is not very visible (which is probably the cause of my initial not understanding of what's going on there), let's make it more visible.