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.

Sign up for GitHub

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

Jump to bottom

Applications: Backend/frontend - Indicate active context in the navigation #933

Closed
Tracked by #311
juliushaertl opened this issue Mar 15, 2024 · 0 comments · Fixed by #1080
Closed
Tracked by #311

Applications: Backend/frontend - Indicate active context in the navigation #933

juliushaertl opened this issue Mar 15, 2024 · 0 comments · Fixed by #1080
Assignees
@blizzz

Comments

@juliushaertl
Copy link
Member

juliushaertl commented Mar 15, 2024 •
edited
Loading

Follow up from #904 (comment)

We should:

  • Have a template response endpoint that is used for applications accessed from the app menu that will set the proper active navigation icon
  • We can pass over an initial state value there so the frontend knows when it has been accessed through the app navigation bar instead of just through tables directly
    • Maybe we want to hide the tables navigation by default then to focus on the current application
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@blizzz blizzz

Labels
None yet
Projects
📝 Office team
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

enh(App): navigation tweaks for Contexts nextcloud/tables
2 participants
@blizzz @juliushaertl