Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve nine patch behavior of TextureProgressBar #45815

Merged
merged 1 commit into from
Jun 21, 2021

Conversation

floppyhammer
Copy link
Contributor

@floppyhammer floppyhammer commented Feb 8, 2021

This PR does a few things related to TextureProgressBar node.

  1. Fixing TextureProgress's nine_patch_stretch property does not stretch textures for radial and bilinear fill_mode #45791. The fix enables basic texture stretch for radial fill mode when nine_patch_stretch is enabled.
    screen-record0
    ->
    screen-record

  2. Removing a redundant space character from the CLOCKWISE_AND_COUNTER_CLOCKWISE entry in Fill Mode.
    image
    ->
    image

  3. Fixing incorrect stretch of the progress texture for RIGHT_TO_LEFT and BOTTOM_TO_TOP fill mode when asymmetric nine patch margins are applied.
    linear_right_to_left
    linear_bottom_to_top
    ->
    linear_right_to_left_new
    linear_bottom_to_top_new

  4. Implementing nine patch stretch for bilinear fill mode.
    bilinear
    bilinear2

@Calinou Calinou added bug cherrypick:3.x Considered for cherry-picking into a future 3.x release topic:gui labels Feb 8, 2021
@Calinou Calinou added this to the 4.0 milestone Feb 8, 2021
@floppyhammer floppyhammer changed the title Improve nine_patch_stretch's behavior for TextureProgressBar in radial fill_mode Improve nine patch behavior of TextureProgressBar Feb 11, 2021
@floppyhammer floppyhammer requested a review from a team as a code owner February 20, 2021 01:36
scene/gui/texture_progress_bar.cpp Outdated Show resolved Hide resolved
scene/gui/texture_progress_bar.cpp Outdated Show resolved Hide resolved
scene/gui/texture_progress_bar.cpp Outdated Show resolved Hide resolved
scene/gui/texture_progress_bar.cpp Outdated Show resolved Hide resolved
scene/gui/texture_progress_bar.cpp Outdated Show resolved Hide resolved
scene/gui/texture_progress_bar.h Outdated Show resolved Hide resolved
@floppyhammer floppyhammer requested review from Calinou and removed request for a team March 26, 2021 03:50
@floppyhammer floppyhammer deleted the branch godotengine:master March 30, 2021 05:09
@floppyhammer floppyhammer deleted the master branch March 30, 2021 05:09
@floppyhammer floppyhammer restored the master branch March 30, 2021 05:13
@floppyhammer
Copy link
Contributor Author

@Calinou Sorry, messed things up when trying to rename the branch on Github. Can you reopen this?

@akien-mga
Copy link
Member

Could use a rebase to make sure that it still builds and runs fine on latest master, then I guess it's good to merge.

@floppyhammer
Copy link
Contributor Author

@akien-mga Done.

@akien-mga akien-mga merged commit 0d0afca into godotengine:master Jun 21, 2021
@akien-mga
Copy link
Member

Thanks!

@akien-mga
Copy link
Member

Cherry-picked for 3.4.

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

Successfully merging this pull request may close these issues.

3 participants