feat: add completed state for jobs widget #421
Labels
No Label
Blocked
Bug
Contributions welcome
Did not do
Errata
Feature request
Fixed
How To
Invalid
Needs design
No resolution
Priority
Critical
Priority
High
Priority
Low
Priority
Medium
Providers
Digital Ocean
Providers
Hetzner
Refactor
Severity
High
Severity
Low
Severity
Medium
Source
Community
Source
Core Team
Source
Stakeholders
Translations
Under investigation
No Milestone
No project
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: SelfPrivacy/selfprivacy.org.app#421
Loading…
Reference in New Issue
There is no content yet.
Delete Branch "add_complated_state_for_jobs_widget"
Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?
#254
@inex need your mastermind knowledge of all UI specs
@ -77,3 +77,3 @@
getIt<NavigationService>().showSnackBar('jobs.upgrade_failed'.tr());
}
emit(JobsStateEmpty());
emit(JobsCompleted());
Even if a job failed? 🗸 looks a bit misleading for an error.
done
feat: add complated state for jobs widgetto feat: add completed state for jobs widget@ -61,3 +61,3 @@
getIt<NavigationService>().showSnackBar('jobs.reboot_failed'.tr());
}
emit(JobsStateEmpty());
emit(JobsCompleted());
I would suggest looking literally two lines above: the reboot request might fail. We shouldn't show that everything is ok in this case.
@ -45,0 +59,4 @@
'jobs.completed'.tr(),
style: Theme.of(context).textTheme.bodyLarge?.copyWith(
color: Theme.of(context).colorScheme.onSurface,
) ??
No need for
?? const TextStyle()
.style
is totally fine withnull
, it has its own defaults.@inex ?
#440 solved this
Pull request closed