Skip to content

[FEATURE]: Whiteboard Added#257

Merged
HarshYadav152 merged 5 commits into
HarshYadav152:mainfrom
Moresh-Mishra:feature/whiteboard
May 24, 2026
Merged

[FEATURE]: Whiteboard Added#257
HarshYadav152 merged 5 commits into
HarshYadav152:mainfrom
Moresh-Mishra:feature/whiteboard

Conversation

@Moresh-Mishra
Copy link
Copy Markdown
Contributor

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

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update

How Has This Been Tested?

  • Manually created 2 whiteboard notes via /notes/whiteboard
  • Verified autosave and reload restores the canvas

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
Screenshot 2026-05-16 231022 Screenshot 2026-05-16 231041

@vercel
Copy link
Copy Markdown

vercel Bot commented May 16, 2026

@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.

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉 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! 💪

Copy link
Copy Markdown
Owner

@HarshYadav152 HarshYadav152 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Moresh-Mishra Tell me why there is 250 files to be merge.
And also tell me about fonts files .woff2

@Moresh-Mishra
Copy link
Copy Markdown
Contributor Author

Moresh-Mishra commented May 17, 2026

Hello @HarshYadav152 250 files are because of the the whiteboard, it a really heavy changes that were required
New structure redevelopment to fit the whiteboard panel
Whiteboard page to write the tile and description
Everything you can see is backend and database connected
Also i added a samll login bypass to check this and test it out which i removed before pushing it

Now onto the font files .woff2 comes with the excalibur library that i have added to execute the whiteboard
Initially it was executing on previous version and after resolving this font file was added

PS. everything was checked completely and I understand 250 can be exilarating to watch but i have tested everything

@HarshYadav152
Copy link
Copy Markdown
Owner

HarshYadav152 commented May 18, 2026

@Moresh-Mishra Alright,
Currently fonts files are in app/
Kindly move them in public/fonts directory
And let me know

@Moresh-Mishra
Copy link
Copy Markdown
Contributor Author

Will do👍

@Moresh-Mishra
Copy link
Copy Markdown
Contributor Author

I have moved fonts to public/fonts

Copy link
Copy Markdown
Owner

@HarshYadav152 HarshYadav152 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

  1. When user click on New Whiteboard then it will create whiteboard with previous whiteboard title but it should be empty for new whiteboard.
Image
  1. At notes/whiteboard add a button for new whiteboard like a /notes page
  2. In WhiteboardCanvas.js component 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.

@Moresh-Mishra
Copy link
Copy Markdown
Contributor Author

I have made all the changes you ask me to
also i have tested them individually.

@HarshYadav152 HarshYadav152 added NSoC'26 Tells us this project is associated with Nexus Spring of Code level3 Level 3 issue for NSoC26 with 10 points labels May 24, 2026
@HarshYadav152 HarshYadav152 merged commit e0be7cd into HarshYadav152:main May 24, 2026
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

level3 Level 3 issue for NSoC26 with 10 points NSoC'26 Tells us this project is associated with Nexus Spring of Code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants