Skip to content

fix: recalculate FContextMenu position on scroll (refs SFKUI-7591)#1015

Open
NicklasWei wants to merge 2 commits intomainfrom
contextmenu-horizontalscroll
Open

fix: recalculate FContextMenu position on scroll (refs SFKUI-7591)#1015
NicklasWei wants to merge 2 commits intomainfrom
contextmenu-horizontalscroll

Conversation

@NicklasWei
Copy link
Contributor

@NicklasWei NicklasWei commented Feb 17, 2026

Samma problem som med combobox och tabellens dropplista. Infört liknande lösning som var godkänd för dem, men fungerar inte lika bra här då den har lite annorlunda logik kring placering. Då den prioriterar att vara i viewport över att placeras vid sitt ankare leder det till att den kan släpa med om man skrollar horisontellt, även om ankaret inte finns i viewport.

Sandbox: https://forsakringskassan.github.io/designsystem/pr-preview/pr-1015/vue-sandbox/#/

Todo:

  • Återställ sandbox

@github-actions
Copy link

github-actions bot commented Feb 17, 2026

CHANGELOG

Based on commits in this Pull Request this will create a patch release and the following entries will be added to the changelog:


Bug Fixes

  • recalculate `IPopup` horizontal position on scroll (refs SFKUI-7591) 5954def

If this is not correct you can amend the commit message(s).

Read more about the release process (swedish).

@github-actions
Copy link

github-actions bot commented Feb 17, 2026

Förhandsgranskning 🐛 🔍

Dokumentation och exampel applikationer finns att förhandsgranska på:

Sida URL
Dokumentation https://Forsakringskassan.github.io/designsystem/pr-preview/pr-1015/
Vue Sandbox https://Forsakringskassan.github.io/designsystem/pr-preview/pr-1015/vue-sandbox
Applikationsmall https://Forsakringskassan.github.io/designsystem/pr-preview/pr-1015/examples/page-layout

Senast uppdaterad 2026-03-11 19:32 UTC i gh-pages.

@github-actions
Copy link

github-actions bot commented Feb 17, 2026

Artifact sizes

Artifact sizes in this build (unchanged artifacts collapsed below).

Artifact Files Size Change
@fkui/vue (esm) 1 file(s) 1.1MiB → 1.1MiB (+1.0KiB) +0.10%
@fkui/vue (FBadge bundle) 1 file(s) 550.5KiB → 551.5KiB (+1.0KiB) +0.19%
6 unchanged artifacts
Artifact Files Size
@fkui/date (esm) 1 file(s) 32.2KiB
@fkui/design 2 file(s) 189.8KiB
@fkui/logic (esm) 1 file(s) 162.3KiB
@fkui/vue/cypress 1 file(s) 56.0KiB
@fkui/vue/selectors 1 file(s) 3.3KiB
@fkui/vue-labs (esm) 1 file(s) 248.6KiB

@NicklasWei NicklasWei self-assigned this Feb 17, 2026
@udenius udenius added the work in progress Someone is working on this issue label Feb 17, 2026
@NicklasWei NicklasWei force-pushed the contextmenu-horizontalscroll branch 7 times, most recently from 5475876 to 4d89673 Compare March 4, 2026 21:12
Copy link
Contributor

@atfkassan atfkassan left a comment

Choose a reason for hiding this comment

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

Testat och det var lite edgecase som inte såg bra ut men tycker det är ok.

@NicklasWei NicklasWei force-pushed the contextmenu-horizontalscroll branch 2 times, most recently from 8a82223 to 524ec45 Compare March 11, 2026 19:00
@NicklasWei NicklasWei force-pushed the contextmenu-horizontalscroll branch from 524ec45 to 2f9f27b Compare March 11, 2026 19:28
@NicklasWei NicklasWei changed the title [WIP] fix: recalculate FContextMenu position on scroll (refs SFKUI-7591) fix: recalculate FContextMenu position on scroll (refs SFKUI-7591) Mar 11, 2026
@NicklasWei NicklasWei removed the work in progress Someone is working on this issue label Mar 11, 2026
@NicklasWei NicklasWei marked this pull request as ready for review March 11, 2026 20:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants