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

Calculatable Shadow #1455

Merged
merged 18 commits into from
Jul 20, 2023
Merged

Calculatable Shadow #1455

merged 18 commits into from
Jul 20, 2023

Conversation

Jym77
Copy link
Contributor

@Jym77 Jym77 commented Jul 18, 2023

Part of #1202

@Jym77
Copy link
Contributor Author

Jym77 commented Jul 18, 2023

!pr extract

@Jym77 Jym77 self-assigned this Jul 18, 2023
@Jym77 Jym77 added the major Backwards-incompatible change that touches public API label Jul 18, 2023
@Jym77 Jym77 marked this pull request as ready for review July 18, 2023 13:32
@Jym77 Jym77 requested review from a team and rcj-siteimprove July 18, 2023 13:32
@Jym77
Copy link
Contributor Author

Jym77 commented Jul 18, 2023

!pr extract

@Jym77 Jym77 added this pull request to the merge queue Jul 20, 2023
Merged via the queue into main with commit ade5b0a Jul 20, 2023
@Jym77 Jym77 deleted the calculatable-shadow branch July 20, 2023 13:06
Jym77 added a commit that referenced this pull request Aug 9, 2023
* Typo

* Add some package implementation details

* Streamline code

* Streamline parser

* Clean up test indentation

* Add some shadow teest

* Eport Option interface

* Add more shadow parsing tests

* Add a resolve test

* Make Shadow calculatable

* Add changesets

* Extract API

* Typos

* Clean up

* Actually accept calculations

* Add Value.hasCalculation helper

* Rename parameter

* Extract API

---------

Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
github-merge-queue bot pushed a commit that referenced this pull request Oct 3, 2023
* Extract position to its own folder

* Move Offset to its own file

* Move Keywords and Side to their own files

* Move Component to its own file

* Remove Offset and use LengthPercentage directly

* Make Side somewhat calculatable

* Switch parameter order

* Prepare partial resolving

* Make Component always take a Side

* Add missing changeset

* Make offest optional in Side.of

* Make offset optional or Option in Side.of

* Add partial resolver, rework type parameters

* Make Position calculatable

* Add some parsing examples with calculations

* Add some resolving examples with calculations

* Resolve percentages separatly in each dimension

* Add changeset

* Clean up

* Extract API

* Clean up

* Typo

* Extract API

* Add Resolvable implementation

* Extract API

* Calculatable `Shadow` (#1455)

* Typo

* Add some package implementation details

* Streamline code

* Streamline parser

* Clean up test indentation

* Add some shadow teest

* Eport Option interface

* Add more shadow parsing tests

* Add a resolve test

* Make Shadow calculatable

* Add changesets

* Extract API

* Typos

* Clean up

* Actually accept calculations

* Add Value.hasCalculation helper

* Rename parameter

* Extract API

---------

Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>

* Calculatable `transform` functions (#1457)

* Streamline parsers

* Clean up

* Share parsers in tests

* Add Matrix parsing tests

* Simplify Matrix parser

* Improve parseIf

* Improve parser

* Add tests for Rotate prasing

* Add tests for Rotate prasing

* Simplify parser

* Simplify parser

* Add Scale parsing tests

* Simplify Scale parser

* Simplify parsers

* Add Skew parsing tests

* Simplify Skew parsers

* Add Translate tests

* Simplify parsers

* Clean up

* Extract API

* Typos

* Make translate Function calculatable

* Make matrix calculatable

* Make matrix calculatable

* Streamline Perspective

* Make Perspective calculatable

* Make Rotate calculatable

* Add calculated rotate tests

* Add calculated rotate tests

* Make Scale calculatable

* Accept calculation in Skew

* Make Translate calculatable

* Accept calculation in all transform function

* Clean up

* Extract API

* Clean up

* Extract API

* Fix type inferrence

* Extract API

---------

Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>

* SIA-R19: fix regression (#1459)

* Add breaking test

* Improve documentation

* Correctly search for id in the full tree

* Update documentation

* Add changeset

* v0.66.0

* Remove duplicate variable

---------

Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: siteimprove-builduser <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
major Backwards-incompatible change that touches public API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants