[
https://issues.apache.org/jira/browse/PIVOT-754?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13733633#comment-13733633
]
Sandro Martini edited comment on PIVOT-754 at 8/8/13 4:00 PM:
--------------------------------------------------------------
Reassigned, because Piotr is no more active in the project. Is someone
interested ?
Verify the proposed patch, to avoid side effects.
was (Author: smartini):
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
