-
Notifications
You must be signed in to change notification settings - Fork 3
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
Add text block edit functionality #12
Add text block edit functionality #12
Conversation
@mrnossiom is attempting to deploy a commit to the Premier Octet 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 ↗︎
|
Feature needed! 🙏 Thanks @mrnossiom |
Hello @mrnossiom! We have slightly updated the UI related to your PR, see #15 |
4e01324
to
cfa28ab
Compare
I changed the button label of the |
Co-authored-by: Baptiste Adrien <adrien.baptiste@gmail.com>
Co-authored-by: Baptiste Adrien <adrien.baptiste@gmail.com>
Thanks @mrnossiom! I have spot a bug, when the user edits the text and re-edit, the textarea still contains the old text : Kapture.2023-09-15.at.19.00.56.mp4 |
Good catch 👍🏻 |
Thanks a lot @mrnossiom! |
Hi 👋🏻,
To add the ability of editing text blocks, I quickly duplicated the
EditBookmarkCard
file and modified the.../{digestId}/block/{blockId}
endpointPATCH
implementation. Adding anonEditClick
prop to theBlockTextCard
component to show theEdit
button in the Block Menu Popover.Didn't test intensively, but it should work fine 😄
Improvements
AddTextBlockDialog
andEditTextBlockDialog
into one component?.../{digestId}/block/{blockId}
file to avoid nesting. Factorizeif-else
statement in functions?