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.

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

Sidebar toggle not shown when you start in results view #2531

Open
Chartman123 opened this issue Jan 28, 2025 · 2 comments · May be fixed by #2532
Open

Sidebar toggle not shown when you start in results view #2531

Chartman123 opened this issue Jan 28, 2025 · 2 comments · May be fixed by #2532
Labels
1. to develop Accepted and waiting to be taken care of bug Something isn't working javascript Javascript related ticket
Milestone

Comments

@Chartman123
Copy link
Collaborator

Chartman123 commented Jan 28, 2025

Please use the 👍 reaction to show that you are affected by the same issue. Please don't comment if you have no relevant information to add!

Describe the bug
If you start to view a form in the results view, the sidebar toggle button isn't shown.

Expected behavior
Toggle should always be shown when you have the permission to edit the forms settings/sharing

Nextcloud (please complete the following information):

  • Forms-Version: all versions since 4.3.0
@Chartman123 Chartman123 added 0. Needs triage Pending approval or rejection. This issue is pending approval. bug Something isn't working javascript Javascript related ticket 1. to develop Accepted and waiting to be taken care of and removed 0. Needs triage Pending approval or rejection. This issue is pending approval. labels Jan 28, 2025
@Chartman123 Chartman123 added this to the 5.0 milestone Jan 28, 2025
@Chartman123 Chartman123 added the good first issue Good for newcomers label Jan 28, 2025
@Chartman123
Copy link
Collaborator Author

Chartman123 commented Jan 28, 2025

@susnux any idea why this happens and why it's working at all in both other views?

v-if="showSidebarToggle"

I mean we're never setting showSidebarToggle anywhere so it should always be undefined and thus false, shouldn't it?

Edit:

Ok, seems like we're not using that button at all but the built-in toggle button of the NcAppSidebar component. I'll remove the button.

Edit 2:

The button re-appeared in our code, probably during some fixing of merge conflicts when we switched the code styling to prettier. However this has no influence on the missing sidebar toggle in the results view.

@Chartman123 Chartman123 changed the title TopBar doesn't show sidebar toggle when you start in results view Sidebar toggle not shown when you start in results view Jan 28, 2025
@Chartman123 Chartman123 linked a pull request Jan 28, 2025 that will close this issue
@Chartman123 Chartman123 removed the good first issue Good for newcomers label Jan 28, 2025
@susnux
Copy link
Collaborator

susnux commented Jan 29, 2025

The sidebar toggle is part of the sidebar nowadays, I think we can remove that button at all from the TopBar.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Accepted and waiting to be taken care of bug Something isn't working javascript Javascript related ticket
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants