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

Improve word rendering #17

Merged
merged 1 commit into from
Jul 23, 2020
Merged

Improve word rendering #17

merged 1 commit into from
Jul 23, 2020

Conversation

jbaiter
Copy link
Member

@jbaiter jbaiter commented Jul 23, 2020

Previously we'd use a word's Y-Position and its height as the font-size in pixels. This led to a very irregular line appearance, since words consisting of all-lowercase letters would be rendered far too small.
before_close

With this change, we now use the line's Y-position and height instead, making for a far more regular line appearance, with no problems for selecting text.
after_close_000

Previously we'd use a word's Y-Postion and its height as the font-size
in pixels. This led to a very irregular line appearance, since words
consisting of all-lowercase letter would be rendered far too small.
With this change, we now use the line's Y-position and height instead,
making for a far more regular line appearance, with no problems for
selecting text.
@codecov-commenter
Copy link

Codecov Report

Merging #17 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #17   +/-   ##
=======================================
  Coverage   62.17%   62.17%           
=======================================
  Files          10       10           
  Lines         423      423           
  Branches      130      130           
=======================================
  Hits          263      263           
  Misses        134      134           
  Partials       26       26           
Impacted Files Coverage Δ
src/components/PageTextDisplay.js 0.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 72f90ee...47ce2cc. Read the comment docs.

@morpheus-87 morpheus-87 self-requested a review July 23, 2020 10:41
@jbaiter jbaiter merged commit ca60904 into master Jul 23, 2020
@jbaiter jbaiter deleted the nicer-word-rendering branch July 23, 2020 10:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants