-
Notifications
You must be signed in to change notification settings - Fork 1
Open
Labels
enhancementNew feature or requestNew feature or requestlow-priorityLow priority issueLow priority issue
Description
Based on CodeRabbit review in PR #11: #11
Event Handling Improvements:
1. Fix Event Dispatching Logic (src/allonsh.js:506-515)
- Currently dispatches both
dragenteranddragleaveon drop - This is semantically confusing
- Suggestion: Only dispatch
allonsh-dropon drop completion - Future: Track hover states during drag for proper enter/leave events
2. Event Consistency
- Ensure events are dispatched at appropriate times
- Document expected event sequence in docs
Files to Update:
src/allonsh.jsdocs/allonsh.md
Priority: Low
Improves API consistency and developer experience with events.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or requestlow-priorityLow priority issueLow priority issue