Skip to content

Commit c106753

Browse files
committed
fix: mobile notif panel shows X close button instead of Mark all read
1 parent da81f35 commit c106753

2 files changed

Lines changed: 36 additions & 3 deletions

File tree

public/css/style.css

Lines changed: 29 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1343,12 +1343,22 @@ tbody tr:hover {
13431343
flex-shrink: 0;
13441344
}
13451345

1346+
.notif-header-actions {
1347+
display: flex;
1348+
align-items: center;
1349+
gap: 8px;
1350+
}
1351+
13461352
.notif-panel-header h3 {
13471353
font-size: 1.1rem;
13481354
font-weight: 700;
13491355
color: var(--text-primary);
13501356
}
13511357

1358+
.notif-close-mobile {
1359+
display: none;
1360+
}
1361+
13521362
.notif-mark-all {
13531363
background: none;
13541364
border: none;
@@ -1976,8 +1986,25 @@ tbody tr:hover {
19761986
}
19771987

19781988
.notif-mark-all {
1979-
font-size: 0.78rem;
1980-
padding: 6px 10px;
1989+
display: none;
1990+
}
1991+
1992+
.notif-close-mobile {
1993+
display: flex;
1994+
align-items: center;
1995+
justify-content: center;
1996+
background: none;
1997+
border: none;
1998+
color: var(--text-secondary);
1999+
cursor: pointer;
2000+
padding: 4px;
2001+
border-radius: var(--radius-sm);
2002+
transition: all var(--transition);
2003+
}
2004+
2005+
.notif-close-mobile:hover {
2006+
color: var(--text-primary);
2007+
background: var(--bg-3);
19812008
}
19822009

19832010
.notif-item {

public/js/app.js

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1015,7 +1015,12 @@ async function renderDashboard() {
10151015
<div class="notif-panel" id="notif-panel">
10161016
<div class="notif-panel-header">
10171017
<h3>Notifications</h3>
1018-
<button class="notif-mark-all" id="notif-mark-all">Mark all read</button>
1018+
<div class="notif-header-actions">
1019+
<button class="notif-mark-all" id="notif-mark-all">Mark all read</button>
1020+
<button class="notif-close-mobile" id="notif-close-mobile" aria-label="Close notifications">
1021+
<i data-lucide="x" style="width:20px;height:20px"></i>
1022+
</button>
1023+
</div>
10191024
</div>
10201025
<div class="notif-panel-list" id="notif-panel-list">
10211026
<div class="notif-empty">No notifications yet</div>
@@ -1074,6 +1079,7 @@ async function renderDashboard() {
10741079
});
10751080

10761081
$('#notif-backdrop').addEventListener('click', closeNotifPanel);
1082+
$('#notif-close-mobile').addEventListener('click', closeNotifPanel);
10771083

10781084
$('#notif-mark-all').addEventListener('click', markAllAsRead);
10791085

0 commit comments

Comments
 (0)