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

Modmail sidebar adjustments #595

Merged
merged 12 commits into from
Aug 24, 2022
Merged

Modmail sidebar adjustments #595

merged 12 commits into from
Aug 24, 2022

Conversation

creesch
Copy link
Member

@creesch creesch commented Aug 23, 2022

Basically the title. When the old sidebar is still in place it will look the same as it was, with the new sidebar the buttons are moved around.

image

Fixes #579. Fixes #581.

Copy link
Member

@Venefilyn Venefilyn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👀

extension/data/modules/newmodmailpro.js Outdated Show resolved Hide resolved
@creesch creesch marked this pull request as ready for review August 23, 2022 18:47
@creesch creesch changed the title Modmail sidebar adjustments, fixes #579 Modmail sidebar adjustments, fixes #579 & #581 Aug 23, 2022
@eritbh eritbh changed the title Modmail sidebar adjustments, fixes #579 & #581 Modmail sidebar adjustments Aug 24, 2022
@creesch creesch merged commit 873decb into master Aug 24, 2022
@creesch creesch deleted the modmail-sidebar-fixes branch August 24, 2022 06:16
@eritbh eritbh added this to the v6.0.2 milestone Aug 24, 2022
eritbh pushed a commit that referenced this pull request Sep 5, 2024
* TB buttons position based on sidebar type

* Use different events for old and new sidebar

* Remove extra css

* Fix getThingInfo username for new modmail sidebar

* Add avatar styling options

* Moar clicky! (Also fixes for currenty clicky)

* Clicky styling

* Reposition bracket buttons to fit un preview

* I don't like particles...

* Fix admin distinguish style in compact mode

* Also align admin distinguish without banner

* Adjust bracket buttons height
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

UI improvement options for new Modmail sidebar Toolbox buttons break modmail sidebar
3 participants