-
-
Notifications
You must be signed in to change notification settings - Fork 392
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Hide "Mark all as read" button when no notifications #1948
Conversation
16fab5f
to
dbca533
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Check my comments. Also; can you implement the "no notifications" message as a callout?
CHANGELOG.md
Outdated
@@ -6,6 +6,7 @@ | |||
|
|||
- **decidim-core**: Show a disabled "Follow" button to anonymous users (not logged in) with a prompt to sign in [\#1903](https://github.com/decidim/decidim/pull/1903) | |||
- **decidim-core**: Adds an option to the initializer file to enable/disable header snippets (`true` by default) [\#1923](https://github.com/decidim/decidim/pull/1923) | |||
- **decidim-core**: Hides the "MArkk all as read" notifications button whn no notifications are found [\#1948](https://github.com/decidim/decidim/pull/1948) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two typos in a row, jackpot!
dbca533
to
10b54a3
Compare
@josepjaume solved everything! |
10b54a3
to
772b4b4
Compare
Codecov Report
@@ Coverage Diff @@
## master #1948 +/- ##
==========================================
+ Coverage 98.48% 98.48% +<.01%
==========================================
Files 1145 1145
Lines 25729 25730 +1
==========================================
+ Hits 25338 25339 +1
Misses 391 391 |
Rebased against master so that codecov would run (it was stuck) |
772b4b4
to
11db99b
Compare
馃帺 What? Why?
Hides the "Mark all as read" notifications button when no notifications are found.
馃搶 Related Issues
馃摲 Screenshots (optional)