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

Enhance sidebar tooltip behavior and improve UI consistency #1412

Closed
wants to merge 2 commits into from

Conversation

nitin01-cmd
Copy link

Enhance Sidebar Tooltip Behavior and Improve UI Consistency
Fixes:

Improved sidebar tooltip behavior to ensure tooltips remain visible when necessary and disappear appropriately on mouseout.
Enhanced UI consistency for sidebar tooltips by properly positioning them relative to their parent elements.

Platforms this PR was tested on:
✅ Windows
⬜ macOS
⬜ Linux (specify distro)

Self-review checklist ✅ Self-reviewed changes for clarity and maintainability. ✅ Ensured variable names, code reuse, and readability are improved. ✅ Documented technical choices and potential concerns in code comments. ✅ Verified logic using automated tests where applicable. ✅ Commit history follows best practices with clear commit messages. ✅ UI appearance, responsiveness, and internationalization reviewed. ✅ Checked tooltips, button interactions, and error handling.

@zulipbot zulipbot added size: XL and removed size: L labels Mar 1, 2025
@nitin01-cmd
Copy link
Author

Title:
Enhance Tooltip Handling and Fix Null Checks in displayInitialCharLogo

Description:
Improved tooltip handling by ensuring proper display and hiding behavior.
Fixed potential null errors in displayInitialCharLogo by adding necessary checks.
Optimized getCurrentActiveServer to handle server tab instances more robustly.
Ensured the dnd (Do Not Disturb) button initializes correctly based on user settings.

@andersk
Copy link
Member

andersk commented Mar 1, 2025

This is full of syntax errors and doesn’t do anything.

@andersk andersk closed this Mar 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants