Release: Merge development into main#4715
Merged
Merged
Conversation
feat(desktop): update Rustino.NET to 0.3.8 and expose About app API
Adds global keydown handler to prevent Backspace from navigating back when pressed outside text input fields. The browser's "Backspace = go back" behavior is inappropriate in single-page desktop apps and causes loss of application state. - Add preventDefault() for Backspace outside INPUT/TEXTAREA/contentEditable - Install listener at module load to ensure protection is always active - Add tests for backspace prevention and registered shortcut interaction - Update test to reflect listener always being installed Plan: 00093
Update _resetForTesting() to reinstall the listener after cleanup, matching module-load state. This ensures backspace prevention tests work correctly since the listener is always installed at module load for production use. Plan: 00093
…ckspaceBrowserBackNavigationInDesktopApps [00093] Disable Backspace Browser Back Navigation in Desktop Apps
The showCompleted state machine was inverted in two places.
showCompleted was set whenever cardCount > 0, so the flag meaning "the
user just cleared the last card, briefly show the caught-up message" was
true from first render. And the early return read
if (cards.length === 0 && showCompleted) return null;
which bails out precisely when the message should be visible.
For a single-article feed the result was: dismissing the card returned
null immediately (the "You're all caught up!" message never appeared),
then 2700ms later the timer flipped showCompleted to false, the guard
stopped firing, and the component rendered again -- as nothing but its
invisible sizer div, which reserved the card's full height and clipped
the sidebar menu beneath it.
Set showCompleted only on a real >0 -> 0 transition, and return null when
there is nothing to show and the message is not being displayed, so the
widget unmounts for good and the menu reflows.
hasDismissedNews no longer gates anything and the dismissedNews === null
branches were unreachable (it is typed string[] and its initializer never
returns null), so both are removed.
chore: upgrade TypeScript to 7.0.2
Change toolbarVariant density classes from pb-* to py-* to give the paperclip icon symmetric vertical spacing from the container edge. This fixes the visual alignment issue where the icon row was flush against the top border. tendril/00641 Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Use npx vp instead of bare vp in MSBuild Exec commands to resolve the tool from local node_modules/.bin/ without requiring PATH setup. tendril/00641 Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
…onMissingTopSpacingInContentInput [00641] Fix Clip Icon Missing Top Spacing in ContentInput
Picks up the fix for the macOS "Where is use_default?" dialog on first desktop notification (Ivy-Interactive/Rustino#20, Ivy-Tendril#1682). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
chore: bump Rustino.NET to 0.3.9
Collaborator
Staging removedStaging environment has been deleted for this PR. |
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
4 participants
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.
Automated release PR to merge development into main.
Includes Rustino.NET 0.3.9 bump (fix for macOS "Where is use_default?" notification dialog, Ivy-Tendril#1682).
🤖 Generated with Claude Code