Skip to content

UI: rounded border should use camera instead of windows#12601

Merged
alice-i-cecile merged 1 commit intobevyengine:mainfrom
mockersf:rounded-border-camera-driven-ui
Mar 21, 2024
Merged

UI: rounded border should use camera instead of windows#12601
alice-i-cecile merged 1 commit intobevyengine:mainfrom
mockersf:rounded-border-camera-driven-ui

Conversation

@mockersf
Copy link
Member

Objective

Solution

@mockersf mockersf added C-Bug An unexpected or incorrect behavior A-UI Graphical user interfaces, styles, layouts, and widgets labels Mar 20, 2024
@mockersf mockersf added this to the 0.14 milestone Mar 20, 2024
@alice-i-cecile alice-i-cecile requested a review from rparrett March 20, 2024 23:17
@alice-i-cecile
Copy link
Member

@chompaa could I get your review here?

@chompaa
Copy link
Member

chompaa commented Mar 20, 2024

LGTM 👍

@alice-i-cecile alice-i-cecile added the S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it label Mar 20, 2024
@alice-i-cecile alice-i-cecile added this pull request to the merge queue Mar 20, 2024
Merged via the queue into bevyengine:main with commit 7b842e3 Mar 21, 2024
@rparrett
Copy link
Contributor

rparrett commented Mar 21, 2024

Tested with a modified render_ui_to_texture using viewport coordinates for border radius / size.

Before Screenshot 2024-03-20 at 5 01 05 PM
After Screenshot 2024-03-20 at 5 01 26 PM

👍

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

Labels

A-UI Graphical user interfaces, styles, layouts, and widgets C-Bug An unexpected or incorrect behavior S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants