[
https://issues.apache.org/jira/browse/PIVOT-754?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13047443#comment-13047443
]
Piotr Kołaczkowski commented on PIVOT-754:
------------------------------------------
I don't think it is needed to distribute proguarded jars. I suggested only
proguarding and packing the applications used as examples on the site, and
reviewing if they are deployed correctly. The first time I ran the examples,
the progress bar disappeared too early, and the application was still
downloading, while presenting me an ugly gray box. When jars have been cached,
everything is fine, so you might not notice this.
> 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: Bug
> Components: wtk
> Affects Versions: 2.0
> 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.0.1
>
> 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.
For more information on JIRA, see: http://www.atlassian.com/software/jira
