Webstar/LibrePortal
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

ui(tasks): brighten empty-state $ line + fix "No all tasks tasks found"

Browse Source 
.task-command was still using var(--status-success) (#28a745) which reads
muddy olive against the nebula gradient — the same dimming the status
pills and apps-installed pill already work around with #86efac. The
empty-state row ("$ No tasks found …") was the most visible offender.
Switches .task-command to the same bright mint already used elsewhere.

Same edit, while I was there: the empty-state copy interpolated
categoryName.toLowerCase() as `No ${cat} tasks found`, so the "All Tasks"
category produced "No all tasks tasks found". Special-cases the all
bucket and strips the trailing word when the category name already
includes it ("Running Tasks" → "No running tasks found", not "running
tasks tasks").

Signed-off-by: librelad <librelad@digitalangels.vip>
This commit is contained in:
librelad 2026-05-27 15:39:38 +01:00
parent 4d982680b3
commit 22203a7f60
2 changed files with 17 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
containers/libreportal/frontend/css/tasks.css
View File

@ -329,7 +329,11 @@
}
.task-command {
color: var(--status-success);
/* Bright mint to match the .status-running / .status-completed pills.
The theme's --status-success (#28a745) reads muddy olive on nebula —
this is the same #86efac treatment used by the status pills + the
setup-wizard valid border + the apps "Installed" pill. */
color: #86efac;
font-family: 'Courier New', monospace;
font-size: 11px;
flex: 1;

14
containers/libreportal/frontend/js/components/tasks/tasks-manager.js
View File

@ -491,8 +491,18 @@ class TasksManager {
if (this.highlightedTaskId) {
message = `Task ${this.highlightedTaskId} not found`;
} else {
const categoryName = this.getCategoryDisplayName(this.currentCategory);
message = `No ${categoryName.toLowerCase()} tasks found`;
// Category display names sometimes already end in "Tasks" (e.g.
// "All Tasks", "Running Tasks") — naive interpolation produced
// "No all tasks tasks found". Strip the trailing word when
// present, and special-case the "all" bucket to read naturally.
const catLow = this.getCategoryDisplayName(this.currentCategory).toLowerCase();
if (catLow === 'all tasks' || catLow === 'all') {
message = 'No tasks found';
} else if (catLow.endsWith(' tasks')) {
message = `No ${catLow} found`;
} else {
message = `No ${catLow} tasks found`;
}
}
container.innerHTML = `