[PR #2367] [MERGED] fix(menu): prevent dropdown from closing when cursor crosses offset gap #8218

Closed
opened 2026-04-20 18:05:45 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/go-vikunja/vikunja/pull/2367
Author: @n-nkm
Created: 3/5/2026
Status: Merged
Merged: 3/5/2026
Merged by: @kolaente

Base: mainHead: main


📝 Commits (2)

  • 6ac9d57 fix(frontend): Add an invisible inset element between buttons and dropdown elements
  • 17f521c Merge branch 'main' of github.com:n-nkm/vikunja

📊 Changes

1 file changed (+10 additions, -1 deletions)

View changed files

📝 frontend/src/components/misc/Dropdown.vue (+10 -1)

📄 Description

Fix #2311 by adding an invisible ::before inset element that is as big as the offset from floating UI so the cursor does not accidentally hover a different, underlying element and closes the dropdown.

Added the offset as a style variable so any future changes don't bring up a "magic number" issue.

Testing efforts:

  • Ran unit tests in frontend
  • Tested (manually) several resolutions ranging from 1920X1080 to 800 x 400 (so the hover goes from the bottom up)
  • Cross checked the secondary dropdown on the top right to keep working

I did NOT get playwright to run on this setup, unfortunately. I would appreciate if somebody could run it through


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/go-vikunja/vikunja/pull/2367 **Author:** [@n-nkm](https://github.com/n-nkm) **Created:** 3/5/2026 **Status:** ✅ Merged **Merged:** 3/5/2026 **Merged by:** [@kolaente](https://github.com/kolaente) **Base:** `main` ← **Head:** `main` --- ### 📝 Commits (2) - [`6ac9d57`](https://github.com/go-vikunja/vikunja/commit/6ac9d57c90b1c00e89f091ffcdb500ecc3805950) fix(frontend): Add an invisible inset element between buttons and dropdown elements - [`17f521c`](https://github.com/go-vikunja/vikunja/commit/17f521c3ee43314ba400562f81b4d89d41123d7d) Merge branch 'main' of github.com:n-nkm/vikunja ### 📊 Changes **1 file changed** (+10 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `frontend/src/components/misc/Dropdown.vue` (+10 -1) </details> ### 📄 Description Fix #2311 by adding an invisible ::before inset element that is as big as the offset from floating UI so the cursor does not accidentally hover a different, underlying element and closes the dropdown. Added the offset as a style variable so any future changes don't bring up a "magic number" issue. Testing efforts: - Ran unit tests in frontend - Tested (manually) several resolutions ranging from 1920X1080 to 800 x 400 (so the hover goes from the bottom up) - Cross checked the secondary dropdown on the top right to keep working I did NOT get playwright to run on this setup, unfortunately. I would appreciate if somebody could run it through --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-20 18:05:45 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/vikunja#8218