Skip to content

fix(ui): Popups follow open/close animation - #1151

Open
Tektonikal wants to merge 1 commit into
Polyfrost:v1from
Tektonikal:popup-fix
Open

Tektonikal wants to merge 1 commit into
Polyfrost:v1from
Tektonikal:popup-fix

Conversation

@Tektonikal

Copy link
Copy Markdown
Contributor

Description

Add a ShellTransition modifier that allows popups to follow along with the main screen scale and opacity animations. Has weird behaviour with the right-click menu for config options though.

Checklist

  • I made a clear description of what was changed
  • I stated why these changes were necessary
  • I updated documentation or said what needs to be updated
  • I made sure these changes are backwards compatible
  • This pull request is for one feature/bug fix

@saadndm

saadndm commented Oct 4, 2026

Copy link
Copy Markdown
Contributor

Will close #766

@Wyvest Wyvest left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

clanker comments


val scale: Float get() = hiddenScale + (1f - hiddenScale) * alpha
}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

scale derived from alpha: alpha includes alphaMultiplier (= dragAlpha in OneConfigInterface.kt:175). Shell itself scales from progress only. HUD drag at opacity 0.3 → popup shrinks to 0.93 while shell stays 1.0. Fix: pass progress and alpha as separate providers, scale from progress.

scaleY = shellScale
this.transformOrigin = TransformOrigin.Center
}
}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

pivot mismatch: shell scales about shell centre, popup scales about own centre (dropdowns: about anchor edge). Popup shrinks in place while its anchor slides toward screen centre → gap up to 10% of distance from centre. Probably the “weird right-click menu” in PR description and the //TODO: fix this always moving right at Options.kt:181 — unverified. Fix: pivot = shell centre in popup-local coords (ShellState.shellBounds.center already exists; popup position from onGloballyPositioned).

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

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants