这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@Weibye
Copy link
Collaborator

@Weibye Weibye commented Jun 23, 2022

Objective

Fixes #114

Replaces #160 and #148

Context

The new Resolve-traits really helped clean this up :)

Feedback wanted

  • I'm not yet sure why the simple_child test are failing

@Weibye
Copy link
Collaborator Author

Weibye commented Jun 23, 2022

I suspect the current failures comes from Measure functions assuming Size<Dimension> and not Size<Option<Dimension>> so that is what I will investigate next

@Weibye Weibye changed the title Remove Dimension::Undefined v3 Remove Dimension::Undefined Jun 24, 2022
@Weibye
Copy link
Collaborator Author

Weibye commented Jun 24, 2022

The generated files are still showing in the review. Is #149 not working as intended?

Weibye and others added 14 commits June 26, 2022 20:07
* Cleanup `new_with_children()` uses

* Format

* Update scripts/gentest/src/main.rs

Co-authored-by: Alice Cecile <alice.i.cecile@gmail.com>
Updates the requirements on [rstest](https://github.com/la10736/rstest) to permit the latest version.
- [Release notes](https://github.com/la10736/rstest/releases)
- [Changelog](https://github.com/la10736/rstest/blob/master/CHANGELOG.md)
- [Commits](la10736/rstest@0.14.0...0.15.0)

---
updated-dependencies:
- dependency-name: rstest
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Forest unit tests

* Removing old Forest compute_layout uses

* Forest unit tests additions and refactoring

* Code style adjustments

* Removing Forest node_measure_eq warning

Co-authored-by: Alice Cecile <alice.i.cecile@gmail.com>
…oxusLabs#196)

* feat: use slab instead of bespoke arena

* fix: all but one test

* chore: rip out forest and merge it into taffy

* fix: final tests

* polish: fix up some warnings

* chore: clean up forest and data fails

* chore: make clippy happier

* fix: compilation for no-std targets

* chore: clean up some more clippy stuff and renable more tests

* chore: remove outdated indexmap

* fix: update benches to not unwrap new elements

* fix: revert back to errors everywhere

* fix: revert complex bench too

* chore: remove unused import

* fix: removing parents

* chore: clean up and remove references to forest

* chore: remove final reference to forest

* chore: update release log

* Update error name in RELEASES

* Add single parent fix to RELEASES

* Remove extra whitespace

* chore: add warning to mark dirty function

Co-authored-by: Alice Cecile <alice.i.cecile@gmail.com>
@Weibye
Copy link
Collaborator Author

Weibye commented Jul 14, 2022

🤔 Tests fails on height calculation but all width calculations are correct. Trying to dig into why that is but progress is slow (lots of new things to learn)

@alice-i-cecile
Copy link
Collaborator

Going to close this out: this will be a pain to rebase. Anyone can feel free to remake this though!

@Weibye
Copy link
Collaborator Author

Weibye commented Nov 19, 2022

Yeah, it was a worthwhile attempt but I couldn't get it across the finish line due to some nuances in the algorithm I could not fully figure out. Now with taffy undergoing some great improvements I think this could be easier to attempt again after #238 and #246 should it still prove useful to do.

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

Labels

breaking-change A change that breaks our public interface code quality Make the code cleaner or prettier.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Consider removing Dimension::Undefined