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

Fix EditorAudioBus corner radius #97113

Merged
merged 1 commit into from
Sep 17, 2024
Merged

Conversation

Giganzo
Copy link
Contributor

@Giganzo Giganzo commented Sep 17, 2024

Fixes: #95410

Don't think the difference between top corner radius on Master an other buses was intended.
In Godot 3 the disabled button style didn't have any corner radius at all. Now in Godot 4 it has for all corners.

Plus in 4.0.alpha13 audio buses had corner radius set for all corners:
Screenshot_20240917_135631

Changed the Bus to use BottomPanel style instead of TabContainer

Before:
Screenshot_20240915_123744

After:
Screenshot_20240915_123932

@Mickeon
Copy link
Contributor

Mickeon commented Sep 17, 2024

This is extremely subtle but the change makes sense to me. Would increasing the editor's own corner radius in the Editor Settings affect this look as well?

@Giganzo
Copy link
Contributor Author

Giganzo commented Sep 17, 2024

Would increasing the editor's own corner radius in the Editor Settings affect this look as well?

If you mean interface/theme/corner_radius, yes.

Screenshot_20240917_141312

Copy link
Contributor

@Mickeon Mickeon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see nothing wrong with this, then.

Copy link
Member

@Calinou Calinou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great!

@akien-mga akien-mga modified the milestones: 4.x, 4.4 Sep 17, 2024
@akien-mga akien-mga merged commit 922ae7e into godotengine:master Sep 17, 2024
20 checks passed
@akien-mga
Copy link
Member

Thanks!

@Giganzo Giganzo deleted the audio-corners branch September 22, 2024 01:28
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.

Round corners don't take effect on some EditorAudioBus.
5 participants