<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
Fri Sep 15 09:51:08 UTC 2017
+1
Regards
Prasanta
On 9/15/2017 3:16 PM, Shashidhara Veerabhadraiah wrote:
>
> Hi Prasanta, Please find the updated Webrev @
> http://cr.openjdk.java.net/~sveerabhadra/8004029/webrev.05/
> <http://cr.openjdk.java.net/%7Esveerabhadra/8004029/webrev.05/>
>
> Thanks and regards,
>
> Shashi
>
> *From:*Prasanta Sadhukhan
> *Sent:* Friday, September 15, 2017 3:11 PM
> *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.
>
> looks better. but, disposeUI() do not throw any exception so no need
> to catch in
>
> try {
> 122 if(test != null) {
> 123 test.disposeUI();
> 124 }
> 125 } catch (Exception ex) {
> 126 throw new RuntimeException("Exception while
> disposing UI");
> 127 }
> Regards
> Prasanta
>
> On 9/15/2017 2:57 PM, Shashidhara Veerabhadraiah wrote:
>
> Hi, Here is the updated webrev. Removed unnecessary try catch blocks!!
>
> http://cr.openjdk.java.net/~sveerabhadra/8004029/webrev.04/
> <http://cr.openjdk.java.net/%7Esveerabhadra/8004029/webrev.04/>
>
> Thanks and regards,
>
> Shashi
>
> *From:*Prasanta Sadhukhan
> *Sent:* Thursday, September 14, 2017 4:26 PM
> *To:* 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>
> *Cc:* Sergey Bylokhov <sergey.bylokhov at oracle.com>
> <mailto: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.
>
> 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>
> <mailto:shashidhara.veerabhadraiah at oracle.com>;
> swing-dev at openjdk.java.net <mailto:swing-dev at openjdk.java.net>
> *Cc:* Sergey Bylokhov <sergey.bylokhov at oracle.com>
> <mailto: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/20170915/9d10db5d/attachment.html>
More information about the swing-dev
mailing list