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
Restore clean-conversation-sidebar
feature
#4445
Restore clean-conversation-sidebar
feature
#4445
Conversation
Co-authored-by: Federico Brigante <me@fregante.com>
@cheap-glitch you need to take in account users with triage access only This is what I see on download-directory/download-directory.github.io#37 |
This comment has been minimized.
This comment has been minimized.
@fregante can you add @cheap-glitch to download-directory so he can test? |
Everything should work properly now (cf. new screenshots in first post), @fregante you can remove my access. Any other special cases this feature should handle? If not, I think it's done. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
🥳 |
@cheap-glitch Can you take a look at xojs/xo#338 its missing the label header. Thanks |
The "Label" heading is always hidden, except when the user can edit the sidebar. This PR doesn't change that |
Fixes #4238
Fixes #4258
Test URLs
Screenshots
This PR
Merged PR
Open issue with linked PR
Open issue with milestone and assignee
PR with no labels (user has triage access)
Issue with labels (user has triage access)