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

Cursor Height #4

Open
scientificware opened this issue Jul 30, 2022 · 0 comments
Open

Cursor Height #4

scientificware opened this issue Jul 30, 2022 · 0 comments
Assignees
Labels
Bug Something isn't working WIP Work In Progress

Comments

@scientificware
Copy link
Owner

scientificware commented Jul 30, 2022

The purpose of this issue is to solve problems with MathML and caret height.

This is tracked in WebKit Bugzilla Stream as

Status :

Description Understood Patch Writing Internal Tests Pull Resqueted Accepted
✔️ ✔️ 🚧

?

  • Description : The caret has the same size than the character (but not the font size, see JavaFX and MathML. javafxports/openjdk-jfx#71). This problem is specific to the MathML support. With normal text, the caret height is the font height. Solution : Anyway the implemented logic is not correct : the cursor size has to be equal to the parent or height (that's why, it is important to correct MathML code when loads a file or paste, see below about these add and remove corrections). In the following example, move from the left to the right, you can see that the caret height is good outside the formula and too small inside.

    <p>Just a fraction example to test the Caret moves : 
       <math>
          <mrow>
             <mi>A</mi>
             <mo>=</mo>
             <mfrac>
                <mrow><mn>2</mn></mrow>
                <mrow><mn>8</mn></mrow>
             </mfrac>
             <mo>+</mo>
             <mfrac>
                <mrow>
                   <mn>12</mn>
                   <mo>-</mo>
                   <mfrac>
                      <mrow><mn>567</mn></mrow>
                      <mrow><mn>17</mn></mrow>
                   </mfrac>
                </mrow>
                <mrow><mn>37858727</mn></mrow>
             </mfrac>
          </mrow>
       </math>
      . But you can do the calculus, if you want.
    </p>
    

    The caret height should be the same on the two pictures. Its height should be its height just after the word 'test'. That's the usual behavior. So, its size before '+' is wrong.
    screenshot_20180813_004449
    screenshot_20180813_004544

@scientificware scientificware added the Bug Something isn't working label Aug 8, 2022
@scientificware scientificware self-assigned this Aug 9, 2022
@scientificware scientificware added the WIP Work In Progress label Aug 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working WIP Work In Progress
Projects
None yet
Development

No branches or pull requests

1 participant