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

@terrysahaidak/paragraph read pixels #1

Draft
wants to merge 183 commits into
base: main
Choose a base branch
from

Conversation

terrysahaidak
Copy link

No description provided.

hans00 and others added 30 commits September 21, 2023 17:15
This reverts commit 6a99797.
)

Bumps [react-devtools-core](https://github.com/facebook/react/tree/HEAD/packages/react-devtools-core) from 4.27.1 to 4.28.4.
- [Release notes](https://github.com/facebook/react/releases)
- [Changelog](https://github.com/facebook/react/blob/main/CHANGELOG.md)
- [Commits](https://github.com/facebook/react/commits/HEAD/packages/react-devtools-core)

---
updated-dependencies:
- dependency-name: react-devtools-core
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
)

Bumps [react-devtools-core](https://github.com/facebook/react/tree/HEAD/packages/react-devtools-core) from 4.27.1 to 4.28.4.
- [Release notes](https://github.com/facebook/react/releases)
- [Changelog](https://github.com/facebook/react/blob/main/CHANGELOG.md)
- [Commits](https://github.com/facebook/react/commits/HEAD/packages/react-devtools-core)

---
updated-dependencies:
- dependency-name: react-devtools-core
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: William Candillon <[email protected]>
chrfalch and others added 30 commits November 14, 2023 17:09
Color is the one we're using.
To simplify and prepare for doing tests, the above types are now declared as simple JS types which are serializable and does not need any C++ backing.
To be able to serialize/deserialize paragraphs we also need to know the style and commands that created the paragraph. This commit fixes serialization of ParagraphStyle.
Added function getRectsForPlaceholders to both JS and C++ implementation.
When width/paragraph is the same as last time we ran layout we skip the layouting.
To align with CanvasKit API I've now added options foreground and background paint parameters to pushStyle.

Also added a tryGetArgumentAsHostObject to the JsiHostObject class that returns a nullptr if a parameter is undefined or null or another non-host object value.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants