This is going to directly impact the selection script and permutation-based
compilation of GWT 2.x.
I've read several times that some people use a single user.agent value
(can't remember if it's gecko1_8 or safari), do we have some guarantees
that this will work reliably across "evergreen
On Wednesday, May 1, 2019 at 8:58:03 PM UTC+2, foal wrote:
>
> Easly to update in upcoming releases than explain each other that it isn't
> critical :)
>
> BTW GWT-RPC user protobuf?
>
The protobuf in gwt-servlet is an internal dependency for sourcemaps and
streamhtmlparser (used in
On Wednesday, May 1, 2019 at 8:39:37 PM UTC+2, ga...@rstudio.com wrote:
>
> Existing application using Gwt 2.8.2, and I'm undertaking effort to make
> it accessible (keyboard, screen reader, and so forth). Always a challenge
> to retrofit accessibility when it wasn't considered from the start,
Hi all,
3½ years ago, we announced 2.8.0-beta1 and that it now required JDK 7, and
that started quite a long discussion:
https://groups.google.com/d/topic/google-web-toolkit-contributors/TzsINiDf5xg/discussion
A few days ago, there's been renewed interest into upgrading the Jetty
version GWT
In my app GWT has a single RPC, I want to know if Atmosphere Async-IO can
help Multiplexing.
if possible, any resource would be great ?
Thank you!
--
You received this message because you are subscribed to the Google Groups
Google Web Toolkit group.
To unsubscribe from this group and stop
Which version of GWT supports fully 508 compliance
On Friday, 3 September 2010 05:49:39 UTC+5:30, Google Integrator wrote:
How would I make my GWT application 508 Compliant? What are the best
strategies currently out there for this. I am writing an application
that makes a lot of calls to
Hi,
I am working on a web project where in the UI part is developed using
GWT 2.3 version, but recently we were asked to check for 508 compliance and
we are not sure which version of GWT is 508 complaint.
Could any one help out on this.
--
You received this message because you are
)
CSVWriter writer = new CSVWriter(new FileWriter(export.csv),
'\t'); // Using OpenCSV library to write a new file export.csv
writer.writeNext(bookArray);
// Writes the string array line by line into
export.csv
Hi Thad, thanks for responding. Currently, I tried using OpenCSV, and had
successfully create a CSV file when I click the export button, but the CSV
file was empty. Do you meant that I could use Apache Commons instead of
OpenCSV?
On Thursday, August 8, 2013 10:27:06 PM UTC+8, Thad Humphries
Hi Ido,
could you explain how your successfully generate the search results in step
3. I had done step 1 and 2 . In step 3, I managed to generate a csv file,
but it is empty. Thank you.
WinnieT
--
You received this message because you are subscribed to the Google Groups
Google Web Toolkit
Moved to https://gwt-review.googlesource.com/3831
http://gwt-code-reviews.appspot.com/1594804/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
---
You received this message because you are subscribed to the Google Groups GWT Contributors group.
To unsubscribe from this group
Moved to https://gwt-review.googlesource.com/3832
http://gwt-code-reviews.appspot.com/1430801/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
---
You received this message because you are subscribed to the Google Groups GWT Contributors group.
To unsubscribe from this group
Moved to Gerrit: https://gwt-review.googlesource.com/3834
http://gwt-code-reviews.appspot.com/1822803/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
---
You received this message because you are subscribed to the Google Groups GWT Contributors group.
To unsubscribe from
Patch moved to Gerrit: https://gwt-review.googlesource.com/3650
http://gwt-code-reviews.appspot.com/1839803/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
---
You received this message because you are subscribed to the Google Groups GWT Contributors group.
To unsubscribe
If you still care about it, could you please move the patch over to
Gerrit?
http://www.gwtproject.org/makinggwtbetter.html
http://gwt-code-reviews.appspot.com/1893803/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
---
You received this message because you are subscribed to
Can someone move that patch over to Gerrit? Would you prefer that I do
it?
(BTW, any reason this hadn't been reviewed yet?)
http://gwt-code-reviews.appspot.com/1839803/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
---
You received this message because you are subscribed
On 2013/05/28 08:52:04, larsaaslin wrote:
Can I run only MostToLeastDerivedPlaceTypeComparatorTest with an ant
command?
Yes ant test
-Dgwt.junit.testcase.includes=**/MostToLeastDerivedPlaceTypeComparatorTest.class
or cd user ant test.nongwt
Sounds good!
http://gwt-code-reviews.appspot.com/1899803/diff/5005/user/test/com/google/web/bindery/requestfactory/gwt/client/RequestFactoryTest.java
File
user/test/com/google/web/bindery/requestfactory/gwt/client/RequestFactoryTest.java
(right):
We just need someone with commit rights. Calling in Matthew (at random),
who will delegate if needed.
http://gwt-code-reviews.appspot.com/1895803/
--
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
---
You received this message because you are subscribed to the Google Groups
Marius: I agree with your comments, but this change has actually been
merged already (despite not being closed here).
Please file an issue and possibly provide a patch (preferably to Gerrit
rather than Rietveld)
As for the reasons of the method overrides, it probably has to do with
this:
LGTM
http://gwt-code-reviews.appspot.com/1895803/
--
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
---
You received this message because you are subscribed to the Google Groups Google Web Toolkit Contributors group.
To unsubscribe from this group and stop receiving emails
http://gwt-code-reviews.appspot.com/1893803/
--
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
---
You received this message because you are subscribed to the Google Groups Google Web Toolkit Contributors group.
To unsubscribe from this group and stop receiving emails from
According to
http://code.google.com/p/google-web-toolkit/issues/detail?id=7919 it
worked the way you describe in GWT 2.4 so let's go ahead and fix this.
Apparently the regression was introduced in
https://code.google.com/p/google-web-toolkit/source/detail?r=11174
Daniel: how's 2.5.1 going? is it
On 2013/01/28 21:06:02, mdempsky wrote:
On 2013/01/12 15:35:19, tbroyer wrote:
We're in dev-time code so we can use Guava (the re-packaged one) and
it's
Objects.equal(a, b) which takes care of nulls.
I don't think that's true. This code still needs to be translatable
to
JavaScript to
Can you please make your patch from trunk/ rather than
trunk/user/src/com/google/gwt/user/datepicker/client/ ?
That said, there are other implications than just selecting from the
calendar view:
- what if I setValue() with a disabled date? (I mean, what should be
the expected behavior)
- what
LGTM. Nothing controversial I believe.
This is a pretty big change though, so maybe an additional pair of eyes
would be better (but maybe you also have an internal review ongoing?)
http://gwt-code-reviews.appspot.com/1827804/diff/1/user/src/com/google/gwt/core/client/impl/Impl.java
File
On 2013/01/22 15:46:27, arobison wrote:
Removed the new cell table builder and reworked to make the change in
AbstractCellTable instead. Removed the isKeyboardSelectable stuff.
Can you update the description to match the actual changes? (I must say
I'm a bit lost).
If it indeed is a fix for
http://gwt-code-reviews.appspot.com/1879804/diff/1/user/src/com/google/gwt/xhr/client/XMLHttpRequest.java
File user/src/com/google/gwt/xhr/client/XMLHttpRequest.java (right):
http://gwt-code-reviews.appspot.com/1879804/diff/1/user/src/com/google/gwt/xhr/client/XMLHttpRequest.java#newcode377
(adding jtamplin as a reviewer as it's in I18N)
http://gwt-code-reviews.appspot.com/1882803/diff/1/user/src/com/google/gwt/i18n/client/TimeZoneInfo.java
File user/src/com/google/gwt/i18n/client/TimeZoneInfo.java (right):
FYI, an issue related to this change:
https://code.google.com/p/google-web-toolkit/issues/detail?id=7677
http://gwt-code-reviews.appspot.com/1879804/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
Superseded by http://gwt-code-reviews.appspot.com/1883803
miichal, you can close this review.
http://gwt-code-reviews.appspot.com/1882803/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
http://gwt-code-reviews.appspot.com/1587803/diff/5002/user/src/com/google/gwt/editor/client/adapters/EditorSource.java
File user/src/com/google/gwt/editor/client/adapters/EditorSource.java
(right):
http://gwt-code-reviews.appspot.com/1587803/diff/1/user/src/com/google/gwt/editor/client/adapters/EditorSource.java
File user/src/com/google/gwt/editor/client/adapters/EditorSource.java
(right):
http://gwt-code-reviews.appspot.com/1587803/diff/1/user/src/com/google/gwt/editor/client/adapters/EditorSource.java
File user/src/com/google/gwt/editor/client/adapters/EditorSource.java
(right):
Ping; would be great to have it in 2.5.1! (IIRC, we initially talked
about including it in 2.5.0 and finally decided to postpone it to the
next release)
http://gwt-code-reviews.appspot.com/1587803/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
http://gwt-code-reviews.appspot.com/1649803/diff/1/src/test/java/com/google/gwt/junit/client/impl/JUnitHost.java
File src/test/java/com/google/gwt/junit/client/impl/JUnitHost.java
(right):
http://gwt-code-reviews.appspot.com/1880804/diff/1/user/src/com/google/gwt/user/client/ui/SplitLayoutPanel.java
File user/src/com/google/gwt/user/client/ui/SplitLayoutPanel.java
(right):
LGTM
http://gwt-code-reviews.appspot.com/1880804/diff/4001/user/src/com/google/gwt/user/client/ui/SplitLayoutPanel.java
File user/src/com/google/gwt/user/client/ui/SplitLayoutPanel.java
(right):
http://gwt-code-reviews.appspot.com/1880803/diff/7001/dev/core/src/com/google/gwt/util/tools/Utility.java
File dev/core/src/com/google/gwt/util/tools/Utility.java (right):
http://gwt-code-reviews.appspot.com/1880803/diff/7001/dev/core/src/com/google/gwt/util/tools/Utility.java#newcode54
LGTM
(BTW why was the GWT-Contrib group removed from CC?)
http://gwt-code-reviews.appspot.com/1880803/diff/1/user/src/com/google/gwt/resources/rg/ImageBundleBuilder.java
File user/src/com/google/gwt/resources/rg/ImageBundleBuilder.java
(right):
Only suggestions, we probably need microbenchmarks to choose between the
alternatives, as AbstractMap is used a lot within applications (at least
one per Widget, then an additional one per event type with a registered
handler).
And +1 to everything Matthew already said.
http://gwt-code-reviews.appspot.com/1875803/diff/1/user/src/com/google/gwt/http/client/RequestImpl.java
File user/src/com/google/gwt/http/client/RequestImpl.java (right):
was wondering if anyone else is seeing this behavior before I proceed with
trying to debug the code and reproduce a test case. We're running 2.5.0.
Thank you
Alexander T
--
You received this message because you are subscribed to the Google Groups
Google Web Toolkit group.
To view
http://gwt-code-reviews.appspot.com/1873803/diff/1/user/src/com/google/gwt/dom/client/Element.java
File user/src/com/google/gwt/dom/client/Element.java (right):
http://gwt-code-reviews.appspot.com/1873803/diff/1/user/src/com/google/gwt/dom/client/Element.java#newcode104
On 2012/11/02 16:37:08, rkj wrote:
On 2012/11/02 02:14:47, tbroyer wrote:
FYI, there's an issue for that:
http://code.google.com/p/google-web-toolkit/issues/detail?id=6865
(understand: whose status should be updated)
Thanks, that's exactly this issue. I have a UI problem though - I do
not
FYI, there's an issue for that:
http://code.google.com/p/google-web-toolkit/issues/detail?id=6865
(understand: whose status should be updated)
http://gwt-code-reviews.appspot.com/1864803/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
You're basically trading a compile-time error for a runtime error. I
must say I don't quite like it.
IIRC, the Editor framework mandates that the editor type (from the
EditorT parameterization) and the edited type (from the property or
field, or the EditorDriver parameterization) must be the
http://gwt-code-reviews.appspot.com/1859803/diff/2001/user/src/com/google/web/bindery/requestfactory/apt/ValidationTool.java
File
user/src/com/google/web/bindery/requestfactory/apt/ValidationTool.java
(right):
D'oh, where was I when Element#setInnerSafeHtml was added? Didn't know
it even existed!
LGTM++
Except you missed one that spans 2 lines in ImageLoadingCell.
(caught using grep -R -A 1 'setInnerHTML' user/src/com/google/gwt/
|grep asString |grep -v setInnerHTML ;-) )
On 2012/10/19 12:50:40, tbroyer wrote:
Except you missed one that spans 2 lines in ImageLoadingCell.
(caught using grep -R -A 1 'setInnerHTML' user/src/com/google/gwt/
|grep
asString |grep -v setInnerHTML ;-) )
Ah, missed it in the patch; you already caught it, great!
On 2012/10/13 03:42:46, cromwellian wrote:
One option is to have an extra method like encodeForHtmlContext()
where you only
pay the tax in that circumstance.
Would it cost really that much to post-process the output from AutoBeans
(possibly depending on context)?
For instance, using OWASP
Should HTML-specific escaping be done on top of the value produced by
AutoBeans? Do we really want to pay the tax (each one of =' is
replaced by 6 chars!) for each AutoBean-based exchanged over the wire
(for example, RequestFactory uses Base64-encoding for its stableId and
version, which makes
On 2012/10/07 02:00:09, rajneeshg wrote:
On 2012/05/21 20:03:28, stephenh wrote:
I am on 2.5.0-rc1. I am still having this issue. Anybody knows when is
this fix
scheduled for.
2.5.1 or 2.6: the version after 2.5.0.
http://gwt-code-reviews.appspot.com/1601804/
--
LGTM
http://gwt-code-reviews.appspot.com/1836803/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
http://gwt-code-reviews.appspot.com/1828803/diff/9001/samples/dynatablerf/pom.xml
File samples/dynatablerf/pom.xml (right):
http://gwt-code-reviews.appspot.com/1828803/diff/9001/samples/dynatablerf/pom.xml#newcode120
samples/dynatablerf/pom.xml:120: version2.3.0-1/version
Oh, I'm really sorry I
Do you think it would be possible to share the BUILD file(s), or a
stripped-down version of it, even privately?
I'd love to see how it compares to Maven and other build systems.
Now back to the CL: given the move to Git soon, which will require some
changes on your side, is it wise to make such
I'm sorry I've only started to review the files (over the last few days)
but I have a first question/comment about where this is going:
There are many things that are not needed in the case of MobileWebApp as
the host page is protected behind authentication. Because the user won't
ever see this
My 2 c€nts
http://gwt-code-reviews.appspot.com/1831803/diff/1/user/src/com/google/gwt/aria/client/Roles.java
File user/src/com/google/gwt/aria/client/Roles.java (right):
http://gwt-code-reviews.appspot.com/1831803/diff/1/user/src/com/google/gwt/aria/client/Roles.java#newcode137
On 2012/09/13 17:16:52, skybrian wrote:
Since this API is essentially just a wrapper around the JavaScript
object and
there's no place to stash the desired response type, adding the setter
and
deprecating create() with a response type seems like the way to go.
Done.
I wonder if we
https://gwt-code-reviews.appspot.com/1830803/diff/2003/user/src/com/google/gwt/xhr/client/XMLHttpRequest.java
File user/src/com/google/gwt/xhr/client/XMLHttpRequest.java (right):
;
}
-return toReturn;
}
public static T AutoBeanT doDecode(EncodeState state, ClassT
clazz, Splittable data) {
@@ -562,12 +564,14 @@ public class AutoBeanCodexImpl {
}
public static E extends Enum? Coder enumCoder(ClassE type) {
-Coder toReturn = coders.get(type
On 2012/09/13 18:35:35, manolo.carrasco wrote:
We are using domain interfaces in a couple of projects and finally we
decided to
patch this file to avoid such amount of extra code.
I thought this was a very small piece of code which IMHO fixes an
important
issue, and could be related with
On 2012/09/13 18:23:04, skybrian wrote:
I don't see any usages of XmlHttpRequest.ResponseType in Google code.
OK, so I'll remove the create(ResponseType).
We could have constants instead of the enum.
I kind of like the enums as in com.google.gwt.dom.client.Style (also
used in SafeStyles),
https://gwt-code-reviews.appspot.com/1829804/diff/1/user/src/com/google/web/bindery/autobean/shared/impl/AutoBeanCodexImpl.java
File
user/src/com/google/web/bindery/autobean/shared/impl/AutoBeanCodexImpl.java
(right):
Reviewers: jtamplin, cromwellian,
Message:
Follow-up to https://gwt-code-reviews.appspot.com/1820806
See http://code.google.com/p/google-web-toolkit/issues/detail?id=7386#c8
Description:
Add a getter for XHR's responseType.
Issue 7386
Please review this at
http://gwt-code-reviews.appspot.com/1828803/diff/1/samples/dynatablerf/pom.xml
File samples/dynatablerf/pom.xml (right):
http://gwt-code-reviews.appspot.com/1828803/diff/1/samples/dynatablerf/pom.xml#newcode55
samples/dynatablerf/pom.xml:55: !-- Doesn't yet work in eclipse. See
On 2012/09/12 14:32:37, jtamplin wrote:
Did you test it? It probably needs testing on IE6 if GWT still
officially
supports it, since I recall some weirdness about getting exceptions if
you
referenced properties that didn't exist on native objects that were
exposed to
JS.
Just ran the
On 2012/09/12 16:03:13, tbroyer wrote:
I'll try on IE7, IE8 and IE9 next (currently downloading the VMs from
the MSDN).
Results from https://browserlab.adobe.com
IE7, IE8 and IE9: works as an expando (no error; prints undefined,
arraybuffer, dummy)
Chrome 18 / Windows: throws on setting to
I haven't looked at the changes (from the description, they should be
simple and straightforward) but what bothers me is that it introduces a
cyclic dependency: c.g.g.user depends on c.g.g.aria which now also
depends on c.g.g.user.
Also, that dependency isn't reflected in the .gwt.xml (so one
On 2012/09/12 19:18:12, skybrian wrote:
We've decided against this change for now (increases the size of the
API too
much). A HasElement interface sounds somewhat reasonable but we can
add that
later if needed.
I suppose if Element implemented HasElement, returning itself, we
could avoid
On 2012/09/10 22:57:14, skybrian wrote:
I don't know this code, but since pushViolations() and
iterableFromConstraintViolations() are both public methods, under the
principle
of being conservative about what you send and liberal about what you
receive, it
seems like
I think I prefer the other option about changing SuggestBox.
https://codereview.appspot.com/6494102/diff/1/user/src/com/google/gwt/user/client/ui/SuggestBox.java
File user/src/com/google/gwt/user/client/ui/SuggestBox.java (right):
My preferred option, with one small adjustment to make it a non-breaking
change.
https://codereview.appspot.com/6492092/diff/1/user/src/com/google/gwt/user/client/ui/SuggestBox.java
File user/src/com/google/gwt/user/client/ui/SuggestBox.java (right):
On 2012/09/10 12:03:20, jtamplin wrote:
BTW: you should create code reviews at
http://gwt-code-reviews.appspot.com/
instead.
There is a also a TODO to support SafeHtml here. I'm not entirely
sure what
that means in this case (Doesn't TextBox already ensure the string is
uninterpreted?
LGTM
https://codereview.appspot.com/6492092/diff/5001/user/src/com/google/gwt/user/client/ui/SuggestBox.java
File user/src/com/google/gwt/user/client/ui/SuggestBox.java (right):
https://codereview.appspot.com/6492092/diff/5001/user/src/com/google/gwt/user/client/ui/SuggestBox.java#newcode878
Reviewers: unnurg,
Description:
CellTree disappeared when clicking in the widget but outside tree nodes.
Issue 6359
Please review this at https://gwt-code-reviews.appspot.com/1827803/
Affected files:
M user/src/com/google/gwt/user/cellview/client/CellTree.java
Index:
Reviewers: atincheva,
Message:
Note: I only applied the suggested fix (after checking the WAI-ARIA
spec), I haven't tried it with a screenreader (actually, it's so
straightforward I haven't tried it at all).
Description:
Add aria-hidden state to layout panel rulers.
Issue 7646
Please review
Adding Brian as reviewer, as discussed with Unnur.
https://gwt-code-reviews.appspot.com/1820807/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
Brian: Unnur thinks you'd review that patch faster than her ;-)
https://gwt-code-reviews.appspot.com/1827803/diff/1/user/src/com/google/gwt/user/cellview/client/CellTree.java
File user/src/com/google/gwt/user/cellview/client/CellTree.java (right):
Reviewers: jtamplin, unnurg,
Description:
xhr.responseType was misspelled as responsetype (lowercase t)
Issue 7386
Please review this at https://gwt-code-reviews.appspot.com/1820806/
Affected files:
M user/src/com/google/gwt/xhr/client/XMLHttpRequest.java
Index: user/src/com/google/gwt
On 2012/09/09 23:24:59, jtamplin wrote:
We really need to get an updated HtmlUnit so we can have automated
tests for
some of these things.
I don't know the status of HtmlUnit, but trying new versions should be
made much easier with the move Maven. That being said, I skimmed
HtmlUnit's Web
Reviewers: skybrian,
Description:
EditorDriver#setConstraintViolations used to thrown NPE if arg was null.
Issue 6578
Please review this at https://gwt-code-reviews.appspot.com/1826803/
Affected files:
M user/src/com/google/gwt/editor/client/impl/SimpleViolation.java
M
https://gwt-code-reviews.appspot.com/1826803/diff/2001/user/src/com/google/web/bindery/requestfactory/gwt/client/impl/AbstractRequestFactoryEditorDriver.java
File
user/src/com/google/web/bindery/requestfactory/gwt/client/impl/AbstractRequestFactoryEditorDriver.java
(right):
ping!
(would be great to have it in 2.5)
https://gwt-code-reviews.appspot.com/1772803/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
Note this fixes
http://code.google.com/p/google-web-toolkit/issues/detail?id=6197 and
there was already a patch pending review, submitted almost one year ago:
http://gwt-code-reviews.appspot.com/1568803/
http://gwt-code-reviews.appspot.com/1823803/
--
Hi!
You should be able to use custom table rendering of a GWT 2.5 cell table to
do what you want in your point (1) (extending AbstractCellTableBuilder and
implement the buildStandardRowImpl method to put the relevant cell type
into the column that alters based upon your condition in your point
LGTM
http://gwt-code-reviews.appspot.com/1820805/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
https://gwt-code-reviews.appspot.com/1712803/diff/2001/user/src/com/google/web/bindery/requestfactory/vm/impl/Deobfuscator.java
File
user/src/com/google/web/bindery/requestfactory/vm/impl/Deobfuscator.java
(right):
Reviewers: cromwellian, rdayal,
Description:
Remove long-deprecated event listeners and EventPreview.
Please review this at https://gwt-code-reviews.appspot.com/1822803/
Affected files:
M
reference/code-museum/src/com/google/gwt/museum/client/common/EventReporter.java
D
Reviewers: cromwellian,
Message:
See
https://groups.google.com/d/topic/google-web-toolkit-contributors/6z9x8S0FYh0/discussion
for the rationale.
Description:
Remove Windows-specific JNI for the check for updates
Remove -Dgwt.devjar in many places, as we no longer have native libs.
Please
Reviewers: cromwellian,
Description:
Remove long-deprecated GWTShell and GWTCompiler tools.
Added a test to make sure JSP works.
Please review this at https://gwt-code-reviews.appspot.com/1821804/
Affected files:
M dev/build.xml
M dev/core/src/com/google/gwt/dev/CompileTaskRunner.java
https://gwt-code-reviews.appspot.com/1821804/diff/1/user/test/com/google/gwt/dev/shell/jsp/client/JspTest.java
File user/test/com/google/gwt/dev/shell/jsp/client/JspTest.java (right):
http://gwt-code-reviews.appspot.com/1819803/diff/1/user/src/com/google/gwt/user/client/ui/Tree.java
File user/src/com/google/gwt/user/client/ui/Tree.java (right):
http://gwt-code-reviews.appspot.com/1819803/diff/1/user/src/com/google/gwt/user/client/ui/Tree.java#newcode641
Reviewers: skybrian,
Message:
For inclusion in GWT 2.5
Description:
Use c.g.g.core.shared.GWT in shared code.
Issue 7527
Please review this at https://gwt-code-reviews.appspot.com/1818803/
Affected files:
M user/src/com/google/gwt/dom/builder/shared/ElementBuilderFactory.java
M
On 2012/08/28 13:17:58, jtamplin wrote:
LGTM, as far as it goes
However, when you use GWT.create in non-client code, there is some
setup you
need to do - you have to create the ServerGwtBridge, set the deferred
binding
properties (either globally or per-thread), and you need to register
Are there tests that could be added / updated?
http://gwt-code-reviews.appspot.com/1815803/diff/1/src/com/google/gwt/dom/client/DOMImpl.java
File src/com/google/gwt/dom/client/DOMImpl.java (right):
absoluteTop/Left are not enough:
http://code.google.com/p/google-web-toolkit/issues/detail?id=7575
And WebKit is probably not enough too (subpixel rendering is becoming
standard, and will soon be mainstream).
http://gwt-code-reviews.appspot.com/1797805/
--
FYI, my take on authentication and RequestFactory can be found at
https://github.com/tbroyer/gwt-maven-archetypes/tree/master/guice-rf-activities
(also uses Guice, GIN and Guava).
I'm in the process of trading the userName and isAdmin JS variables for
an object (managing ser/deserialization with
I'm not against this kind of change but IMO any deviation from the JSON
and/or ECMAScript specs should be documented, or we risk removing them
at a later time and break things again (that being said, I don't
understand what's broken here and how this CL fixes it wrt JDK 7, as
all I see is
On 2012/07/25 10:01:17, manolo.carrasco wrote:
A bit of discipline maybe, but no pain at all (not for me at
least).
You are right s/pain/discipline/.
When you use gin for places and activities you have to be aware of
this
circumstance to avoid multiple instances of the event bus, so
1 - 100 of 719 matches
Mail list logo
