feat: zero bell to all notifications page [2823]

[Tirokk]

Authored by rbeer
Mar 5, 2020
Merged Mar 10, 2020

---

🍰 Pullrequest

I've tried using filled instead of ghost to make the difference in behavior clear, but that always turned out to be too aggressive/standing out, since nothing else up there has this much color/fill.
Removing the counter all together feels like a good visual cue that the behavior of the button changed.

Issues

• solves 🚀 [Feature] Access all notifications via 🔔 icon when there are 0 unread #2823

Remake of #3170

Todo

• Is there a way to test the :to parameter of the <nuxt-link>?

[Tirokk]

Authored by cypress[bot]
Mar 5, 2020

---

---

Test summary

66 • 0 • 0 • 0

---

Run details

Project	Human-Connection
Status	Passed
Commit	8ffda5e
Started	Mar 5, 2020 11:20 AM
Ended	Mar 5, 2020 11:41 AM
Duration	20:48 💡
OS	Linux Ubuntu Linux - 16.04
Browser	Firefox 68

View run in Cypress Dashboard ➡️

---

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

[Tirokk]

Authored by cypress[bot]
Mar 5, 2020

---

---

Test summary

66 • 0 • 0 • 0

---

Run details

Project	Human-Connection
Status	Passed
Commit	795f2d6f84 ℹ️
Started	Mar 5, 2020 11:30 AM
Ended	Mar 5, 2020 11:52 AM
Duration	21:52 💡
OS	Linux Ubuntu Linux - 16.04
Browser	Firefox 68

View run in Cypress Dashboard ➡️

---

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

[Mogge]

Cool! I like it. The only concern I have is that this component suddenly does something else if the notifications counter is 0. I tried it out and to me it does not feel like terrible UX, so I'm OK with this! 👍 thank you @rbeer for your contribution 💖

[Mogge]

I like it, too! Great work! 👍