-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
feat(builder): Enable ctrl+z to undo changes in editor #255
feat(builder): Enable ctrl+z to undo changes in editor #255
Conversation
@Jorgelig is attempting to deploy a commit to the Typebot Team on Vercel. A member of the Team first needs to authorize it. |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
7ba1799
to
3850754
Compare
apps/builder/src/features/editor/components/TypebotHeader/TypebotHeader.tsx
Outdated
Show resolved
Hide resolved
apps/builder/src/features/editor/components/TypebotHeader/TypebotHeader.tsx
Outdated
Show resolved
Hide resolved
apps/builder/src/features/editor/components/TypebotHeader/TypebotHeader.tsx
Outdated
Show resolved
Hide resolved
3850754
to
14f4ee3
Compare
14f4ee3
to
2910c3e
Compare
2910c3e
to
cc19346
Compare
cc19346
to
26d9ea5
Compare
I'm concerned with this scenario: What happens if we edit a text bubble, and we just want to hit "Cmd + z" to undo the text? It would undo the whole typebot |
26d9ea5
to
e547cbb
Compare
e547cbb
to
df258a7
Compare
I'll have to test it a bit and will let you know 👍 |
The undo functionality already exists within Typebot js. When reusing that function, it should behave the same as directly clicking the undo button in the header. |
df258a7
to
f2928d9
Compare
Make sure it doesn't trigger if the user is typing. Also gives a better visual indication when the undo shortcut is triggered
78972a7
to
17b2741
Compare
So I had to improve it a bit:
See 17b2741 Let me know if it's ok with you! |
I did not imagine that scenario, the change looks good. Thanks |
Added shortcut Ctrl + z requested in task #217
Closes #217