Skip to content
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

improved design and UX in /dash page #2

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

rudrodip
Copy link

Result

clack-p-1.mp4

Changes

  1. used react fragment instead of blank div
  2. refactored import path for toast component in graph/page.tsx
  3. Removed the SelectTheme functionality from the /dash page & created a separate component for it
Screenshot 2024-03-25 at 1 35 24 AM
  • now all graph customization is within the graph component, no prop drilling from different component
  • improves ui
  1. Used shadcn button component as copy 🔗 button in the graph/page.tsx for UI consistency
  2. Removed theme prop from graph and graph-layout component, as they are now solely responsible for changing their theme state (Future imrpovement: using custom context for theme)
Screenshot 2024-03-25 at 1 35 56 AM Screenshot 2024-03-25 at 1 36 39 AM
  1. Added loader indicator for graph canvas (scope of improvement: using html elements to create activity calender, it gives more flexibility to work with web specific features like click, hover etc)
  2. Slight tailwindcss changes

Copy link

vercel bot commented Mar 24, 2024

Someone is attempting to deploy a commit to a Personal Account owned by @phukon on Vercel.

@phukon first needs to authorize it.

Copy link
Owner

@phukon phukon left a comment

Choose a reason for hiding this comment

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

Great work! Things are much better now.
However could you please make sure the rendered canvas is responsive?

output.mp4

</Button>
<SelectTheme handleThemeChange={handleThemeChange} />
</div>
<div className="relative w-[400px] md:w-[900px]">
Copy link
Owner

Choose a reason for hiding this comment

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

Great work! Things are much better now.
However could you please make sure the rendered canvas is responsive?

output.mp4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants