<Swing Dev> [10] JDK-8004029: [macosx] The long Tab titles are not clipped with dots at the end.
Prasanta Sadhukhan
prasanta.sadhukhan at oracle.com
Thu Sep 14 10:56:16 UTC 2017
One more thing I noticed
106 } catch (Exception e) {
107 throw new RuntimeException(e);
108 }
109
110 // disposing the frame
111 SwingUtilities.invokeAndWait(() -> {
112 frame.dispose();
113 });
if there is any exception thrown as above, then we are not going to
dispose. I believe it will be better if we do try-catch-finally and add
frame.dispose() in finally block.
Also, I think it should be disposeUI() rather than frame.dispose() as we
have already called createUI() by that time so testUI frame is also present.
Also, for 2nd change, I got your point. In that case, I guess you can
bring l122-123 inside if (!status)
117 if (!status) {
118 System.out.println("Test timed out.");
119 }
120
121 if (test.testResult == false) {
122 disposeUI();
123 throw new RuntimeException("Test Failed.");
124 } Maybe you can call full thing under try-catch-finally try{
86 for(UIManager.LookAndFeelInfo laf :
UIManager.getInstalledLookAndFeels()) { ......... } catch () { throw
RuntimeException } finally { disposeUI(); } Regards Prasanta
On 9/14/2017 3:43 PM, Shashidhara Veerabhadraiah wrote:
>
> Hi Prasanta, Here is the updated Webrev.
>
> http://cr.openjdk.java.net/~sveerabhadra/8004029/webrev.03/
> <http://cr.openjdk.java.net/%7Esveerabhadra/8004029/webrev.03/>
>
> Please note that I have not done the 2^nd change(disposeUI() case) as
> this is required for the time out case where in the testresult had
> been initialized with false and would call the disposeUI() accordingly.
>
> Thanks and regards,
>
> Shashi
>
> *From:*Prasanta Sadhukhan
> *Sent:* Tuesday, September 12, 2017 11:45 AM
> *To:* Shashidhara Veerabhadraiah
> <shashidhara.veerabhadraiah at oracle.com>; swing-dev at openjdk.java.net
> *Cc:* Sergey Bylokhov <sergey.bylokhov at oracle.com>
> *Subject:* Re: <Swing Dev> [10] JDK-8004029: [macosx] The long Tab
> titles are not clipped with dots at the end.
>
> Few observations on test:
>
> init() needs to be called under EDT
> you are calling disposeUI() in line 233, so it seems there is no need
> calling again in line122
>
> Regards
> Prasanta
>
> On 9/12/2017 12:54 AM, Sergey Bylokhov wrote:
>
> Looks fine.
>
> On 9/11/17 01:34, Shashidhara Veerabhadraiah wrote:
>
> Hi, I have updated the webrev to indicate the removal of a
> file which is not required anymore owing to the change in test
> from appletviewer based to standard java execution. This is
> only for the reference.
>
> http://cr.openjdk.java.net/~sveerabhadra/8004029/webrev.02/
> <http://cr.openjdk.java.net/%7Esveerabhadra/8004029/webrev.02/>
>
> Thanks and regards,
> Shashi
>
> -----Original Message-----
> From: Shashidhara Veerabhadraiah
> Sent: Monday, September 11, 2017 9:37 AM
> To: Sergey Bylokhov <sergey.bylokhov at oracle.com>
> <mailto:sergey.bylokhov at oracle.com>; Prasanta Sadhukhan
> <prasanta.sadhukhan at oracle.com>
> <mailto:prasanta.sadhukhan at oracle.com>;
> swing-dev at openjdk.java.net <mailto:swing-dev at openjdk.java.net>
> Subject: Re: <Swing Dev> [10] JDK-8004029: [macosx] The long
> Tab titles are not clipped with dots at the end.
>
> Right Sergey. I was not sure on how to represent it under this
> Webrev. I will update the Webrev if need be.
>
> Thanks and regards,
> Shashi
>
> -----Original Message-----
> From: Sergey Bylokhov
> Sent: Saturday, September 9, 2017 7:33 AM
> To: Shashidhara Veerabhadraiah
> <shashidhara.veerabhadraiah at oracle.com>
> <mailto:shashidhara.veerabhadraiah at oracle.com>; Prasanta
> Sadhukhan <prasanta.sadhukhan at oracle.com>
> <mailto:prasanta.sadhukhan at oracle.com>;
> swing-dev at openjdk.java.net <mailto:swing-dev at openjdk.java.net>
> Subject: Re: <Swing Dev> [10] JDK-8004029: [macosx] The long
> Tab titles are not clipped with dots at the end.
>
> Hi, Shashi.
> The fix looks fine, but it looks like bug4310381.html should
> be removed because it is not used?
>
> On 9/4/17 02:33, Shashidhara Veerabhadraiah wrote:
>
> Hi All, Please find the updated Webrev at the below link.
>
> http://cr.openjdk.java.net/~sveerabhadra/8004029/webrev.01/
> <http://cr.openjdk.java.net/%7Esveerabhadra/8004029/webrev.01/>
>
>
> I have added a new test that would go through different
> laf's and runs
> the test of the tab titles. Previously this test had been
> a single
> default laf test with appletviewer and now replaced for
> multi laf
> testing along with the normal java execution. There are
> instances of
> inaccurate rendering of the titles with respect to the tab
> pane
> component like clipping, starting offset and going beyond
> the tab pane
> size as shown below in some of the examples:
>
> This requires a different thread to really understand why
> this
> variation across the multiple laf representation and needs
> more proper
> bounding the title to fit properly within the tab pane
> space. Hence I
> will be raising multiple bugs to indicate these current
> issues and
> will be resolved later. Please note that this issue is
> /_not_/ because
> of the clipping of the text that these current changes
> that are done
> under this bug fix.
>
> Thanks and regards,
>
> Shashi
>
> -----Original Message-----
> From: Sergey Bylokhov
> Sent: Thursday, August 24, 2017 4:24 AM
> To: Prasanta Sadhukhan <prasanta.sadhukhan at oracle.com>
> <mailto:prasanta.sadhukhan at oracle.com>; Shashidhara
> Veerabhadraiah <shashidhara.veerabhadraiah at oracle.com>
> <mailto:shashidhara.veerabhadraiah at oracle.com>;
> swing-dev at openjdk.java.net
> <mailto:swing-dev at openjdk.java.net>
> Subject: Re: <Swing Dev> [10] JDK-8004029: [macosx] The
> long Tab
> titles are not clipped with dots at the end.
>
> I am not sure is it possible to prove, but I think that we
> need to
> pass tabPane as a component to ClipStrinUfNecessary.
>
> On 17.08.2017 3:48, Prasanta Sadhukhan wrote:
>
> > Fix looks good.
>
> >
>
> > But, can you update
>
> > test/javax/swing/JTabbedPane/4310381/bug4310381.java
>
> > to include the test for all installed l&fs so that we
> can see if
> there
>
> > is problem in any other l&fs?
>
> >
>
> > Regards
>
> > Prasanta
>
> > On 8/17/2017 2:08 PM, Shashidhara Veerabhadraiah wrote:
>
> >>
>
> >> Hi All, Please review a fix for the JDK- 8004029
> where the long
> tab
>
> >> titles were not clipped with the dots at the end to
> indicate a
>
> >> lengthy title. The aqua look and feel which is the
> default laf for
>
> >> mac does the title clipping only leading to an
> ambiguous UI where
> in
>
> >> the clipped title would look like the /_complete_/
> title though it
> is not.
>
> >> Hence it is good to show dots at the end indicating a
> much bigger
>
> >> title exists though the tab does not have enough real
> estate to
>
> >> display the complete title.
>
> >>
>
> >> _Solution and fix:_ I have updated the aqua laf
> module to clip the
>
> >> title text and put the dots at the end if the text
> size is larger
>
> >> than the tab size. Below is the picture /_after_/ the
> fix.
>
> >>
>
> >> Bug: https://bugs.openjdk.java.net/browse/JDK-8004029
>
> >>
>
> >> Webrev:
>
> >>
> http://cr.openjdk.java.net/~aghaisas/shashi/8004029/webrev.00/
> <http://cr.openjdk.java.net/%7Eaghaisas/shashi/8004029/webrev.00/>
>
>
> >>
> <http://cr.openjdk.java.net/%7Eaghaisas/shashi/8004029/webrev.00/>
> <http://cr.openjdk.java.net/%7Eaghaisas/shashi/8004029/webrev.00/>
>
>
> >>
>
> >> Thanks and regards,
>
> >>
>
> >> Shashi
>
> >>
>
> >
>
> --
>
> Best regards, Sergey.
>
>
>
> --
> Best regards, Sergey.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/swing-dev/attachments/20170914/13a9bf1d/attachment.html>
More information about the swing-dev
mailing list