Skip to content

Add border radius release notes #1353

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

Merged
merged 2 commits into from
Jun 4, 2024

Conversation

chompaa
Copy link
Member

@chompaa chompaa commented Jun 4, 2024

Fixes #1246.

I also have some questions:

  • Are the doc links correct?
  • Is the example too bloated?
  • Are these notes too brief?

@alice-i-cecile alice-i-cecile added this to the Release v0.14 milestone Jun 4, 2024
@alice-i-cecile
Copy link
Member

Are the doc links correct?

Can you swap to https://dev-docs.bevyengine.org/bevy/index.html instead for now? That way they'll work before the release, and we can do a final find-and-replace during editing.

Is the example too bloated?

No, this is solid.

Are these notes too brief?

I like the brevity. This is a simple well-understood feature, even if the impact is high.

@chompaa
Copy link
Member Author

chompaa commented Jun 4, 2024

Can you swap to https://dev-docs.bevyengine.org/bevy/index.html instead for now? That way they'll work before the release, and we can do a final find-and-replace during editing.

This makes much more sense, thanks! I've updated the links :)

@alice-i-cecile alice-i-cecile added S-Ready-For-Final-Review Ready for a maintainer to consider for merging and removed S-Waiting-On-Author labels Jun 4, 2024
@alice-i-cecile alice-i-cecile added this pull request to the merge queue Jun 4, 2024
Merged via the queue into bevyengine:main with commit 9e8eb87 Jun 4, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Release-Notes C-Content S-Ready-For-Final-Review Ready for a maintainer to consider for merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Write release notes for PR #12500: Add border radius to UI nodes (adopted)
2 participants