nosignal/nosignal-shell
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

notifs: use adaptive timer for timeStr instead of reactive binding

Browse source 
Replace the per-second reactive binding with an imperative timer
that adapts its interval based on notification age: 5s for <1min,
30s for <10min, 60s for <1h, 5min for <1d, 1h for older.
This commit is contained in:
2 * r + 2 * t 2026-03-12 21:49:55 +11:00
parent b4514a7820
commit 30c36e356d
1 changed files with 27 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

38
services/Notifs.qml
View file

@ -146,21 +146,37 @@ Singleton {
property var locks: new Set()
property date time: new Date()
readonly property string timeStr: {
const diff = Time.date.getTime() - time.getTime();
property string timeStr: qsTr("now")
function updateTimeStr(): void {
const diff = Date.now() - time.getTime();
const m = Math.floor(diff / 60000);
if (m < 1)
return qsTr("now");
if (m < 1) {
timeStr = qsTr("now");
timeStrTimer.interval = 5000;
} else {
const h = Math.floor(m / 60);
const d = Math.floor(h / 24);
const h = Math.floor(m / 60);
const d = Math.floor(h / 24);
if (d > 0) {
timeStr = `${d}d`;
timeStrTimer.interval = 3600000;
} else if (h > 0) {
timeStr = `${h}h`;
timeStrTimer.interval = 300000;
} else {
timeStr = `${m}m`;
timeStrTimer.interval = m < 10 ? 30000 : 60000;
}
}
}
if (d > 0)
return `${d}d`;
if (h > 0)
return `${h}h`;
return `${m}m`;
readonly property Timer timeStrTimer: Timer {
running: !notif.closed
repeat: true
interval: 5000
onTriggered: notif.updateTimeStr()
}
property Notification notification