Repository navigation
Bump @astra-spec/ui to 0.0.6 - #21
Merged
Merged
Conversation
No source changes are needed: the removed public exports come from layers the theme never imports, the dropped prior_insights inventory section is independent of the in-document record kind we render, and every astra-ui class our stylesheet overrides still exists. Typecheck, the test suite and a production article build all pass on the new version. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Contributor
|
Lightcone previews: Paper · Playground — inventory, viewers, dialogs and glyphs b00569a · @astra-spec/ui 0.0.6 from 6248ae0 · Lightcone brand 0.0.3 · astra-theme b00569a · content 3735fcc · MySTRA https://github.com/LightconeResearch/MySTRA/releases/download/v0.0.8/mystra.mjs |
The reusable preview workflow packs the astra-ui checkout it is given, not the version this theme pins on npm, so the PR preview was still rendering the inventory with 0.0.5. Point both the workflow and the packed `ui` ref at the commit that carries the 0.0.6 release. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
astra-ui 0.0.6 shows analysis navigation in the inventory sidebar by default. The theme controls which analysis is read through the page itself and reaches sub-analyses through their own pages, so the tree would either be inert or duplicate site navigation. Turn it off. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
A citation hovered inside a shared record preview opens its bibliography card in a sibling portal, not inside the parent. Its fixed z-index of 70 therefore sat below the shared preview level (1100) and the card was painted behind the record it was opened from. Stack it one above that token instead, and pin the relationship in the stylesheet contract. Co-Authored-By: Claude Fable 5.1 <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.
Summary
@astra-spec/uito 0.0.6 inpackages/astra; the lockfile picks up its new transitive dependencyreact-zoom-pan-pinch.uiref) to the astra-ui commit carrying the 0.0.6 release. The preview packs that checkout rather than our npm pin, so without this the PR preview kept rendering with 0.0.5.showHierarchy={false}). The page fixes which analysis is read and sub-analyses have their own pages, so the tree would be inert or duplicate site navigation.OutputCard/PriorInsightsListfrom blocks,indirectDecisionPathsfrom model) come from layers the theme never imports. The droppedprior_insightsinventory section is independent of the in-document record kind we still render. Every astra-ui class our stylesheet overrides still exists in 0.0.6, and our inventory dialog override remains more specific than the reworked upstream panel widths.Visible changes from upstream
New opt-in
Inventoryprops (getOutputStatus,renderProvenance,renderCodeLink) are not wired yet.Test plan
npm run typechecknpx vitest run(140 tests, including new assertions for no hierarchy and citation stacking)npm run build:article@astra-spec/ui 0.0.6🤖 Generated with Claude Code