Skip to content

Update --duration when opening an existing window to get rid of the flickering#1326

Open
Elvyria wants to merge 2 commits into
elkowar:masterfrom
Elvyria:fix-osd-flicker
Open

Update --duration when opening an existing window to get rid of the flickering#1326
Elvyria wants to merge 2 commits into
elkowar:masterfrom
Elvyria:fix-osd-flicker

Conversation

@Elvyria

@Elvyria Elvyria commented May 24, 2025

Copy link
Copy Markdown

Description

This is an alternative rewrite of #1120 that removes timer if no duration was provided per #1120 (comment) and simplifies the code by utilizing CancellationToken from tokio-utils, since the package is already here anyways.

  • select! macro was used without biased, so no more RNG overhead.
  • Got rid of a useless String clone.
  • Bumps tokio-util version from 0.7.11 to 0.7.15 for run_until_cancelled_owned.

Additional Notes

Fixes #892
Closes #1120

Checklist

  • All widgets I've added are correctly documented.
  • I added my changes to CHANGELOG.md, if appropriate.
  • The documentation in the docs/content/main directory has been adjusted to reflect my changes.
  • I used cargo fmt to automatically format all code before committing

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.

[BUG] Opening a window that is already open closes and reopens it, causing flickering

1 participant