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

Add UI for enabling/disabling poll notifications #10255

Merged
merged 4 commits into from
Mar 13, 2019

Conversation

ClearlyClaire
Copy link
Contributor

No description provided.

@Aditoo17
Copy link
Contributor

Could you also add poll notifications to the quick filter bar?

@Gargron
Copy link
Member

Gargron commented Mar 13, 2019

I'm wondering if the strings are understandable enough for the end-user. Polls - why polls? Maybe call it "Poll results" or something like that? What does the notification text say when you get a notification like that?

@ClearlyClaire
Copy link
Contributor Author

Fair point, “poll results” is probably a better name.

Here is what it looks like:
Screenshot_2019-03-13 Dev instance(1)

And looking at it, I realize it hasn't been updated since the point such notifications were only for your own polls…

@Gargron
Copy link
Member

Gargron commented Mar 13, 2019

Okay! So "Poll results", and also "A poll you voted in has ended"

“Your poll has ended” → “A poll you have voted in has ended”
@ClearlyClaire
Copy link
Contributor Author

Screenshot_2019-03-13 Dev instance(2)

@Gargron Gargron merged commit 65d9004 into mastodon:master Mar 13, 2019
@ClearlyClaire ClearlyClaire deleted the features/poll-notifications-ui branch March 14, 2019 13:12
hiyuki2578 pushed a commit to ProjectMyosotis/mastodon that referenced this pull request Oct 2, 2019
* Add UI for enabling/disabling poll notifications

* Add poll notifications to the (advanced) quick filter bar

* Update poll notification message

“Your poll has ended” → “A poll you have voted in has ended”

* Clear up associated notifications when a poll is deleted
messenjahofchrist pushed a commit to Origin-Creative/mastodon that referenced this pull request Jul 30, 2021
* Add UI for enabling/disabling poll notifications

* Add poll notifications to the (advanced) quick filter bar

* Update poll notification message

“Your poll has ended” → “A poll you have voted in has ended”

* Clear up associated notifications when a poll is deleted
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants