settings: Restructure 'Personal settings > Notifications' section.

We remove the 'Other notification settings' heading and increase the
font size of 'Desktop message notifications' and 'Mobile message
notifications' headings as these are now on the same level with
'Notification triggers' heading.

We also add status elements to show saving indicator along the mobile,
desktop and email headings which was previously shown with the
"Other notification settings" heading.
This commit is contained in:
Sahil Batra
2021-09-11 13:23:38 +05:30
committed by Tim Abbott
parent 14e383ad7a
commit 60112405ba

View File

@@ -44,12 +44,10 @@
</table>
</div>
<div class="other_notifications m-10 inline-block subsection-parent">
<div class="desktop_notifications m-10 inline-block subsection-parent">
<h3 class="inline-block">{{t "Other notification settings" }}</h3>
<div class="alert-notification other-notify-settings-status"></div>
<h5>{{t "Desktop message notifications" }}</h5>
<h3 class="inline-block">{{t "Desktop message notifications" }}</h3>
<div class="alert-notification"></div>
<p><a class="send_test_notification">{{t "Test desktop notification" }}</a></p>
@@ -87,8 +85,12 @@
{{> dropdown_options_widget option_values=desktop_icon_count_display_values}}
</select>
</div>
</div>
<h5>{{t "Mobile message notifications" }}</h5>
<div class="mobile_notifications m-10 inline-block subsection-parent">
<h3 class="inline-block">{{t "Mobile message notifications" }}</h3>
<div class="alert-notification"></div>
{{#each notification_settings.mobile_notification_settings}}
{{> settings_checkbox
@@ -98,8 +100,12 @@
show_push_notifications_tooltip=(lookup ../show_push_notifications_tooltip this)
prefix=../prefix}}
{{/each}}
</div>
<h5>{{t "Email" }}</h5>
<div class="email_notifications m-10 inline-block subsection-parent">
<h3 class="inline-block">{{t "Email" }}</h3>
<div class="alert-notification"></div>
<label for="email_notifications_batching_period">
{{t "Batching period for email notifications" }}