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

Prepare for 0.3.13 release #525

Merged
merged 10 commits into from
Aug 14, 2023
Merged

Prepare for 0.3.13 release #525

merged 10 commits into from
Aug 14, 2023

Conversation

nicoburns
Copy link
Collaborator

@nicoburns nicoburns commented Aug 13, 2023

Objective

Prepare for 0.3.13 release

Context

Backports the following PRs from the main branch:

Feedback wanted

Does it seem reasonable to release #497 in a patch release? It could technically break (change) people's layouts if they currently have border or padding set on a leaf node. However, it would be quite odd if anybody did have this set as it would currently be having no effect. Nevertheless, we could release this in a minor version update instead.

@nicoburns nicoburns added the release Preparing for the next version label Aug 13, 2023
@nicoburns nicoburns added this to the 0.3.13 milestone Aug 13, 2023
@nicoburns nicoburns force-pushed the release/0.3.13 branch 4 times, most recently from 0081509 to 2b739ca Compare August 13, 2023 19:49
Copy link
Collaborator

@TimJentzsch TimJentzsch left a comment

Choose a reason for hiding this comment

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

Personally I think that breaking changes in patch releases are fine if they fix errors, as they are not breaking the intended behavior.

I also agree that it's highly unlikely that anyone is relying on this behavior.

@alice-i-cecile
Copy link
Collaborator

alice-i-cecile commented Aug 14, 2023

I agree fully with @TimJentzsch here.

@alice-i-cecile
Copy link
Collaborator

The final linked PR is merged now: just needs backporting right?

@nicoburns nicoburns force-pushed the release/0.3.13 branch 3 times, most recently from 54c74eb to a6366a8 Compare August 14, 2023 21:07
* Fixed clippy lint errors (needless_pass_by_ref_mut)

* Fixed clippy lint errors (unnecessary_mut_passed)

(cherry picked from commit 8974878)
@nicoburns nicoburns force-pushed the release/0.3.13 branch 2 times, most recently from cbce3d3 to 434ed37 Compare August 14, 2023 21:37
* Switch font used in generated tests to Ahem

This aligns our gentests with the Web Platform Test suite which also
uses Ahem. See https://github.com/Kozea/Ahem

* Set gentest font color to green

(cherry picked from commit 9a29acd)
(cherry picked from commit 25ffaee)
…em's intrinsic main size (+misc fixes) (#522)

* Add tests for bevyengine/bevy#9350

* Pass accurate cross-axis available space when computing a flexbox item's intrinsic main size

* Subtract child margin not parent margin when computing stretch-alignment known size

* Add extra debug logs to flexbox algorithm

* Apply margins to leaf nodes when computing available space for measure functions

(cherry picked from commit dab541d)
(cherry picked from commit c9e45e5)
@nicoburns nicoburns force-pushed the release/0.3.13 branch 4 times, most recently from ec75d6f to b60d588 Compare August 14, 2023 21:51
…ilable space when it is the root node (#491)

* Add viewport control to gentest setup

* Add tests for max-width and available space on a CSS Grid root node

* Only treat definite available grid space as indefinite when container size
is indefinite, and only do so for the final two track sizing steps.

(cherry picked from commit 9958624)
* Upgrade `grid` dependency

* Only enable std feature in grid crate if Taffy's std feature is enabled

* Fix grid+alloc compilation
* Fix divide by zero when using grid_auto_rows/grid_auto_columns with zero
"negative implicit" tracks

* Fix over-allocation in the CellOccupancyMatrix when auto-placing in grids
that contain negative implicit tracks.

* Fix axis conflation when grid_auto_flow is column

* Fix computation of auto track offset when initializing negative implicit tracks

* Add basic tests for grid_auto_rows and grid_auto_columns
@nicoburns nicoburns marked this pull request as ready for review August 14, 2023 21:55
@nicoburns
Copy link
Collaborator Author

Right, let's include #497 then. If there's pushback from users then plan is:

I've backported that last PR, and also found a couple more PRs to include. FWIW, most of these had small conflicts that needed to be resolved (and/or applied cleanly but then caused test failures), so hopefully we won't need to do too many backports to the 0.3.x branch!

@nicoburns nicoburns marked this pull request as draft August 14, 2023 22:02
@nicoburns
Copy link
Collaborator Author

Converting back to a draft as Squash Merge is the only option, and I don't think this one ought to be squashed.

@nicoburns nicoburns marked this pull request as ready for review August 14, 2023 22:07
@nicoburns nicoburns merged commit a0344e2 into 0.3.x Aug 14, 2023
18 checks passed
@nicoburns nicoburns deleted the release/0.3.13 branch August 14, 2023 22:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release Preparing for the next version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants