Skip to content

fix: edit simple buttons running away#8533

Merged
SebastianKrupinski merged 1 commit into
mainfrom
fix/edit-simple-buttons
Jun 25, 2026
Merged

fix: edit simple buttons running away#8533
SebastianKrupinski merged 1 commit into
mainfrom
fix/edit-simple-buttons

Conversation

@GVodyanov

Copy link
Copy Markdown
Contributor

Fix #8523

Fixes:

  • save buttons going off screen

Makes it so that the buttons are anchored to the bottom of the component so they are always accessible. If there is not enough space to display everything in the component you scroll down to the attendees.

🤖 AI (if applicable)

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

@GVodyanov GVodyanov self-assigned this Jun 19, 2026
@GVodyanov GVodyanov added the 3. to review Waiting for reviews label Jun 19, 2026
@codecov

codecov Bot commented Jun 19, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@GVodyanov GVodyanov force-pushed the fix/edit-simple-buttons branch from b74033e to 06ef8cf Compare June 19, 2026 11:09

@SebastianKrupinski SebastianKrupinski 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.

Tested works.

In general binding the pop over to the body instead of the anchor seems to fix unknown size issue when calculating the height, very well.

Don't remember the reasoning for not doing this in the first place, at the moment

Comment thread src/views/EditSimple.vue Outdated
Comment thread src/views/EditSimple.vue Outdated
Comment thread src/views/EditSimple.vue Outdated
Comment thread src/views/EditSimple.vue
Signed-off-by: Grigory V <scratchx@gmx.com>
@GVodyanov GVodyanov force-pushed the fix/edit-simple-buttons branch from 06ef8cf to ac9a676 Compare June 25, 2026 12:59

@SebastianKrupinski SebastianKrupinski 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.

Tested. Changes work great

@SebastianKrupinski SebastianKrupinski merged commit a8ab265 into main Jun 25, 2026
40 checks passed
@SebastianKrupinski SebastianKrupinski deleted the fix/edit-simple-buttons branch June 25, 2026 13:23
@SebastianKrupinski

Copy link
Copy Markdown
Contributor

/backport to stable6.5

@backportbot backportbot Bot added the backport-request A backport was requested for this pull request label Jun 25, 2026
@backportbot backportbot Bot removed the backport-request A backport was requested for this pull request label Jun 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Simple calendar editor bottom is cut off

2 participants