Skip to content

Conversation

@msparkles
Copy link
Contributor

reopened from #210 (we moved the repo from forking from automancy's fork to forking from upstream instead because of weird Github shenanigans)

@msparkles msparkles requested a review from LPGhatguy as a code owner July 29, 2025 14:58
@msparkles msparkles force-pushed the feat/rework-clipping branch from 4294224 to fc0ba1b Compare August 2, 2025 16:28
@msparkles
Copy link
Contributor Author

#210 (comment)

Ran into a case where I'd love to have this feature! I haven't managed to untangle our use of yakui viewports in our game yet though and that's still a case where this PR doesn't intersect.

A related question for this: do you have any ideas for how we can bump widgets around so that they stay on screen? One use for this PR is for popping up boxes from within a heavily clipped context, like for tooltips. You also probably want to ensure those tooltips don't run off the edge of the screen!

@LPGhatguy we did something about this but we don't know how we feel about it. we consulted our partner who was visiting the other day for opinion and she said "i don't think the clipping logic should do that much work", but also making the Tooltip example in the current yakui model of layouting logic is just either asinine or impossible to be featureful enough.

Well, she also pointed out that the example's behaviour isn't what most tooltips do, but the rationale here is "it's an example, one that's meant to showcase capabilities" but if one were to make the same thing completely in widget code it would simply be impossible (or suboptimal) (or requires a lot of hacky patches to yakui, guess what we ended up with in our fork before lol)

@msparkles msparkles force-pushed the feat/rework-clipping branch from 30555b7 to f81137d Compare October 7, 2025 15:27
@msparkles msparkles force-pushed the feat/rework-clipping branch from f81137d to 987eb58 Compare December 28, 2025 00:20
@msparkles msparkles force-pushed the feat/rework-clipping branch from 987eb58 to a99084e Compare January 13, 2026 17:45
@msparkles msparkles force-pushed the feat/rework-clipping branch from f399d81 to 457b774 Compare January 17, 2026 19:37
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.

1 participant