nextcloud / tables

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

enh(Contexts): set active navigation entry #1037

Closed blizzz closed 4 months ago

blizzz commented 5 months ago

This is sort of a PoC actually. Addresses #933

I looked at the backend options first. The problem is that we are currently working with a set anchor, i.e. example.com/index.php/apps/tables/#/application/28, so that is no information that we have on the backend. We'd either have to change this behaviour, or to avoid completely reloading the Tables app, having a mechanism that works with both, an URL that can be understood by the server, and the inside-app URL with the anchor.

Nevertheless, I looked at the frontend bits for a quick win. And there is the quick win:

Screencast_20240430_113348.webm

It is not ideal though, because

What do you think?

blizzz commented 5 months ago

Rebased to resolve conflicts.

While I'd only consider that a workaround, i think this is good enough for now.

Definitely not more than a workaround, that's what i tried to say :)

blizzz commented 4 months ago

@juliushaertl keep it for 0.8 or also bring it down to 0.7?

blizzz commented 4 months ago

/backport to stable0.7