Repository navigation
fix: pre-alpha polish (Command actions, Escape on overlays, Tailwind CLI, init and update output) - #41
Merged
Conversation
added 7 commits
October 3, 2026 17:08
DisposeAsync only caught JSException. After a reload or tab close the circuit is gone, so ShellUI.disposeSidebar throws JSDisconnectedException and Blazor logged it twice per page load.
Command only raised CommandSelected, so items with an Action did nothing on click or Enter. It now runs Action, then raises CommandSelected. CommandPalette is built on Command and gets the same behavior.
…e project An explicit --tailwind now skips the prompt (it was ignored in interactive mode), an unknown value fails before init touches the project, and with --yes init says that standalone is the default and how to choose npm.
…lui update The summary counted every requested component as updated, including skipped and failed ones. InstallComponent now reports success.
The standalone Build/ShellUI.targets silently skipped Tailwind when .shellui/bin had no binary, for example after a fresh clone, leaving a stale app.css. The build now downloads the pinned version with MSBuild's DownloadFile and warns if it can't.
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
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.
Summary
Bug fixes and CLI polish before 0.4.0-alpha.1.
DisposeAsynconly caughtJSException, so after a reload or tab close theJSDisconnectedExceptionfromShellUI.disposeSidebarwas logged twice per page load. It's caught now.CommandItem.Action: clicking an item or pressing Enter ran nothing; onlyCommandSelectedfired. Command now runsAction, then raisesCommandSelected. CommandPalette is built on Command and gets the same behavior.init --tailwind: an explicit value now skips the prompt (it was ignored in interactive mode), an unknown value fails beforeinitchanges the project, and with--yesthe output says standalone is the default and how to choose npm.shellui update: the summary reported every requested component as updated. It now counts updated, skipped and failed separately.Build/ShellUI.targetssilently skipped Tailwind when.shellui/binhad no binary (e.g. after a fresh clone), leaving a staleapp.css. The build now downloads the pinned version with MSBuild'sDownloadFileand warns if it can't. Applies to projects initialized from now on.Tests
ActionbeforeCommandSelected, and Escape on Dropdown and Popover.Build/ShellUI.targetsparses as XML and pins the Tailwind version (standalone and npm).Verification
dotnet build0 warnings, 303 tests pass..shellui/bindeleted,dotnet builddownloaded Tailwind v4.3.2 and builtapp.css. With an unreachable version, the build succeeded with the warning.shellui updateprinted the right counts.init --tailwind gulpfailed before Bootstrap was removed.