On Wed, 27 Nov 2019 01:30:22 GMT, Kevin Rushforth <k...@openjdk.org> wrote:

> On Wed, 27 Nov 2019 01:17:42 GMT, Scott Palmer <swpal...@openjdk.org> wrote:
>> On Tue, 26 Nov 2019 18:48:38 GMT, Kevin Rushforth <k...@openjdk.org> wrote:
>>> On Tue, 26 Nov 2019 18:40:10 GMT, Scott Palmer <swpal...@openjdk.org> wrote:
>>>> On Thu, 7 Nov 2019 14:56:58 GMT, Kevin Rushforth <k...@openjdk.org> wrote:
>>>>> On Wed, 6 Nov 2019 19:11:48 GMT, Scott Palmer <swpal...@openjdk.org> 
>>>>> 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.
>>>>>> ----------------
>>>>>> Commits:
>>>>>>  - 68d221c7: 8130738: TextFlow's tab width is static
>>>>>> Changes: https://git.openjdk.java.net/jfx/pull/32/files
>>>>>>  Webrev: https://webrevs.openjdk.java.net/jfx/32/webrev.00
>>>>>>   Issue: https://bugs.openjdk.java.net/browse/JDK-8130738
>>>>>>   Stats: 324 lines in 8 files changed: 260 ins; 0 del; 64 mod
>>>>>>   Patch: https://git.openjdk.java.net/jfx/pull/32.diff
>>>>>>   Fetch: git fetch https://git.openjdk.java.net/jfx pull/32/head:pull/32
>>>>>>  I should clamp so it reads back as 1 - Fixed.
>>>>> This has implications for binding, etc., so I suspect this is not what we 
>>>>> want. I'll look into this more when I review it.
>>>>> NOTE: This will need a CSR in addition to two reviewers. You can file the 
>>>>> CSR any time, but I suggest leaving it in Draft state until Phil and I 
>>>>> have had a first pass look at the API.
>>>> The indenting was wrong in that section and I had to make some minor 
>>>> changes, so I corrected the bad indenting.  Kevin indicated, "In your 
>>>> specific case, reformatting the methods in the StyleableProperties nested 
>>>> class that are adjacent to code you add or modify seems fine, as long as 
>>>> you only change the indentation and not the line wrapping."
>>>> I didn't intend to trigger an update to this pull request...I find Git 
>>>> awkward to work with (Mercurial makes more sense to me) so I'm making 
>>>> mistakes as I bumble around. 
>>>> I do expect an update to address the way I've clamped to 1.  I agree with 
>>>> Kevin that it is probable wrong. Just waiting for feedback.
>>> I'll take a preliminary look today. I also plan to change the bug title to 
>>> make it more clear that it is an enhancement request, since it currently 
>>> reads like a bug report (I'll also change the CSR and PR title to match).
>> I just noticed that Text.toString is missing any indication of the tabSize.  
>> (Found by accident when I search for wrappingWidth to see how the javadocs 
>> were done.)
>> Is there any rule as to where in the order of properties ", tabSize = X" 
>> should be in the string?  I assume it should only be included when not set 
>> to the default value, much like lineSpacing?
> Yes, I think your idea of including `tabSize` in `Text.toString` when != 
> default is a good one.

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?

PR: https://git.openjdk.java.net/jfx/pull/32

Reply via email to