Skip to content

Add persisted pause countdown to EVR views (#506) - #507

Open
mudinthewater wants to merge 2 commits into
release/6.0.2from
feat-506-evr-pause-countdown
Open

mudinthewater wants to merge 2 commits into
release/6.0.2from
feat-506-evr-pause-countdown

Conversation

@mudinthewater

Copy link
Copy Markdown
Collaborator

Persist configuration.pauseTimerSeconds for vista.evrView and show an orange pause-countdown status action while the table is paused. At zero the view resumes as if Play were pressed; selecting the countdown stays paused, and marking more rows restarts it. Absent or invalid values keep the previous pause-until-Play behaviour.

The setting lives in the Config tab between Layout and Columns via an EVR fork of core TableConfiguration. The view observes the parent configuration object (writers mutate that path; child-path observers never fire) so edits apply to the open view immediately.

Persist configuration.pauseTimerSeconds for vista.evrView and show an
orange pause-countdown status action while the table is paused. At zero
the view resumes as if Play were pressed; selecting the countdown stays
paused, and marking more rows restarts it. Absent or invalid values keep
the previous pause-until-Play behaviour.

The setting lives in the Config tab between Layout and Columns via an
EVR fork of core TableConfiguration. The view observes the parent
configuration object (writers mutate that path; child-path observers
never fire) so edits apply to the open view immediately.
Use Number.NaN in the countdown spec, move orderCountdownAfterPlay to
module scope, chain the countdown invoke through an optional view, and
drop the unneeded !important from the countdown button background (its
selector already outranks the core button background).
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant