<Swing Dev> [12] RFR JDK-8203281:[Windows] JComboBox change in ui when editor.setBorder() is called

[Martin M]

Hi Prasanta,

As far as I know, there are only 2 options in swing on windows.
WindowsLookAndFeel (includes XP theme, aero for Vista, Win7, Win10...) and
WindowsClassicLookAndFeel (includes classic theme for win95, win98).
The fix was done for WindowsLookAndFeel, so it affects not only Vista
version but also win7 aero and win10.

I think that EmptyBorder(0,0,0,0) should be replaced by
EmptyBorder(1,1,1,1) to see animated native border and therefore
BasicComboboxEditor will not be placed over native border. But then
arrowbutton will be placed on wrong coordinates and also its size will not
be ok.
I will create webrev with needed changes.

br,
Martin

št 6. 9. 2018 o 19:34 Prasanta Sadhukhan <prasanta.sadhukhan at oracle.com>
napísal(a):

> Ping? It's been a month.
>
> Even if JDK-6490753 fix is not only for Vista, but a portion of it
> [where I propose a fix] is causing issue in windows7, 10, so it is
> modified to make sure it works correctly in windows7-10. It will also
> not affect windows vista.
> Please provide comments on this.
>
> Regards
> Prasanta
> On 8/9/2018 3:09 PM, Prasanta Sadhukhan wrote:
> > Gentle reminder.
> >
> > I guess it's for vista as the bug description states it's for
> >
> > Vista Info:
> > ===========
> > Vista Build No: 5840
> > Vista Theme: Default Theme(Aero)
> >
> > Regards
> > Prasanta
> > On 8/2/2018 12:27 AM, Sergey Bylokhov wrote:
> >> Hi, Prasanta.
> >> I am not sure that previous fix[JDK-6490753] was implemented for
> >> Vista only.
> >> Maybe Martin (CC) can take a look to this?
> >>
> >> On 01/08/2018 10:11, Prasanta Sadhukhan wrote:
> >>> Hi All,
> >>>
> >>> Please review a fix for an issue where it is seen that combobox gets
> >>> an unwanted border around it.
> >>> This is a aftereffect of JDK-6490753
> >>> <https://bugs.openjdk.java.net/browse/JDK-6490753> where am empty
> >>> border is added to combobox to make it look like Vista native combobox.
> >>> Proposed fix is to make sure no empty border is drawn if windows
> >>> version is not vista.
> >>> Here's the combobox looks like before and after fix in windows 10
> >>> and 7.
> >>>
> >>> Before fix
> >>>
> >>> After Fix
> >>>
> >>> Bug: https://bugs.openjdk.java.net/browse/JDK-8203281
> >>> webrev: http://cr.openjdk.java.net/~psadhukhan/8203281/webrev.0/
> >>>
> >>> Regards
> >>> Prasanta
> >>
> >>
> >
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/swing-dev/attachments/20180908/5cc121b3/attachment.html>