Mobile notifications in nav bar, separate mobile navbar to its own component
1 unresolved thread
Compare changes
Files
25+ 4
− 1
@@ -252,6 +252,9 @@ const afterStoreSetup = async ({ store, i18n }) => {
The notifs button in the navbar opens a drawer from the right, closing the drawer marks notifications as read. With this change going between notifications/timeline/post-status is done with just one tap again without losing state in any of these. you don't even lose scroll position when you just want to check notifications, at the cost of some minor awkwardness when you try to over-scroll the notifications.
Some remaining awkwardness to be fixed:
One slightly questionable but also really useful change: keeping track of mobile layout in JS as well so you can actually do logic based on the layout. this was done here to avoid having 2 notifications components at the same time