Skip to content

Commit 201faa9

Browse files
shubham-padiatimabbott
authored andcommitted
settings: Make Do not disturb icon brigther when it's on.
It was not so obvious to users when they were in DND mode, making the icon brigther when in DND mode hopes to address that. https://chat.zulip.org/#narrow/channel/101-design/topic/zulip-desktop.20DND.20icon.20contrast
1 parent 4125de4 commit 201faa9

File tree

2 files changed

+16
-3
lines changed

2 files changed

+16
-3
lines changed

app/renderer/css/main.css

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -122,6 +122,14 @@ body {
122122
color: hsl(202.22deg 15.08% 64.9%);
123123
}
124124

125+
.action-button > .dnd-on {
126+
color: hsl(200.53deg 14.96% 85%);
127+
}
128+
129+
.action-button:hover > .dnd-on {
130+
color: hsl(202.22deg 15.08% 95%);
131+
}
132+
125133
.action-button.active {
126134
/* background-color: rgba(255, 255, 255, 0.25); */
127135
background-color: rgb(239 239 239 / 100%);

app/renderer/js/main.ts

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -798,9 +798,14 @@ export class ServerManagerView {
798798
toggleDndButton(alert: boolean): void {
799799
this.$dndTooltip.textContent =
800800
(alert ? "Disable" : "Enable") + " Do Not Disturb";
801-
this.$dndButton.querySelector("i")!.textContent = alert
802-
? "notifications_off"
803-
: "notifications";
801+
const $dndIcon = this.$dndButton.querySelector("i")!;
802+
$dndIcon.textContent = alert ? "notifications_off" : "notifications";
803+
804+
if (alert) {
805+
$dndIcon.classList.add("dnd-on");
806+
} else {
807+
$dndIcon.classList.remove("dnd-on");
808+
}
804809
}
805810

806811
async isLoggedIn(tabIndex: number): Promise<boolean> {

0 commit comments

Comments
 (0)