Skip to content

fix: migrate from 'vue-draggable-resizable' to '@vueuse' composables - #19489

Open
Antreesy wants to merge 2 commits into
mainfrom
fix/14103/use-draggable
Open

Antreesy wants to merge 2 commits into
mainfrom
fix/14103/use-draggable

Conversation

@Antreesy

Copy link
Copy Markdown
Contributor

☑️ Resolves

  • Migrate from low-maintained library to vueuse
    • wrapper composable added to combine useDraggable (moving by mouse) and useElementSize (moving by container changes)
    • both consumers updated
    • old library dropped

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

🖌️ UI Checklist

No visual changes

🏁 Checklist

  • 🌏 Tested with different browsers / clients:
    • Chromium (Chrome / Edge / Opera / Brave)
    • Firefox
    • Safari
    • Talk Desktop
    • Integrations with Files sidebar and other apps
    • Not risky to browser differences / client
  • 🖌️ Design was reviewed, approved or inspired by the design team
  • ⛑️ Tests are included or not possible
  • 📗 User documentation in https://github.com/nextcloud/documentation/tree/master/user_manual/talk has been updated or is not required

- wrapper composable added to combine useDraggable (moving by mouse) and useElementSize (moving by container changes)

Assisted-by: ClaudeCode:claude-sonnet-5
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
@Antreesy Antreesy added this to the ☃️ Next Major (36) milestone Sep 20, 2026
@Antreesy Antreesy self-assigned this Sep 20, 2026
@nextcloud-command nextcloud-command added the AI assisted This PR contains AI-assisted commits label Sep 20, 2026

@ShGKme ShGKme left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good, but I haven't tested it yet

Comment on lines +106 to +107
const presenterOverlayContainer = ref(null)
const presenterOverlay = ref(null)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

useTemplateRef?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI assisted This PR contains AI-assisted commits feature: call 📹 Voice and video calls feature: frontend 🖌️ "Web UI" client technical debt

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants