[gwt-contrib] Re: Working CodeSplitter2. (issue1631803)

2012-02-02 Thread acleung
http://gwt-code-reviews.appspot.com/1631803/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: CellTable performance is very slow on IE8. Internally, AbstractCellTable detects (issue1635803)

2012-02-02 Thread Aladdin
It's IE , but be sure that you disabled Dev Tools and then check. I had to uninstall IE-7-Dev Tools that was installed in IE-8. My problem with IE was not related to your problem but I think it might help. Good Luck On Feb 2, 10:19 pm, is...@google.com wrote: > Reviewers: jlabanca, > > Descripti

[gwt-contrib] Fixed issue 1394 : Need a new getSplitter() method in SplitPanel (issue1398801)

2012-02-02 Thread jlabanca
LGTM A couple of minor nits, but looks good. I'll submit it the next time I make a pass over external submissions. I can fix the nits that I noted. http://gwt-code-reviews.appspot.com/1398801/diff/1/user/src/com/google/gwt/user/client/ui/DockLayoutPanel.java File user/src/com/google/gwt/user/

[gwt-contrib] Re: CellTable performance is very slow on IE8. Internally, AbstractCellTable detects (issue1635803)

2012-02-02 Thread jlabanca
LGTM I wonder if the slowness is also related to descendant selectors. Descendant selectors that affect TD elements are very slow on tables in IE and would affect row hover. http://gwt-code-reviews.appspot.com/1635803/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] CellTable performance is very slow on IE8. Internally, AbstractCellTable detects (issue1635803)

2012-02-02 Thread isoos
Reviewers: jlabanca, Description: CellTable performance is very slow on IE8. Internally, AbstractCellTable detects row hover events, and part of that code will access absolute coordinates of the row (to safeguard the case where we have items with fixed positions over the table). Unfortunately, IE

[gwt-contrib] Re: Firefox 10 DevMode Plugin (issue1634803)

2012-02-02 Thread conroy
LGTM http://gwt-code-reviews.appspot.com/1634803/diff/1/plugins/xpcom/ExternalWrapper.cpp File plugins/xpcom/ExternalWrapper.cpp (right): http://gwt-code-reviews.appspot.com/1634803/diff/1/plugins/xpcom/ExternalWrapper.cpp#newcode267 plugins/xpcom/ExternalWrapper.cpp:267: bool remember = false;

Re: [gwt-contrib] CSS3 support in CssResource: Closure Stylesheets?

2012-02-02 Thread Thomas Broyer
On Thu, Feb 2, 2012 at 7:05 AM, Ray Cromwell wrote: > Sorry to hear that. This is actually on our list of priorities for > GWT, so we will have to take this up internally. Just a clarification: I was trying to replace Flute with the parser from Closure Stylesheets, replacing the whole CssResource