-
Notifications
You must be signed in to change notification settings - Fork 0
feat(board): improve board editor UI with tables, callouts, and drag-drop #32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
23 commits
Select commit
Hold shift + click to select a range
f1cd1a2
fix(frontend): keep board add-tab button fixed outside scroll
tendtoyj 8a91015
fix(frontend): keep add-tab button adjacent until tab overflow
tendtoyj b93f04e
fix(frontend): adjust board toolbar and editor horizontal padding
tendtoyj afc6156
fix(frontend): darken board editor body text to match theme foreground
tendtoyj 9bade65
fix(frontend): add bottom padding to board toolbar container
tendtoyj bf30d4b
feat(frontend): add board editor divider support
tendtoyj d4ec947
feat(frontend): add board editor callout block support
tendtoyj d866eda
test(frontend): add board editor callout/slash regressions
tendtoyj 15e42c6
chore(frontend): add temporary Figma HTML-to-Design capture script
tendtoyj f8db48e
refactor(frontend): fork draggable block plugin to local module
tendtoyj 42432cd
fix(frontend): use viewport coordinates in draggable block plugin
tendtoyj c727250
fix(frontend): add top:0 to drag target line for correct positioning
tendtoyj 1ea7636
feat(editor): add markdown table node scaffold
tendtoyj 11944da
feat(editor): add markdown table component and parser utils
tendtoyj 3f9bab4
feat(editor): import markdown tables into board lexical nodes
tendtoyj 77a90b3
feat(editor): style board markdown tables for overflow and headers
tendtoyj 1a3b210
style(editor): normalize paragraph and list margins in board theme
tendtoyj b5a3f7e
feat(board): add tab toolbar right action menu placeholders
tendtoyj 1c2846a
feat(frontend): add board title block in editor
tendtoyj 76ba7ba
fix(frontend): stabilize board tab state sync
tendtoyj a681013
test(frontend): add board title helper regression tests
tendtoyj 2da6ff5
style(editor): adjust board editor spacing and placeholder alignment
tendtoyj 9653445
style(editor): reduce board title font size from 4xl to 2xl
tendtoyj File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,42 @@ | ||
| .board-editor-prose .markdown-table-node { | ||
| margin: 8px 0; | ||
| overflow-x: auto; | ||
| } | ||
|
|
||
| .board-editor-prose .markdown-table-node table { | ||
| font-size: 0.8rem; | ||
| border-collapse: collapse; | ||
| width: max-content; | ||
| min-width: 100%; | ||
| max-width: none; | ||
| } | ||
|
|
||
| .board-editor-prose .markdown-table-node thead th { | ||
| text-align: left; | ||
| font-weight: 600; | ||
| padding: 8px 12px; | ||
| background: hsl(var(--muted)); | ||
| white-space: nowrap; | ||
| border-bottom: 2px solid hsl(var(--border)); | ||
| } | ||
|
|
||
| .board-editor-prose .markdown-table-node tbody td { | ||
| text-align: left; | ||
| padding: 7px 12px; | ||
| color: hsl(var(--muted-foreground)); | ||
| white-space: nowrap; | ||
| background: hsl(var(--background)); | ||
| border-bottom: 1px solid hsl(var(--border)); | ||
| } | ||
|
|
||
| .board-editor-prose .markdown-table-node tbody tr:last-child td { | ||
| border-bottom: none; | ||
| } | ||
|
|
||
| .board-editor-prose .markdown-table-node tbody tr:hover td { | ||
| background: hsl(var(--accent)); | ||
| } | ||
|
|
||
| .board-editor-prose .markdown-table-node :is(th, td)[contenteditable="true"] { | ||
| outline: none; | ||
| } |
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
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
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Loading
https://mcp.figma.com/mcp/html-to-design/capture.jsinRootLayoutexecutes remote third-party code for every page load in all environments, which introduces unnecessary security/privacy risk and an external runtime dependency (and can fail under strict CSP or restricted egress). Since this is marked temporary, it should be removed or explicitly gated to local/dev-only tooling.Useful? React with 👍 / 👎.