Skip to content

fix(tangle-cloud): readable Create-service nav button#3250

Merged
tangletools merged 3 commits into
developfrom
fix/nav-create-button-readability
May 26, 2026
Merged

fix(tangle-cloud): readable Create-service nav button#3250
tangletools merged 3 commits into
developfrom
fix/nav-create-button-readability

Conversation

@drewstone
Copy link
Copy Markdown
Contributor

The 'Create {service}' button in the blueprint nav used the sandbox Button variant, which left dark, italic text on its purple fill — unreadable. Render it as an explicit styled link (white text on a solid accent, non-italic, sized to match the Details button). Also silences the react-refresh lint rule on the TopNavSlot context module so the project lints clean.

drewstone added 3 commits May 25, 2026 15:41
The Header computed breadcrumbs but never rendered them. Show the trail
("Blueprints / Trading") in the nav left slot by default, with the leading
"Cloud" dropped and section roots (Blueprints/Operators/Instances) linking
back. Pages can still override via useTopNavSlot; the blueprint detail now
injects a matching "Blueprints / <name>" trail alongside its Create/Details
actions instead of a "← Catalog" link.
The sandbox Button variant left dark, italic text on its purple fill
(unreadable). Render the Create action as an explicit styled link: white
text on a solid accent fill, non-italic, sized to match the Details button.
…xt module

The context module intentionally exports its Provider alongside its hooks;
the react-refresh/only-export-components rule doesn't apply. Disable it at
the two hook exports so the project lints clean.
@drewstone drewstone requested a review from AtelyPham as a code owner May 26, 2026 16:03
@tangletools tangletools merged commit 5c72113 into develop May 26, 2026
@tangletools tangletools deleted the fix/nav-create-button-readability branch May 26, 2026 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants