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

Showcase bounds issue #411

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mickaelistria
Copy link
Contributor

This shows the bug in TextLayout.getTextBounds(...) that later affects caret positioning or other rendering with glyph metrics and code minings

@github-actions
Copy link
Contributor

Unit Test Results

     304 files       304 suites   6m 27s ⏱️
  4 021 tests   4 011 ✔️   5 💤   5
12 035 runs  11 958 ✔️ 67 💤 10

For more details on these failures, see this check.

Results for commit de2acaa.

@mickaelistria
Copy link
Contributor Author

and as expected, this same test passes on RHEL 7 (see #412)

@github-actions
Copy link
Contributor

github-actions bot commented Jan 5, 2023

Test Results

     306 files       306 suites   6m 3s ⏱️
  4 049 tests   4 035 ✔️   8 💤   5  1 🔥
12 074 runs  11 993 ✔️ 70 💤 10  1 🔥

For more details on these failures and errors, see this check.

Results for commit 456b356.

♻️ This comment has been updated with latest results.

This shows the bug in TextLayout.getTextBounds(...) that later affects
caret positioning or other rendering with glyph metrics and code minings
@jukzi
Copy link
Contributor

jukzi commented Jun 15, 2023

@mickaelistria if you do not plan to merge this please close or convert to draft

@mickaelistria mickaelistria marked this pull request as draft June 15, 2023 11:16
@mickaelistria
Copy link
Contributor Author

mickaelistria commented Jun 15, 2023

I don't intend to merge now as it would fail similarly to #411 ; but this test is still interesting, particularly if anyone is interested in fixing the issue, as it's the current minimal reproducer.

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.

None yet

2 participants