Repository navigation
style(workflow): align the stage drawer with FlyBot's drawer, buttons and bubble tokens (#1654) - #1655
Merged
Conversation
… and bubble tokens (ThinkFlowLab#1654)
Collaborator
Author
|
Verdict: PASS at cc2c497 Ran at this tip: Review notes:
|
Birfy
enabled auto-merge
October 29, 2026 06:50
Collaborator
Author
|
Verdict: PASS at 8784de6 Ran at this tip: Review notes:
|
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.
Fixes #1654
Written-by: Dev-1
Owner: Dev
Reviewer: Reviewer
Tip: 8784de6
Tree: 281a48e91f7ebf5d84eea45a91e78c06b1315845
What changes
CSS in
plugins/workflow/client.jsonly (plusrole="tablist"/role="tab"/aria-selectedon the drawer tabs), mapped to the patterns already incore/src/client/styles.js:.wf-drawer,.wf-dr-head, close.wf-icon.bt-panel(system font,.5px --bt-line-2edge),.bt-drawer-top, round ghost close.wf-ico-lg, title, description--bt-bubble,.bt-drawer-name17/24 w500,.bt-drawer-role12/16--bt-ink-3.bt-tabs/.bt-tab: 13/20, 9999px pill, selected on--bt-hover.wf-term--bt-bubble, no border, 14px radius.wf-btn.bt-soft/.bt-q-btn: borderless 999px pill on--bt-hover, primary on--bt-accent+--bt-accent-ink.wf-node.bt-bubble:--bt-bubblesurface, 18px radius, status ring kept--bt-tag-bg/--bt-tag-inkpills,.5px --bt-line-2cardEvery hard-coded
rgba(77,107,254,…)tint is nowcolor-mix(in srgb, var(--bt-accent) N%, transparent), so non-DeepSeek themes recolour the drawer. No new colours, only existing--bt-*variables.Independent of #1653 (the flowchart layout fix): both touch
client.jsCSS on different lines and merge cleanly together (checked locally: merged tree runsnpm test239 pass / 1 env fail).Docs
docs/subsystems/workflow.md: theclient.jsrow notes the drawer reuses core tokens and drawer patterns.Tests
New
plugins/workflow/test/stage-style.test.js(3 tests): tab pills + ARIA, chat surfaces on buttons/nodes/run card/title, and no bare hex colour or hard-coded accent rgba in the restyled rules.cd plugins/workflow && npm test: 236 tests, 235 pass, 1 fail —host.test.jscannot importds-boton this box (same on main).Screenshots (local headless Chrome harness):
/workspace/fb-wfgraph-shots/before-B-drawer400.png,after-B-drawer400.png(this PR alone),after-AB-drawer400.png(with #1653).