Skip to content

Conversation

@Minamiyama
Copy link
Contributor

@Minamiyama Minamiyama commented Jul 10, 2025

Summary

  • Implement ReactSortable for draggable variable items
  • Add drag handle visibility on hover
  • Include unique IDs for sortable items
  • Update VarItem component to accept className prop

Screenshots

after-start-sortable.mp4

Styling update

start-sortable-styling-update.mp4

Checklist

  • This change requires a documentation update, included: Dify Document
  • I understand that this PR may be closed in case there was no previous discussion or issues. (This doesn't apply to typos!)
  • I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change.
  • I've updated the documentation accordingly.
  • I ran dev/reformat(backend) and cd web && npx lint-staged(frontend) to appease the lint gods

- Implement ReactSortable for draggable variable items
- Add drag handle visibility on hover
- Include unique IDs for sortable items
- Update VarItem component to accept className prop
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. 🌊 feat:workflow Workflow related stuff. labels Jul 10, 2025
@crazywoola crazywoola self-assigned this Jul 11, 2025
crazywoola
crazywoola previously approved these changes Jul 11, 2025
Copy link
Member

@crazywoola crazywoola left a comment

Choose a reason for hiding this comment

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

LGTM

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jul 11, 2025
@crazywoola crazywoola self-requested a review July 11, 2025 03:14
@crazywoola crazywoola removed the lgtm This PR has been approved by a maintainer label Jul 11, 2025
@asrguru20
Copy link

this very good feature

Update drag handle positioning from negative to positive left value and change its text color. Also add hover opacity effect to variable icon for better visual feedback.
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jul 11, 2025
@crazywoola crazywoola merged commit 5f9628e into langgenius:main Jul 11, 2025
5 checks passed
yijiaquan pushed a commit to yijiaquan/dify that referenced this pull request Jul 14, 2025
tutkun pushed a commit to tutkun/dify that referenced this pull request Aug 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🌊 feat:workflow Workflow related stuff. lgtm This PR has been approved by a maintainer size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants