[FEATURE]: Whiteboard Added#257
Conversation
|
@Moresh-Mishra is attempting to deploy a commit to the Harsh Yadav's projects Team on Vercel. A member of the Team first needs to authorize it. |
There was a problem hiding this comment.
🎉 Thank you @Moresh-Mishra for your first PR to SaveBook!
We really appreciate your contribution 🙌
What happens next:
- 🔍 Maintainers will review your PR
- 🧪 Automated checks will run
- ✨ Feedback may be shared if needed
Please confirm your PR includes:
- ✔️ Clear summary of changes
- ✔️ Linked issue (e.g., Fixes #123)
- ✔️ Steps to test
- ✔️ Screenshots (for UI changes)
📘 Contribution Standards:
👉 https://github.com/HarshYadav152/SaveBook/blob/main/CONTRIBUTING.md
💬 Stay Connected with Our Community
🎯 Discord (Official Communication)
For formal discussions, code reviews, and project updates:
👉 https://discord.gg/bZ47fac2jn
💚 WhatsApp (Friendly Community)
For informal chats, quick help, and building friendships with contributors:
👉 https://chat.whatsapp.com/I8GYXd3mHlDCC2iXhNGeqV
🌟 Our Philosophy: We value both professional collaboration (Discord) and personal connections (WhatsApp). Join both to get the complete SaveBook community experience!
Thanks for helping improve SaveBook 🚀
Let's build something amazing together! 💪
HarshYadav152
left a comment
There was a problem hiding this comment.
@Moresh-Mishra Tell me why there is 250 files to be merge.
And also tell me about fonts files .woff2
|
Hello @HarshYadav152 250 files are because of the the whiteboard, it a really heavy changes that were required Now onto the font files .woff2 comes with the excalibur library that i have added to execute the whiteboard PS. everything was checked completely and I understand 250 can be exilarating to watch but i have tested everything |
|
@Moresh-Mishra Alright, |
|
Will do👍 |
|
I have moved fonts to public/fonts |
HarshYadav152
left a comment
There was a problem hiding this comment.
have you run it before pushing recent changes. if yes then it is running or not.
By the consequence of this you forgot to update font path in excalidraw.css update ./fonts/* to /fonts/* so it serve from /public/ directory.
Also do these changes before proceed.
- When user click on New Whiteboard then it will create whiteboard with previous whiteboard title but it should be empty for new whiteboard.
- At
notes/whiteboardadd a button for new whiteboard like a/notespage - In
WhiteboardCanvas.jscomponent where whiteboard-canvas class is used you have to add explicit height and width for rendering excalidraw canvas, becaue it need explicait height.
by the effect of this excalidraw canvas is not showing properly.
Do these changes and let me know.
And yes keep your branch updated with upstream main.
|
I have made all the changes you ask me to |
Description
Summary: Added interactive whiteboard support to notes with Excalidraw, encrypted canvas data, autosave, and a right-side whiteboard tab in the notes UI. Added whiteboard routes, components, and sharing support for whiteboard payloads.
Fixes Issue #254
Type of change
How Has This Been Tested?
/notes/whiteboardChecklist: