-
Notifications
You must be signed in to change notification settings - Fork 0
OUT-2895 | OUT-2886 | OUT-2894 | Scaffold working rich text editor with formatting and slash menu for IUs #4
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
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
| <TopBar /> | ||
| <div className="h-[calc(100vh-57px)] bg-background-primary"></div> | ||
| <div className="h-[calc(100vh-64px)] overflow-y-scroll bg-background-primary px-6 py-6.5"> | ||
| <Editor content="" /> |
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.
lets add a todo comment to fetch and pass content here.
src/features/app-bridge/constants.ts
Outdated
| // @deprecated - Drop support for copilot-staging.com/copilot-staging.app | ||
| // 'https://dashboard.copilot-staging.com', | ||
| ] | ||
| export const API_DOMAIN = 'https://api.assembly.com' |
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.
i think we should take api domain value from .env. Its dynamic for staging and the original platform.
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.
I removed the constant entirely. will take from .env in future if needed. Good callout
| return true | ||
| case 'ArrowDown': | ||
| event.preventDefault() | ||
| down() |
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.
are other keyboard events supported by default? for example if we press side arrows/escape key, does the command list close?
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.
Escape key is handled in another place globally. Side arrows don't do anything
Co-authored-by: SandipBajracharya <sndp106.sb@gmail.com>
Changes
Testing Criteria
Screen.Recording.2026-01-07.at.19.13.30.mov