Conversation
- Add inline mark input rules: _italic_, *bold*, ~strikethrough~, `code` - Add Cmd+Enter as hard break (alongside Shift+Enter) - Add underline mark with Cmd+U shortcut - Add --- input rule to insert horizontal rule - Restore caretPlugin (pulled from main) - Fade-in animation on editor mount - Remove first-heading top margin - Tighten typography: line-height 1.75, paragraph spacing 0.5em Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
@codex[agent] review. |
|
@usamaasfar I've opened a new pull request, #19, to work on those changes. Once the pull request is ready, I'll request review from you. |
Title is hidden by default and fades in (400ms) on topbar hover. Untitled stays always visible as a nudge to rename. Flashes visible for 1.2s on file switch via useTitleVisibility hook. Stays visible while renaming. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
apps/desktop/src/editor/Editor.tsx
Outdated
| markRule(/_([^_]+)_$/, em), // _italic_ | ||
| markRule(/\*([^*]+)\*$/, strong), // *bold* | ||
| markRule(/~~([^~]+)~~$/, strikethrough), // ~~strikethrough~~ | ||
| markRule(/~([^~]+)~$/, strikethrough), // ~strikethrough~ | ||
| markRule(/`([^`]+)`$/, code), // `code` |
There was a problem hiding this comment.
inline mark rules conflict - typing *bold* will match both the em rule on line 56 (_italic_) and the strong rule on line 57 (*bold*) because both use similar patterns, but more critically, these patterns use $ (end of string) which won't match mid-paragraph - the anchor $ means end of input, so typing *test* more text won't trigger the rule at all
| markRule(/_([^_]+)_$/, em), // _italic_ | |
| markRule(/\*([^*]+)\*$/, strong), // *bold* | |
| markRule(/~~([^~]+)~~$/, strikethrough), // ~~strikethrough~~ | |
| markRule(/~([^~]+)~$/, strikethrough), // ~strikethrough~ | |
| markRule(/`([^`]+)`$/, code), // `code` | |
| // inline mark rules | |
| markRule(/_([^_]+)_ /, em), // _italic_ | |
| markRule(/\*([^*]+)\* /, strong), // *bold* | |
| markRule(/~~([^~]+)~~ /, strikethrough), // ~~strikethrough~~ | |
| markRule(/~([^~]+)~ /, strikethrough), // ~strikethrough~ | |
| markRule(/`([^`]+)` /, code), // `code` |
@codex[agent] can you fix it. |
|
@usamaasfar I've opened a new pull request, #21, to work on those changes. Once the pull request is ready, I'll request review from you. |
* Initial plan * Fix inline markdown input rule anchors --------- Co-authored-by: openai-code-agent[bot] <242516109+Codex@users.noreply.github.com>
|
@greptile re-review, please. |
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Greptile Summary
This PR enhances the editor's markdown input rules and improves the overall writing experience with visual refinements.
Key Changes:
*bold*,_italic_,~~strikethrough~~,`code`) that trigger after space or end of line---) and underline mark with Cmd+U shortcutuseTitleVisibilityhookNote: Underline formatting is supported in the editor but is not preserved when serializing to markdown (rendered as plain text in saved files).
Confidence Score: 5/5
useTitleVisibilityhook has proper cleanup, and all CSS transitions are straightforward.Important Files Changed
Last reviewed commit: d8d2d07