Fix/status countdown and language list - #34
Merged
Conversation
…p line The overscroll fix killed one way the border vanished and left the other, which needed a real device-pixel-ratio to see: scrollTop and scrollHeight clamp on integers while a 125% or 150% display lays the rows out on fractions, so at maximum scroll the last row's bottom sat a third of a pixel below the crop - permanently, with the hover repaint deciding whether the straddling border was drawn at all. A quarter-rem of slack under the last row means the clamp error eats padding, never border. Reproduced and verified with a headless Chrome at 1.25 and 1.5 device scale driving real wheel and hover input against the preview: the clip measured +0.36px before and -3.6px after, stable across hovers. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…er it With the schedule offset enabled the 19:00 job fires and then sleeps for up to five minutes before it tests anything - and the countdown is cron arithmetic from now, so any status poll during that sleep answered 19:30. The bar rolled to the next slot while the 19:00 test was still on its way, which read as it having been skipped. The delay handle now carries its wake moment, pendingRunAt() answers the earliest one, and /status asks it ahead of the cron - falling back exactly as before when nothing is pending. A wake moment is not an estimate, so the tilde drops with it: "Next at 19:03", then the running state, then "~19:30" once the run is genuinely done. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.