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

Some editor icons are broken in the ui (master branch) #81492

Closed
bitsawer opened this issue Sep 9, 2023 · 3 comments · Fixed by #81516
Closed

Some editor icons are broken in the ui (master branch) #81492

bitsawer opened this issue Sep 9, 2023 · 3 comments · Fixed by #81516

Comments

@bitsawer
Copy link
Member

bitsawer commented Sep 9, 2023

Godot version

Godot v4.2.dev (fc99492)

System information

Godot v4.2.dev (fc99492) - Windows 10.0.19045 - Vulkan (Forward+) - dedicated NVIDIA GeForce GTX 970 (NVIDIA; 31.0.15.3623) - Intel(R) Core(TM) i7-4790K CPU @ 4.00GHz (8 Threads)

Issue description

Some editor icons are broken and show a red placeholder icon in current master at fc99492. Looks like a recent regression, 4.2 dev4 shows these icons correctly.

Steps to reproduce

Icons that I have noticed that are affected, there can be more:

  • Path3D gizmo handle icons
  • MeshInstance3D toolbar button icon
  • VoxelGI toolbar button icon
  • LightmapGI toolbar button icon

icons

mesh

Minimal reproduction project

Not needed.

@scgm0
Copy link
Contributor

scgm0 commented Sep 9, 2023

c# build icon
图片

@AThousandShips
Copy link
Member

AThousandShips commented Sep 9, 2023

Looks like a possible regression from:

Edit: Not exactly from there in fact, will bisect further

Edit: regression from

@Calinou
Copy link
Member

Calinou commented Sep 9, 2023

Can we make it so requesting an unknown icon in the editor always prints an error at least once? This would make it easier to track down those issues in the future.

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

Successfully merging a pull request may close this issue.

6 participants