search

hash-define-organization / Hash-Whiteboard

Whiteboard Built with ReactJS and Material UI 🚀❤️
https://hashdefine-whiteboard.netlify.app/
MIT License
23 stars 39 forks source link

feat: add fill functionality for background color #22

Closed HrishabhCodes closed 2 years ago

HrishabhCodes commented 2 years ago

This PR is related to issue #4. I have removed the theme toggler and added the filling functionality to change the background color of the canvas.

fixes #4

https://user-images.githubusercontent.com/71200145/156388308-896e6913-4f82-44c0-b64d-aa4447ad989f.mov

netlify[bot] commented 2 years ago

✔️ Deploy Preview for hashdefine-whiteboard ready!

🔨 Explore the source changes: da17a3ff41c4f091fa01706cddec6f4c648942f1

🔍 Inspect the deploy log: https://app.netlify.com/sites/hashdefine-whiteboard/deploys/621f882b1e3af400085e56b2

😎 Browse the preview: https://deploy-preview-22--hashdefine-whiteboard.netlify.app/

DhairyaBahl commented 2 years ago

@HrishabhCodes You don't have to remove that toggle button ! You have to change the background color of canvas as a part of this issue.

HrishabhCodes commented 2 years ago

@HrishabhCodes You don't have to remove that toggle button ! You have to change the background color of canvas as a part of this issue.

Just to be sure with my tasks, I have to add the theme toggler again. Is the background color functionality added by me fine? I have one more doubt regarding themes. What should be changed? Only the canvas or sidebar and header as well?