search

SolarArbiter / SolarArbiter.github.io

Contains the Solar Forecast Arbiter static website
https://solarforecastarbiter.org/
MIT License
9 stars 8 forks source link

update permissions reference table with sites and update permissions #184

Closed lboeman closed 3 years ago

lboeman commented 3 years ago

closes #183 Rendered here: https://lboeman.github.io/documentation/dashboard/administration/#permissions-reference-table

wholmgren commented 3 years ago

Thanks, this is good but I have a bunch of nitpicky things:

We usually talk about sites, then observations and forecasts, so I think it might help slightly to make sites the first line of the table.

Superscript 7 is a subscript.

Can you also reorganize the superscripts so that create is 1, delete is 2, and so on?

When a user sets up new permissions and roles, do they need to set up the update permission or is it linked to something else?

Can you make the table font a little smaller? That would help with scrolling. It would also be nice if the columns were slightly wider - for example it's hard to read the aggregate update column.

lboeman commented 3 years ago

When a user sets up new permissions and roles, do they need to set up the update permission or is it linked to something else?

The act of creating the role or permission does not grant a user permission to update it. An admin would need to grant them update permissions or, what seems more likely is they would just be an Admin and have all permissions by default. Was that what you were asking about?

Can you make the table font a little smaller? That would help with scrolling. It would also be nice if the columns were slightly wider - for example it's hard to read the aggregate update column.

I set the text of the table to 80% of the pages text size and set a min-width on the table. The minimum width causes the scrolling to stay, but the scroll bar is almost always visible now.