<html>
<head>
<meta name="viewport" content="width=device-width">
<meta http-equiv="Content-Type" content="text/vnd.ui.insecure+html;charset=utf-8">
</head>
<body style="overflow-wrap:break-word; word-break: break-word;"><div class="mail_android_message" style="line-height: 1; padding: 0.5em">I think this is a good idea.<br/><br/>I have checked the CSS code some time myself and also removed all the special property checks just to see what will happen and why it is needed.<br/>With the gathered knowledge, I can confirm what you wrote down in your proposal and that you have the correct idea to solve this problem. <br/><br/>Right now, the CSSParser tries to guess what the value is used for. If there is something with "px", well it must be some kind of unit hopefully. If there is "ms", it must be time. That works most of the time, but not always. It therefore also works most of the time for your own properties, but again, may not always.<br/><br/>So the proposed solution of a parsing and a interpretation phase sound correct and should solve the problem. It also simplify things.<br/><br/>So this is something I will also give my +1 to it.<br/>Regarding tests, when you work on that proposal and add tests for the new CSSParser, we could check if it is easy to "cherrypick" them to the existing CSSParser so that we can verify and guarantee that everything still works, maybe also that some things do not work and fail as expected.<br/><br/>-- Marius</div><div class="mail_android_quote" style="line-height: 1; padding: 0.3em"><html><body>Am 26.08.24, 05:47 schrieb "Michael Strauß" <michaelstrau2@gmail.com>:</body></html><blockquote class="gmail_quote" style="margin: 0.8ex 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
I've written up a proposal to implement a new CSS parser for JavaFX:
<br><a href="https://gist.github.com/mstr2/f416996caf48e11193f0b6a5883a3926">https://gist.github.com/mstr2/f416996caf48e11193f0b6a5883a3926</a>
<br>
<br>
The goal is not add new features at this point, but to resolve some
<br>
long-standing issues with the existing CSS parsing (though if you read
<br>
carefully, you might spot a new feature).
<br>
<br>
I'm interested in your opinion whether this is a good idea to move forward.
<br>
</blockquote></div></body>
</html>