Improve Ui for mobile size devices#2087
Conversation
|
@nikita-9112 is attempting to deploy a commit to the PRIYANSHU DOSHI's projects Team on Vercel. A member of the Team first needs to authorize it. |
GSSoC Label Checklist 🏷️@Priyanshu-byte-coder — please apply the appropriate labels before merging: Difficulty (pick one):
Quality (optional):
Validation (required to score):
|
|
The changes look good overall, but wrapping the entire right section (notification bell, theme toggle, sign-out button) in |
|
hi @Priyanshu-byte-coder , now I rearrange the code and make the signout and notification icon visible on mobile size without toggling. Please review it and tell me if any changes or improvement is required. |
|
Thank you for the contribution! However, hiding the right action bar (Settings, Notifications, Theme Toggle, Sign Out) on mobile with The right approach for mobile is to make these controls accessible in a different way (e.g., in the mobile nav drawer, a hamburger menu, or by making the toolbar scrollable/collapsible) — not to hide them entirely. Please revise the approach to ensure all controls remain accessible on mobile. |
|
Hey, @Priyanshu-byte-coder , As you mentioned, I have made signout and notification buttons visible on mobile screen without the need of clicking the hamberger menu icon. When user clicks that hamberger menu icon the theme, username and shortcut button will be visible. Means the toggling functionality is only applicable for username, theme, and shortcut button , not for signout and notification icon. So please explain me the functionality of hamberger menu you want. |
Summary
Improved the UI for mobile and tablet screen size devices.
Closed #1152
on mobile size screen
on tablet size screen
on desktop size screen
Let me know if any changes are requied.