Notifications - "unread only" filter

like the last image we can give option for that too in that dropdown

1 Like

Thanks for the mockups @Ahmed_Gagan, how would you see this work at /my/notifications? Or are you envisioning this only in the user menu?

2 Likes

Should I do /my/notifications page in a separate PR

I’d make seperate PRs. The design for a full-page interface is likely to be different from one that works in a narrow-width dropdown.

3 Likes

So, ill focus on user menu dropdown in this PR and make another PR for full page interface. thoughts on the mockup?

I’m not one with an eye for UX design, best to wait to hear from one of our designers.

1 Like

I would start with the full notifications page PR first. Once that’s done, we can revisit the idea of adding the filters to the notification panel in the header.

We have some concerns that adding the filters to the notification panel in the header is unnecessary at this point.

7 Likes

Okay i will focus on that first, and will send you mockups of that page soon :slightly_smiling_face:

5 Likes


Thoughts? @Johani, @jomaxro

4 Likes

Ooo I like it! Nicely done.

4 Likes

so is it final? May i start working on it?

Sure, but make sure you use select kit there and not a custom control

3 Likes

sure, ill use select kit :slightly_smiling_face:

1 Like

Ive used select kit and made a PR.

11 Likes

Ive completed this task and my pull request is merged.

7 Likes

@Ahmed_Gagan

One thing I noticed - it currently doesn’t pull all unread notifications. It only filters the first loaded ‘page’ of them, but if you have more unread ones in the past, you have to scroll down in the All view and only then switch to unread ones.

It would be fun if switching to Unread filter would load all unread notifications and not just the unread ones from the first page, if that makes sense :slight_smile:

EDIT.
I’ve just found out another bug, I think.
When you middle click in this view:

It should open a topic in a new tab, but instead it clears the notification without opening any new tabs.

4 Likes

I’ll test it once again and will get back to you :slightly_smiling_face:

3 Likes

Very excited to see this rolled out!

Thanks to everyone incubating this since January 2016 :heart_eyes:

4 Likes

Started working on these changes will be submitted soon.

8 Likes

I just merged @Ahmed_Gagan’s PR for this, and it’s working nicely for me on Meta :heart_eyes:

I’ll get that change deployed to your site in the next couple of hours @Maciej_Kuzmicz

9 Likes