Please go ahead and move it, but with a couple of team members away we
probably won't get to it this week.
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 Group
On 2013/06/04 09:01:51, tbroyer wrote:
Can someone move that patch over to Gerrit? Would you prefer that I do
it?
(BTW, any reason this hadn't been reviewed yet?)
No particular reason other than it dropping off the radar with other
stuff going on. Of course we will want unit tests for this.
On 2013/05/06 08:05:00, John A. wrote:
On 2013/04/30 23:05:08, skybrian wrote:
> Hello, could you avoid reformatting code that you're not working on?
Or if you
> want to just fix the javadoc, do that in a separate CL.
Appears the eclipse auto formatting had changed Element JavaDo
Hello, could you avoid reformatting code that you're not working on? Or
if you want to just fix the javadoc, do that in a separate CL.
http://gwt-code-reviews.appspot.com/1815803/
--
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
---
You received this message because you ar
I did a Google search for getPermuationId and didn't find any hits other
than our own code.
In theory anything in core/ext/linker is a public linker API, but I
think this is pretty safe. Not many people write their own linkers.
Since it is an API change, we should remember it when writing GWT 2.
Why protected? Also, shouldn't the unit tests be updated?
http://gwt-code-reviews.appspot.com/1891804/
--
--
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.
T
LGTM
http://gwt-code-reviews.appspot.com/1890803/
--
--
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
Seems fine. (I'll take care of these nits.)
http://gwt-code-reviews.appspot.com/1587803/diff/10006/user/src/com/google/gwt/editor/client/adapters/HasDataEditor.java
File user/src/com/google/gwt/editor/client/adapters/HasDataEditor.java
(right):
http://gwt-code-reviews.appspot.com/1587803/diff/
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/5002/user/src/com/google/gwt/editor/client/adapte
I'm not all that familiar with the editor framework but perhaps there's
a cleaner way to do this? (See comments.)
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.j
LGTM. Thanks!
http://gwt-code-reviews.appspot.com/1881803/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
http://gwt-code-reviews.appspot.com/1859804/diff/12001/user/src/com/google/gwt/user/client/Window.java
File user/src/com/google/gwt/user/client/Window.java (right):
http://gwt-code-reviews.appspot.com/1859804/diff/12001/user/src/com/google/gwt/user/client/Window.java#newcode223
user/src/com/goog
LGTM
http://gwt-code-reviews.appspot.com/1859804/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
On 2012/12/05 23:41:03, cromwellian wrote:
LGTM. General potentially unrelated question, what about the case
static class Covariant extends Item
Is this handled properly?
It looks like it. In that case, isClassOrInterface() will return null,
so it won't look at covariant types, just gener
Reviewers: cromwellian,
Description:
Fix a GWT-RPC issue where covariant array types for lists of raw types
were
being generated inconsistently, causing serialization policy generation
to
be nondeterministic. Includes tests.
Fixes issue 7791.
Please review this at http://gwt-code-reviews.appsp
On 2012/11/29 05:45:46, cromwellian wrote:
On 2012/11/29 03:35:49, skybrian wrote:
LGTM. No API/Source breakages this time around?
There were but I already fixed them when I did Linux. For the Mac it was
upgrading XCode and figuring out how much I need to mirror Firefox's
compiler op
Reviewers: mdempsky,
Description:
Update devmode Firefox plugin to support Firefox 17 on the Mac:
- started compiling with C++11, using clang (from XCode 4.5.2)
- dropped PPC from build
Includes release candidate.
Review by: mdemp...@google.com
Please review this at http://gwt-code-reviews.app
Reviewers: mdempsky,
Description:
Add Windows DLL for Firefox 17 to dev mode plugin.
Please review this at http://gwt-code-reviews.appspot.com/1871803/
Affected files:
A plugins/xpcom/VisualStudio/ff170-xpcom.vcxproj
M plugins/xpcom/prebuilt/extension/chrome.manifest
--
http://groups.goo
On 2012/11/23 08:51:26, nfm wrote:
ping?
Sorry about that. It's hard to say which is faster without testing, but
I think having one cache is simpler than having two. So let's go ahead
with merging these variables.
http://gwt-code-reviews.appspot.com/1859804/
--
http://groups.google.com/group
http://gwt-code-reviews.appspot.com/1869804/diff/1/user/test/com/google/gwt/user/rebind/rpc/SerializableTypeOracleBuilderTest.java
File
user/test/com/google/gwt/user/rebind/rpc/SerializableTypeOracleBuilderTest.java
(right):
http://gwt-code-reviews.appspot.com/1869804/diff/1/user/test/com/google
Reviewers: mdempsky,
Description:
Update devmode Firefox plugin to support Firefox 17 on 64-bit Linux.
(Other platforms coming soon.)
Please review this at http://gwt-code-reviews.appspot.com/1870803/
Affected files:
M plugins/xpcom/ExternalWrapper.cpp
M plugins/xpcom/FFSessionHandler.cpp
Reviewers: cromwellian,
Description:
Fix a bug where GWT-RPC would not generate some covariant array types,
possibly causing inconsistent serialization policies. Includes a test.
Fixes issue 7791
Please review this at http://gwt-code-reviews.appspot.com/1869804/
Affected files:
M
user/src
http://gwt-code-reviews.appspot.com/1869803/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
Reviewers: mdempsky,
Description:
Minor cleanup in SerializableTypeOracleBuilder:
- remove unused fields
- TypeInfoComputed can be static
- tighten visibility
- remove some indirection
- fix some warnings
Review by: mdemp...@google.com
Please review this at http://gwt-code-reviews.appspot.com/1
Reviewers: mdempsky,
Description:
Move non-GWTTestCase tests in RpcSuite to RpcSuiteNoBrowser,
so they don't unnecessarily get run multiple times in different browser
environments.
Also fix the build to handle running *SuiteNoBrowser suites.
Review by: mdemp...@google.com
Please review this at
http://gwt-code-reviews.appspot.com/1866803/diff/1/user/src/com/google/gwt/logging/server/StackTraceDeobfuscator.java
File user/src/com/google/gwt/logging/server/StackTraceDeobfuscator.java
(right):
http://gwt-code-reviews.appspot.com/1866803/diff/1/user/src/com/google/gwt/logging/server/StackTr
LGTM
http://gwt-code-reviews.appspot.com/1866803/diff/1/user/src/com/google/gwt/logging/server/StackTraceDeobfuscator.java
File user/src/com/google/gwt/logging/server/StackTraceDeobfuscator.java
(right):
http://gwt-code-reviews.appspot.com/1866803/diff/1/user/src/com/google/gwt/logging/server/S
http://gwt-code-reviews.appspot.com/1859804/diff/7001/user/src/com/google/gwt/user/client/Window.java
File user/src/com/google/gwt/user/client/Window.java (right):
http://gwt-code-reviews.appspot.com/1859804/diff/7001/user/src/com/google/gwt/user/client/Window.java#newcode322
user/src/com/google
LGTM. Not really familiar with this code, though.
http://gwt-code-reviews.appspot.com/1867803/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
On 2012/11/05 16:34:46, perneto wrote:
FYI: I added http://code.google.com/p/rietveld/issues/detail?id=402 to
track this change, as requested by Brian.
Oops, that's actually the rietveld issue tracker, not the GWT issue
tracker. The one you want is at:
http://code.google.com/p/google-web-toolki
Context: the goal is to put a footer in the middle section so it will
scroll into view below all the data rows.
http://gwt-code-reviews.appspot.com/1864803/diff/1/user/src/com/google/gwt/user/cellview/client/DataGrid.java
File user/src/com/google/gwt/user/cellview/client/DataGrid.java (right):
http://gwt-code-reviews.appspot.com/1859804/diff/1/user/src/com/google/gwt/user/client/Window.java
File user/src/com/google/gwt/user/client/Window.java (right):
http://gwt-code-reviews.appspot.com/1859804/diff/1/user/src/com/google/gwt/user/client/Window.java#newcode238
user/src/com/google/gwt/u
LGTM
http://gwt-code-reviews.appspot.com/1863803/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
Reviewers: mdempsky,
Description:
Chrome plugin: don't use an iterator after the underlying entry has been
freed.
Rebuilt Windows DLL and created a release.
(This patch might help on Linux and Mac as well, but for safety I'm not
updating them yet.)
Please review this at http://gwt-code-reviews.
Reviewers: mdempsky,
Description:
Chrome devmode plugin: recompile Windows DLL to be statically linked.
Other fixes:
- Check in zip file as uploaded to Chrome web store; remove crx file.
- Makefile "zip" rule didn't include the manifest.
Please review this at http://gwt-code-reviews.appspot.co
On 2012/10/24 21:24:02, mdempsky wrote:
Alternatively, take a look at Patch Set #1, which simply added an
extra
parameter to exec() to pass in a custom JavaCompiler instance. Would
that patch
(updated to make the exec() methods public) be preferable?
Yeah, seems simpler.
http://gwt-code-r
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):
http://gwt-code-reviews.appspot.com/1859803/diff/2001/user/src/com/google/web/bindery/
Reviewers: mdempsky,
Description:
Update the DevMode Chrome plugin's Makefile to build a zip file to
upload to the Chrome web store.
Instead of building a crx file, we build a zip file containing the files
to upload. Also, the Chrome web store doesn't allow the key or update
URL to be set in the
LGTM
http://gwt-code-reviews.appspot.com/1857803/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
We have no plans to support reflection in client-side code. What problem
are you trying to solve?
http://gwt-code-reviews.appspot.com/1855803/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
http://gwt-code-reviews.appspot.com/1851804/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
LGTM
http://gwt-code-reviews.appspot.com/1850804/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
LGTM
I don't quite understand why it's capturing click events when it's 10x10
and it's 20 units offscreen, but setting visibility seems like the right
thing to do anyhow.
I'd guess that having it explicitly offscreen makes it less likely to be
in the way somehow when using developer tools that s
Hi, one-line typo fixes like this aren't worth bothering with,
particularly since the constructor isn't public. If you want to help, I
would suggest either doing a more extensive spell-checking pass, or
figure out how to say something more useful.
In this case, it looks like all CellTreeNodeViews
http://gwt-code-reviews.appspot.com/1844803/diff/1/plugins/npapi/Makefile
File plugins/npapi/Makefile (right):
http://gwt-code-reviews.appspot.com/1844803/diff/1/plugins/npapi/Makefile#newcode17
plugins/npapi/Makefile:17: ARCH=x86
On 2012/09/29 02:20:02, jtamplin wrote:
I assume this means Chro
===
--- plugins/npapi/npapi/npapi.h (revision 11305)
+++ plugins/npapi/npapi/npapi.h (working copy)
@@ -258,6 +258,16 @@
} NPDrawingModel;
#endif
+// copied from newer npapi.h - skybrian
+#ifdef XP_MACOSX
+typedef enum {
+#ifndef NP_NO_CARBON
+ NPEventModelCarbon
On 2012/09/26 21:53:51, cromwellian wrote:
LGTM. Does this include rebuilt versions of the plugins? Wasn't there
one that
had debugging spew turned on that we needed to disable?
I only rebuilt the Linux binary so far; Mac and Windows will be under a
separate CL. There was a report of debuggin
For comparison, the Firefox extension is now a 7M download, due to
having a separate binary for each version of Firefox * three platforms.
http://gwt-code-reviews.appspot.com/1840803/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
On 2012/09/26 18:51:33, jtamplin wrote:
LGTM, though I still wonder if you want -style PRETTY to be the
default.
It only increases the size of the zipped extension by about 5% (from
349k to 365k). I think I'd rather leave it on since it makes debugging
easier.
http://gwt-code-reviews.appspot.
http://gwt-code-reviews.appspot.com/1840803/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
Reviewers: cromwellian,
Description:
Upgraded the devmode Chrome extension to manifest version 2.
To do this, we need to replace all inline javascript with code in a
separate js file. Also, changed the options page to use the
DirectInstall linker so that GWT bootstrap doesn't use eval() or inlin
LGTM. Thanks!
http://gwt-code-reviews.appspot.com/1836803/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
Could you write a test for this change?
http://gwt-code-reviews.appspot.com/1836803/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
Reviewers: acleung,
Description:
Fix Chrome plugin compile errors by replacing "Value" with "gwt::Value".
Please review this at http://gwt-code-reviews.appspot.com/1835803/
Affected files:
M plugins/npapi/NPVariantWrapper.h
M plugins/npapi/ScriptableInstance.cpp
M plugins/npapi/Scriptabl
http://gwt-code-reviews.appspot.com/1834803/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
On 2012/09/17 20:43:56, tbroyer wrote:
Do you think it would be possible to share the BUILD file(s), or a
stripped-down
version of it, even privately?
There's a high-level overview here:
http://google-engtools.blogspot.com/2011/08/build-in-cloud-how-build-system-works.html
It doesn't explain
Reviewers: cromwellian,
Description:
In the Chrome plugin, rename "src" to "java" for compatibility with
Google's internal build system.
Please review this at http://gwt-code-reviews.appspot.com/1834803/
Affected files:
M plugins/npapi/DevModeOptions/build.xml
A
plugins/npapi/DevModeOpti
LGTM. Thanks!
http://gwt-code-reviews.appspot.com/1831803/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
LGTM
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):
https://gwt-code-reviews.appspot.com/1829804/diff/1/user/src/com/google
LGTM
https://gwt-code-reviews.appspot.com/1830803/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
On 2012/09/13 20:22:43, tbroyer wrote:
I kind of like the enums as in com.google.gwt.dom.client.Style (also
used in
SafeStyles), but I'll make the setResponseType(String) public, and add
a
getResponseTypeString() to the enum.
Oops, I guess I should have responded. I'm fine with it either w
I don't see any usages of XmlHttpRequest.ResponseType in Google code. We
could have constants instead of the enum.
https://gwt-code-reviews.appspot.com/1830803/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
I'm not all that familiar with XMLHttpRequest, but every JavaScript
example I can find sets the response type between open() and send(). I
don't understand John's concern about a race condition since it seems
like the request won't be sent until send() is called? I think we should
follow common pr
On 2012/09/13 09:57:02, manolo.carrasco wrote:
RF fails in the case of @ProxyFor pointing to a domain-interface
instead of a
domain-implementation, could you consider to include the patch
https://gwt-code-reviews.appspot.com/1764804/ as well in 2.5?
I think that counts as a new feature instea
Hmm. If it were named "HasAriaElement", then it would make sense that
not all widgets implement it. Only those that need to be customized for
proper Aria support would have it.
http://gwt-code-reviews.appspot.com/1828804/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
Hmm, I'm inclined to skip this patch for GWT 2.5. The typo fix at least
makes it work on some browsers...
https://gwt-code-reviews.appspot.com/1830803/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
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 doubling the number of methods, but this seems o
There's a usage in Google code as well, but it looks like it can't pass
in a null.
I'd rather fix iterableFromConstrantViolations [sic] but I don't care
much, so submitting.
https://gwt-code-reviews.appspot.com/1826803/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
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 iterableFromConstraintViolations()
should return an empty iterable when
LGTM. I'll commit this soon.
https://codereview.appspot.com/6492092/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
Never mind, looks like a hard-coded list in our build file.
https://gwt-code-reviews.appspot.com/1712803/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
On 2012/09/07 19:57:58, skybrian wrote:
LGTM. Committing...
I'm getting 500 errors in MultipleFactoriesTest.
java.lang.RuntimeException: The RequestFactory ValidationTool must be
run for the
com.google.web.bindery.requestfactory.shared.MultipleFactoriesTest.Factory1
RequestFactory
LGTM. Committing...
https://gwt-code-reviews.appspot.com/1712803/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
/bindery/requestfactory/vm/impl/Deobfuscator.java#newcode122
user/src/com/google/web/bindery/requestfactory/vm/impl/Deobfuscator.java:122:
private Map> merge(Map>
domainToClientType1,
On 2012/09/06 13:37:48, alexandre.ardhuin.dw wrote:
On 2012/09/05 21:21:15, skybrian wrote:
> This method
Oops, I had some draft comments on this CL but never sent them. (I
assume this fix is still wanted...)
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/Deob
LGTM, but I'm not going to be able to submit this right away. (Working
on Firefox plugin.)
https://gwt-code-reviews.appspot.com/1818803/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
http://gwt-code-reviews.appspot.com/1817803/diff/1/plugins/xpcom/JavaObject.cpp
File plugins/xpcom/JavaObject.cpp (right):
http://gwt-code-reviews.appspot.com/1817803/diff/1/plugins/xpcom/JavaObject.cpp#newcode385
plugins/xpcom/JavaObject.cpp:385: (JSVAL_IS_PRIMITIVE(argv[1]) &&
!JSVAL_IS_NULL(a
Reviewers: acleung,
Description:
Fix development mode Firefox plugin to work on with Firefox 15 on Linux.
Tested with 64-bit linux. Now builds 32-bit Linux for 15, but that's not
tested yet.
Please review this at http://gwt-code-reviews.appspot.com/1817803/
Affected files:
M plugins/xpcom/Ja
Turns out I needed to update chrome.manifest. After fixing this, I'm
getting an error on reloading the page. I'm surprised it was working for
jat, unless that was with Firefox 14.
Connect(url=http://localhost:8080/Showcase.html?gwt.codesvr=xx.xx.xx.xx:9997#!CwCheckBox,
sessionKey=XXX, address=xx.
Strangely, this isn't working for me. The plugin seems to install and I
can bring up preferences, but the __gwt_HostedModePlugin javascript
variable doesn't show up, so it goes to the missing plugin page. Any
ideas?
Firefox version:
Mozilla/5.0 (X11; Linux i686 on x86_64; rv:15.0) Gecko/20100101
On 2012/08/24 20:41:13, jtamplin wrote:
#ifndef JSVAL_IS_OBJECT
#define JSVAL_IS_OBJECT(x) (!JSVAL_IS_PRIMITIVE(x))
#endif
They're actually subtly different because the removed JSVAL_IS_OBJECT
would return true for a null, and so does JSVAL_IS_PRIMITIVE. However, I
didn't see any code that dep
Thanks! In that case, I'll commit this and follow up in another CL if
needed.
http://gwt-code-reviews.appspot.com/1816803/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
Reviewers: acleung,
Description:
Got the development mode plugin to compile for Firefox 15 using
xulrunner-15.0b6 on 64-bit Linux.
(This is not ready to commit yet, since it's not tested, but here's a
preview.)
Please review this at http://gwt-code-reviews.appspot.com/1816803/
Affected files:
Reviewers: mdempsky,
Description:
Attempt to fix flakiness in testGetClientSize by bumping up the timeout.
(I didn't track down the root cause.)
Please review this at http://gwt-code-reviews.appspot.com/1814803/
Affected files:
M user/test/com/google/gwt/user/client/WindowTest.java
Index:
http://gwt-code-reviews.appspot.com/1813803/diff/1/user/test/com/google/gwt/dev/js/client/CoverageTest.java
File user/test/com/google/gwt/dev/js/client/CoverageTest.java (right):
http://gwt-code-reviews.appspot.com/1813803/diff/1/user/test/com/google/gwt/dev/js/client/CoverageTest.java#newcode28
http://gwt-code-reviews.appspot.com/1813803/diff/1/user/test/com/google/gwt/dev/js/client/CoverageTest.java
File user/test/com/google/gwt/dev/js/client/CoverageTest.java (right):
http://gwt-code-reviews.appspot.com/1813803/diff/1/user/test/com/google/gwt/dev/js/client/CoverageTest.java#newcode28
LGTM
http://gwt-code-reviews.appspot.com/1813803/diff/1/user/test/com/google/gwt/dev/js/client/CoverageTest.java
File user/test/com/google/gwt/dev/js/client/CoverageTest.java (right):
http://gwt-code-reviews.appspot.com/1813803/diff/1/user/test/com/google/gwt/dev/js/client/CoverageTest.java#ne
LGTM
http://gwt-code-reviews.appspot.com/1800805/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
LGTM
http://gwt-code-reviews.appspot.com/1803804/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
LGTM
http://gwt-code-reviews.appspot.com/1810803/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
Looks good. I was wondering whether the sort would be slower, but I
looked at the implementation and it's not using native sort() in any
case.
Let's wait until tomorrow to submit, to give people a chance to respond.
http://gwt-code-reviews.appspot.com/1807805/
--
http://groups.google.com/group
I'm not familiar with FocusPanel, but it looks like it's used by a
variety of subclasses that need to accept keyboard input for various
reasons. So I don't think this is a good idea in all cases. Perhaps
FocusPanel needs a way to set the ARIA role and you should do this only
for your app?
http:/
Okay, you fixed autohide. They also mentioned that DialogBox can't be
dragged. Does this change fix that problem too?
http://gwt-code-reviews.appspot.com/1810803/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
On 2012/08/14 01:01:20, skybrian wrote:
It makes sense, considering that I don't know anything about iPads.
Could you
describe the bug in more detail? Is there an issue for it in the GWT
issue
tracker?
It looks like this issue is related:
http://code.google.com/p/google-web-toolkit/i
It makes sense, considering that I don't know anything about iPads.
Could you describe the bug in more detail? Is there an issue for it in
the GWT issue tracker?
http://gwt-code-reviews.appspot.com/1810803/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
LGTM
http://gwt-code-reviews.appspot.com/1776803/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
The idea is that any SingleSelectionModel can be thought of as a set
that contains either zero or one items. So it seems appropriate that it
should have the new methods - and after all, the whole point is to be
able to treat them uniformly.
SetSelectionModel isn't intended to imply that the Selec
Yes, by editing the description in Critique. (I just did that for you.)
http://gwt-code-reviews.appspot.com/1798806/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
Ping.
http://gwt-code-reviews.appspot.com/1799804/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
The implementation seems fine.
Could you also create a bug in the external GWT issue tracker and add it
to the CL description, so we can track when this gets into a stable
release?
Also, for those coming in late, please add a bit of justification to the
issue. All our existing widgets are gettin
1 - 100 of 275 matches
Mail list logo