supermemory icon indicating copy to clipboard operation
supermemory copied to clipboard

experimenting to improve extension (drag & tailwind)

Open CodeTorso opened this issue 1 year ago • 3 comments
trafficstars

Enhancements to Extension Functionality (Drag & Tailwind)

Overview

This pull request introduces several enhancements to the extension's functionality, focusing on improving user experience through the integration of drag-and-drop capabilities and the incorporation of Tailwind CSS for improved styling.

Changes

  • New Features:

    • Implemented drag-and-drop functionality to enhance user interaction and experience.
    • Integrated Tailwind CSS to streamline the styling and layout of the extension's user interface.
    • Added support for smoother drag interactions, ensuring seamless item movement within the extension.
  • Refactoring:

    • Refactored the codebase to leverage Tailwind's utility-first approach, improving maintainability and readability.
    • Restructured the CSS codebase to utilize Tailwind's utility classes, reducing the need for custom CSS and enhancing code readability.
    • Updated the package.json file to include necessary dependencies for Tailwind CSS and the motion library, ensuring compatibility and functionality.
  • Other Changes:

    • Adjusted the styling of popover trigger buttons to improve their appearance and behavior.
    • Wrapped the main container in a draggable element, allowing users to move it around the screen easily.
    • Enhanced the overall design and responsiveness of the user interface to provide a more dynamic user experience.

✨ Generated with love by Kaizen ❤️

Original Description None

CodeTorso avatar Aug 08 '24 22:08 CodeTorso

Code Review

All Clear: This PR is ready to merge! 👍

Incorrect Usage of motion.div

The motion.div is used incorrectly without proper configuration.

Potential Solution:

Either remove the motion.div or configure it properly with animation props.

apps/extension/content/ContentApp.tsx | 265 - 265

reason_for_request: Improper use of the motion component can lead to unexpected behavior or errors.

level: [critical] , severity: [8]

✨ Generated with love by Kaizen ❤️


Useful Commands
  • Feedback: Reply with !feedback [your message]

  • Ask PR: Reply with !ask-pr [your question]

  • Review: Reply with !review

kaizen-bot[bot] avatar Aug 08 '24 22:08 kaizen-bot[bot]

image

Looks great!

One small quirk: Too much inertia if i drag and quickly move it, it will flow away

Can we try to make it more sticky/

Dhravya avatar Aug 10 '24 04:08 Dhravya

I made sure it, does not have momentum still will try

CodeTorso avatar Aug 11 '24 18:08 CodeTorso