<Swing Dev> [9] Review request for 8157065: There is no the focus border on the selected tab.

Alexandr Scherbatiy alexandr.scherbatiy at oracle.com
Mon Sep 12 15:50:28 UTC 2016


On 9/12/2016 6:42 PM, Semyon Sadetsky wrote:
> 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?
   All the same methods with different number of arguments which do not 
fall to overridden implementation should be overridden to provide proper 
implementation.

   Thanks,
   Alexandr.
>
> --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