pivot-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sandro Martini (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (PIVOT-754) Pivot displays ugly gray box sometimes before it loads the applet
Date Thu, 08 Aug 2013 16:01:48 GMT

    [ https://issues.apache.org/jira/browse/PIVOT-754?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13733633#comment-13733633
] 

Sandro Martini commented on PIVOT-754:
--------------------------------------

Reassigned, because Poitr is no more active in the project. Is someone interested ?

Verify the proposed patch, to avoid side effects.

                
> Pivot displays ugly gray box sometimes before it loads the applet
> -----------------------------------------------------------------
>
>                 Key: PIVOT-754
>                 URL: https://issues.apache.org/jira/browse/PIVOT-754
>             Project: Pivot
>          Issue Type: Improvement
>          Components: wtk
>    Affects Versions: 2.0, 2.0.1, 2.0.2
>         Environment: Observed both on Linux and Windows Vista, Firefox 3.x and Firefox
4.0.1
> Java 6 update 25.
>            Reporter: Piotr Kołaczkowski
>            Priority: Minor
>             Fix For: 2.5
>
>         Attachments: pivot-graybox.patch
>
>
> When the applet starts, a progress bar is displayed. When the applet finishes downloading,
the progress bar disappears, leaving a gray box until the pivot toolkit loads the application.
Even for the simpliest application displaying an empty window, pivot requires about 0.3 s
to start (on Core2Duo 2.2 GHz), so the gray box effect can be easily seen. The gray box effect
does not happen always, but in about half of the cases.
> Below I submit patch we made to the BrowserApplicationContext.java file. Seems like setting
the applet's background to white and delaying the add(displayHost) call as late as possible
does the trick:
> [code]
> 51a52
> >         private boolean displayHostInstalled = false;
> 151c152,154
> <                 add(displayHost);
> ---
> > 
> >                 // Don't add it now, add it later, to avoid ugly gray box:
> >                 //add(displayHost);
> 160c163,164
> <                 setBackground(null);
> ---
> >                 // Huh? WTF?
> >                 // setBackground(null);
> 191a196
> >                         installDisplayHostIfNeeded();
> 245a251,257
> >         private void installDisplayHostIfNeeded() {
> >             if (!displayHostInstalled) {
> >                 this.add(displayHost);
> >                 displayHostInstalled = true;
> >             }
> >         }
> > 
> 247a260
> >             this.setBackground(Color.white);
> 309a323
> >             installDisplayHostIfNeeded();
> [/code]

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Mime
View raw message