Page MenuHomePhabricator

[Notifications] Add read status filtering
Closed, ResolvedPublic

Description

Filter the displayed notifications by read status.

  • Add toolbar filter button with two states: unfilled (if filters not enabled) and filled (if filters are enabled)
  • Create modal view with All, Read, and Unread filter options, as well as Notification Type toggle options.
  • Persist filter selections to UserDefaults
  • Update filter icon to be filled image when the filter options are Read or Unread (not All).
    • We also need to consider the Notification Types filter toggles in this logic. Confirm with design (see question below).
  • Update status bar text (actual content in forthcoming task)

Product/Design Needs:

[1] Is it intended for your Read Status Filters and Inbox Filters to be persisted between app launches?

Ideally, yes. But it's okay if it doesn't persist after a hard restart of the app.

[2] Confirmation: How is the filter icon filled/not filled state affected by these new type toggles? Is it that if one or more notification type filters are off, that can also trigger a filled icon?

The filter icon should be filled in when any of the following has occurred:

  • One or more of the new types are turned to 'off'
  • The read or unread only states are active
  • A mix of the above two bullet points

Engineering Needs:
[1] The UI needs API on the Notifications Data Controller to interact with here

Dependencies

https://phabricator.wikimedia.org/T287298
https://phabricator.wikimedia.org/T288654

Related

https://phabricator.wikimedia.org/T288874
https://phabricator.wikimedia.org/T288773

Event Timeline

Dmantena lowered the priority of this task from Medium to Low.

[1] Is it intended for your Read Status Filters and Inbox Filters to be persisted between app launches?

Yes, if possible

Engineering sync notes - this all looks good - note we also now have notification filter type toggles on this screen as well. I will update the description to match. We decided to not have this as a separate Phabricator task.

@cmadeo very minor new question for you (see [2] in task description). No rush!

Hi @Tsevener!

[2] Confirmation: How is the filter icon filled/not filled state affected by these new type toggles? Is it that if one or more notification type filters are off, that can also trigger a filled icon?

I was imagining that the filter icon will be filled if any of the new types are toggled to 'off' (eg. we're not showing ' filtering' something from the inbox). I know this is a touch backwards feeling so let me see if I can find a better solution to the toggle presentation.

After some consideration and looking through the options available, we're not going to change the styling of the types filters so I've updated the body of the ticket with answers from Design :)

JMinor raised the priority of this task from Low to Medium.Oct 7 2021, 7:21 PM
JMinor claimed this task.