This PR adjusts the logic of the calculateCrawlSpeed function to better handle short strings in the crawl.
It swaps to calculating animation time based on characters per second.
There are three cases:
Small string, low speed. Every string that will complete within the time the crawl bar is up should move at a consistent comfortable reading speed.
Long string, high speed. Every string that cannot complete within the time the crawl bar is up should move at the fastest comfortable reading speed and is not expected to complete within the time the crawl bar is on-screen.
In between. Medium-length strings should move at the slowest speed that allows them to complete while the crawl bar is on-screen.
Additionally, this PR hides the bar entirely if CONFIG.crawl (set either by config, interactively, or by alerts) is empty.
My apologies for the delayed review. Happy to share that all your recent pull requests have now been merged. Thank you for your valuable contributions!
This PR adjusts the logic of the calculateCrawlSpeed function to better handle short strings in the crawl.
It swaps to calculating animation time based on characters per second. There are three cases:
Additionally, this PR hides the bar entirely if CONFIG.crawl (set either by config, interactively, or by alerts) is empty.