Skip to content

UI and unloaded assets: don't filter out nodes with an unloaded image#11205

Merged
alice-i-cecile merged 3 commits intobevyengine:mainfrom
mockersf:fix-unloaded-ui-images
Jan 24, 2024
Merged

UI and unloaded assets: don't filter out nodes with an unloaded image#11205
alice-i-cecile merged 3 commits intobevyengine:mainfrom
mockersf:fix-unloaded-ui-images

Conversation

@mockersf
Copy link
Member

@mockersf mockersf commented Jan 3, 2024

Objective

Solution

  • don't filter out UI nodes with an unloaded image

@mockersf mockersf added C-Bug An unexpected or incorrect behavior A-Assets Load files from disk to use for things like images, models, and sounds A-UI Graphical user interfaces, styles, layouts, and widgets labels Jan 3, 2024
@JMS55 JMS55 added this to the 0.13 milestone Jan 4, 2024
@alice-i-cecile
Copy link
Member

@mockersf is this still needed? The associated PR was disabled by default.

@JMS55
Copy link
Contributor

JMS55 commented Jan 24, 2024

I think this is still needed. It was disabled by default, but that doesn't mean users won't use it.

@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 Jan 24, 2024
@alice-i-cecile
Copy link
Member

Sounds good. Feel free to merge (or bug me) once merge conflicts are fixed.

@alice-i-cecile alice-i-cecile added this pull request to the merge queue Jan 24, 2024
Merged via the queue into bevyengine:main with commit 95b9230 Jan 24, 2024
tjamaan pushed a commit to tjamaan/bevy that referenced this pull request Feb 6, 2024
…bevyengine#11205)

# Objective

- after bevyengine#10520, UI can't display images or text with a background color

## Solution

- don't filter out UI nodes with an unloaded image
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Assets Load files from disk to use for things like images, models, and sounds 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.

3 participants