search

sd012gfhkhhvh / Slate.AI

A Real Time Whiteboard for doodling with multiple peoples
https://slate-ai.vercel.app
MIT License
11 stars 16 forks source link

Togglable sidebar for Drawing related tools #42

Closed HARSH00M closed 6 months ago

HARSH00M commented 6 months ago

Title : Implementation of a togglable sidebar for Drawing related tools

Issue No. :#38

Description :

  1. Modified some of the Tailwind properties to create a Toggle feature. Also added a function named "openToolbar()" for handling toggle feature.
  2. Added a link in index.html to get the icon used in the Toggle Button.

video:

Slate.webm

Solved this issue under JWOC

vercel[bot] commented 6 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
slate-ai ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 21, 2024 5:48pm
sd012gfhkhhvh commented 6 months ago

@HARSH00M there are several issues with your PR. you did only the toggle feature but the issue is to improvise the entire mobile layout. And there are also some issues with the toggle feature -

  1. its not apealing at all. the colour scheme and how you handle toggle [dont use css transform property for toggle use a modal for mobile view and handle it by controlling its display property otherwise it will create a horizontal scroll]

Screenshot 2024-02-22 143256

  1. Only put the tools inside the modal not the undo, redo, color and eraser, put them seperately above the canvas.
  2. Don't use bootstrap icons

If you have any doubt reach me out for further clarification