Bug 396497

Summary: Timer display maintains opacity level from paused state when resumed/reset
Product: [Plasma] kdeplasma-addons Reporter: Qualimer Judith <qualimerjudith>
Component: timerAssignee: Plasma Bugs List <plasma-bugs>
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version: 5.13.3   
Target Milestone: ---   
Platform: Arch Linux   
OS: Linux   
Latest Commit: Version Fixed In:
Attachments: Timer Pause/Resume Opacity Example

Description Qualimer Judith 2018-07-14 07:58:21 UTC
Created attachment 113927 [details]
Timer Pause/Resume Opacity Example

Plasma's Timer widget uses a fade-in/fade-out animation of the displayed countdown to denote when it is currently paused. However, if a user attempts to resume or reset the timer, the current fade-in/fade-out is maintained.

This can sometimes lead to situations with the countdown being much harder to see, as indicated in the attached video. Instead of the current situation, the timer numbers should instead return to their original opacity levels when it is resumed or reset, keeping the numbers visible to users.
Comment 1 Peter Mühlenpfordt 2018-08-09 13:39:41 UTC
Git commit 28fe01674f6c9a7fd644f9a753665f52d3188eaf by Peter Mühlenpfordt.
Committed on 09/08/2018 at 13:38.
Pushed by muhlenpfordt into branch 'master'.

[Timer applet] Reset opacity on timer start and reset

Summary:
When the Timer plasmoid is paused an opacity animation is used to
signal this state. After resuming, restarting or resetting the timer
the current opacity value is not reset which leads to nearly
invisible and hard to read timer digits sometimes.
This patch sets the opacity to 100% on timer (re)start and reset.

Test Plan:
* Start Timer plasmoid with e.g. 5 minutes countdown
* Pause Timer by click
* When opacity is low
  * Resume by click or
  * Restart with predefined time or
  * Reset timer
* Opacity of running timer should always reset to 100%

Reviewers: #plasma, davidedmundson

Reviewed By: #plasma, davidedmundson

Subscribers: plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D14701

M  +2    -1    applets/timer/package/contents/ui/main.qml

https://commits.kde.org/kdeplasma-addons/28fe01674f6c9a7fd644f9a753665f52d3188eaf