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

Add border_width to ReferenceRect #42907

Merged
merged 1 commit into from
Oct 22, 2020
Merged

Add border_width to ReferenceRect #42907

merged 1 commit into from
Oct 22, 2020

Conversation

Xrayez
Copy link
Contributor

@Xrayez Xrayez commented Oct 19, 2020

With #42906, helps godotengine/godot-proposals#1687.

These changes complement the engine's functionality as provided by CanvasItem.draw_rect() used by both ReferenceRect and ColorRect. Yet ColorRect cannot draw non-filled rectangles, so these changes may be worth to cherry-pick to 3.2. But perhaps it's worth to merge the ReferenceRect functionality into ColorRect in 4.0, as done in #42931.

@Xrayez Xrayez requested a review from a team as a code owner October 19, 2020 17:45
@Calinou Calinou added cherrypick:3.x Considered for cherry-picking into a future 3.x release enhancement topic:gui labels Oct 19, 2020
@Calinou Calinou added this to the 4.0 milestone Oct 19, 2020
Exposes a hidden parameter behind `CanvasItem.draw_rect()`.
@Chaosus Chaosus self-requested a review October 22, 2020 13:05
@akien-mga akien-mga merged commit f0f8864 into godotengine:master Oct 22, 2020
@akien-mga
Copy link
Member

Thanks!

@Xrayez Xrayez deleted the add-ref-rect-border-width branch October 22, 2020 13:58
@akien-mga
Copy link
Member

Cherry-picked for 3.2.4.

@akien-mga akien-mga removed the cherrypick:3.x Considered for cherry-picking into a future 3.x release label Oct 26, 2020
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.

4 participants