<Swing Dev> [9] Review request for 8157065: There is no the focus border on the selected tab.
Semyon Sadetsky
semyon.sadetsky at oracle.com
Mon Sep 12 15:42:22 UTC 2016
GTKPainter does not implement a lot of methods which can be accessed by
public API. Could you, please, explain, why this specific method is more
important than, for example, paintToolBarContentBackground() or
paintToggleButtonBorder(), or all other unimplemented?
In general, how do you separate public API methods of the SynthPainter
class into two sets: the first set that *should be* over-riden and the
second set of methods that *should not be* overr-riden? Are there any
systematic criterium for that differentiation?
--Semyon
On 9/12/2016 6:20 PM, Alexandr Scherbatiy wrote:
> The paintTabbedPaneTabBorder() without orientation should be
> implemented as well because it can be accessed by public API.
>
> Thanks,
> Alexandr.
>
> On 6/3/2016 10:54 PM, Semyon Sadetsky wrote:
>>
>>
>> On 6/3/2016 10:34 PM, Sergey Bylokhov wrote:
>>> On 03.06.16 22:21, Semyon Sadetsky wrote:
>>>>> What reason? Why it is not public? since I provided the code example
>>>>> where these methods are accessed by the user?
>>>> GTK toollkit painting sequence is very different.
>>>
>>> What does it mean "different"? Even in this fix you implement one of
>>> the method according to the spec and skip the same method for some
>>> unknown reason.
>>>
>>>>>
>>>>>> I still did not get why an overload method should have the same
>>>>>> behavior
>>>>>> as its associates. This is a brand new design principle I've
>>>>>> never heard
>>>>>> before.
>>>>>
>>>>> ...........
>>>> That's nice...
>>>> Do you have any other concerns?
>>>
>>> I still do not understand why the first method with default
>>> orientation is not implemented.
>> I guess you meant "is not over-ridden". :) Once again: because it is
>> never used.
>>
>
More information about the swing-dev
mailing list