On Fri, 20 Dec 2019 00:19:43 GMT, Kevin Rushforth wrote:
>> I was thinking of deferring the `apps/toys` demo to avoid any delays in
>> getting the new API into JavaFX 14. I have something for it, I just don't
>> want any feedback on it to hold up the review of this issue. Is there
>> anything
On Thu, 19 Dec 2019 13:48:11 GMT, Scott Palmer wrote:
>>
>
> I was thinking of deferring the `apps/toys` demo to avoid any delays in
> getting the new API into JavaFX 14. I have something for it, I just don't
> want any feedback on it to hold up the review of this issue. Is there
> anything
On Thu, 12 Dec 2019 21:51:56 GMT, Nir Lisker wrote:
>> The pull request has been updated with 1 additional commit.
>
>
I was thinking of deferring the `apps/toys` demo to avoid any delays in getting
the new API into JavaFX 14. I have something for it, I just don't want any
feedback on it to
On Fri, 13 Dec 2019 01:10:48 GMT, Scott Palmer wrote:
>> modules/javafx.graphics/src/main/java/javafx/scene/text/Text.java line 1450:
>>
>>> 1449: private static final CssMetaData TAB_SIZE =
>>> 1450: new CssMetaData("-fx-tab-size",
>>> 1451: SizeConverter
On Fri, 13 Dec 2019 04:42:07 GMT, Scott Palmer wrote:
>> We are referring to the character here aren't we ? ie the actual character
>> and the rest of it is about how it renders.
>> Paraphrasing the java doc it says :
>> If you display it will display as N instances of > space character>
>
> I
On Fri, 13 Dec 2019 01:29:47 GMT, Phil Race wrote:
>> The terms "tab character" or "horizontal tab" refer to the ASCII tab
>> character itself. Since a tab character isn't a fixed number of spaces,
>> changing it to "size of a tab character" could be misleading. I'd be fine
>> with another alt
On Thu, 12 Dec 2019 22:02:53 GMT, Kevin Rushforth wrote:
>> modules/javafx.graphics/src/main/java/javafx/scene/text/Text.java line 1273:
>>
>>> 1272: /**
>>> 1273: * The size of a tab stop in spaces.
>>> 1274: * Values less than 1 are treated as 1.
>>
>> "tab stop" seems to be an
On Thu, 12 Dec 2019 21:41:12 GMT, Nir Lisker wrote:
>> The pull request has been updated with 1 additional commit.
>
> modules/javafx.graphics/src/main/java/javafx/scene/text/Text.java line 1883:
>
>> 1882: }
>> 1883: @Override protected void invalidated(
On Thu, 12 Dec 2019 21:29:06 GMT, Nir Lisker wrote:
>> The pull request has been updated with 1 additional commit.
>
> modules/javafx.graphics/src/main/java/javafx/scene/text/Text.java line 1450:
>
>> 1449: private static final CssMetaData TAB_SIZE =
>> 1450: new CssMeta
On Thu, 12 Dec 2019 21:40:34 GMT, Nir Lisker wrote:
>> The pull request has been updated with 1 additional commit.
>
> modules/javafx.graphics/src/main/java/javafx/scene/text/Text.java line 1882:
>
>> 1881: return StyleableProperties.TAB_SIZE;
>> 1882:
On Thu, 12 Dec 2019 21:16:32 GMT, Nir Lisker wrote:
>> The pull request has been updated with 1 additional commit.
>
> modules/javafx.graphics/src/main/java/javafx/scene/text/Text.java line 1273:
>
>> 1272: /**
>> 1273: * The size of a tab stop in spaces.
>> 1274: * Values less th
On Thu, 12 Dec 2019 21:52:05 GMT, Scott Palmer wrote:
>> Added tabSize property to Text and TextFlow and -fx-tab-size CSS attribute
>> to both. TextFlow's tab size overrides that of contained Text nodes.
>
> The pull request has been updated with 1 additional commit.
modules/javafx.graphics/s
On Thu, 12 Dec 2019 19:48:37 GMT, Scott Palmer wrote:
>> In that case, I recommend just doing the API get/set tests for `TextFlow`
>> without creating a `Scene` or `Stage`. This won't need anything from the
>> `StubToolkit`.
>
> In my attempts to address the issue with StubTextLayout I discove
On Thu, 12 Dec 2019 15:54:11 GMT, Kevin Rushforth wrote:
>> The following comment from @swpalmer sent to the openjfx-dev mailing list
>> wasn't mirrored in the PR (I have alerted the Skara team about this).
>>
>>
>> From: Scott Palmer
>>
>>> I think a new `TextFlowTest.java` would be a go
On Thu, 12 Dec 2019 15:52:23 GMT, Kevin Rushforth wrote:
>> Good catch. Yes, please update it as you suggested.
>
> The following comment from @swpalmer sent to the openjfx-dev mailing list
> wasn't mirrored in the PR (I have alerted the Skara team about this).
>
>
> From: Scott Palmer
>
On Thu, 12 Dec 2019 15:19:39 GMT, Kevin Rushforth wrote:
>> I just noticed while updating the CSR that, while I mentioned it on the
>> mailing list, the fact that TextFlow's tabSize override that of any
>> contained Text nodes is not documented.
>> Shall I add to the javadoc for tabSize in Tex
On Thu, 12 Dec 2019 15:15:38 GMT, Scott Palmer wrote:
>> I think a new `TextFlowTest.java` would be a good place for those tests.
>>
>> Our build is set up to use `ant` so if you want to wire it up to the build,
>> you'll need that (it should be as simple as having `ANT_HOME` set to
>> `apache
On Wed, 11 Dec 2019 01:22:54 GMT, Kevin Rushforth wrote:
>> The unit tests that were already added to `TextTest.java` cover the new
>> methods on Text, but not in every combination. I'll add a couple more to
>> ensure all combinations are covered. `TextFlow` has no existing unit tests
>> that
> On Dec 10, 2019, at 8:23 PM, Kevin Rushforth wrote:
...
>
> I think a new `TextFlowTest.java` would be a good place for those tests.
My first attempt at unit tests for TextFlow are failing. I believe the
StubTextLayout is not equipped to handle TextFlow. This may be a bigger job…
This f
On Wed, 11 Dec 2019 01:04:15 GMT, Scott Palmer wrote:
>> As a follow-on point to the missing public method in TextFlow, can you add
>> unit tests for the API methods on both `Text` and `TextFlow`? A good way to
>> do that is to have a test for all combinations of setting the value via the
>> s
On Tue, 10 Dec 2019 23:51:48 GMT, Kevin Rushforth wrote:
>>> Should this PR also add a tabSize property to controls such as TextArea? Or
>>> should that be a different PR after this one is merged?
>>
>> This would need to be a new enhancement and would first need to be discussed
>> on the open
On Tue, 10 Dec 2019 18:54:08 GMT, Kevin Rushforth wrote:
>> Overall this looks good to me with one "must fix" API issue and one
>> additional minor comment.
>>
>> In addition to the automated unit test, it might be nice to have a simple
>> app (in `apps/toys`) with a slider to control the tab
On Tue, 10 Dec 2019 18:37:50 GMT, Kevin Rushforth wrote:
>> The pull request has been updated with 1 additional commit.
>
> Overall this looks good to me with one "must fix" API issue and one
> additional minor comment.
>
> In addition to the automated unit test, it might be nice to have a sim
On Tue, 10 Dec 2019 18:38:04 GMT, Scott Palmer wrote:
>> Added tabSize property to Text and TextFlow and -fx-tab-size CSS attribute
>> to both. TextFlow's tab size overrides that of contained Text nodes.
>
> The pull request has been updated with 1 additional commit.
Overall this looks good t
The pull request has been updated with additional changes.
Added commits:
- af959665: 8130738: Add tabSize property to Text and TextFlow
Changes:
- all: https://git.openjdk.java.net/jfx/pull/32/files
- new: https://git.openjdk.java.net/jfx/pull/32/files/254c40de..af959665
25 matches
Mail list logo
