nextcloud / tables

🍱 Nextcloud tables app
https://apps.nextcloud.com/apps/tables
GNU Affero General Public License v3.0
143 stars 24 forks source link

feat: Add UI elements to modify navigation display #1295

Open enjeck opened 1 month ago

enjeck commented 1 month ago

Part of https://github.com/nextcloud/tables/issues/1177 and https://github.com/nextcloud/tables/pull/1193

Todo

enjeck commented 1 month ago

@nextcloud/designers Do you have any thoughts on how to go about the following?

We are trying to have settings where users can decide whether or not a Tables Application shows up in their navbar: Screenshot from 2024-08-22 09-49-52

For the owners of the application, there are three possible default options to choose from. I feel like this could be reworded or made more succinct. See the "Navigation bar display" section in the image: Screenshot from 2024-08-22 09-48-35

For share recipients, there's a checkbox. Not sure the wording or design is good enough: Screenshot from 2024-08-22 09-49-31

enjeck commented 1 month ago

So unless I misunderstand something, the settings seem can be simplified. As admins are also just users, they can use the navigation preference like everyone else.

@jancborchardt This is not possible now, from what I see. There's no option for an owner to modify just their visibility, without affecting share recipients. And I'm not sure why it's not supported. I'll clarify at https://github.com/nextcloud/tables/pull/1193 and see what can be changed

enjeck commented 6 days ago

So here are the challenges we've faced so far: Since navigation display setting is tied to shares, there is an additional share from the owner of an application to themselves. From discussions with @blizzz, we believe this is not good practice, but it's what it takes to get a better UX at the moment. The problem we have encountered now is that for owners of an application, the modifying the display does not work properly like with the sharees. So the plan now is: