[gwt-contrib] Re: Initial add of JSON-RPC support to RequestFactory. (issue1355804)

2011-02-16 Thread t . broyer
I must have missed something, or is it mostly about changing serialization of proxies? (which I didn't find in the patch) Is the goal to enable creating a RequestFactory interface for a JSON-RPC service that's not backed by a SimpleRequestProcessor (or similar)? Given that the JSON-RPC request

Re: RequestFactory: /gwtRequest not found

2011-02-11 Thread $ a r j i t h Pullithodi
On Fri, Feb 11, 2011 at 3:00 PM, Thomas Broyer t.bro...@gmail.com wrote: On Friday, February 11, 2011 7:26:11 AM UTC+1, Sarjith wrote: On Fri, Feb 11, 2011 at 11:41 AM, Y2i yur...@gmail.com wrote: There should be something like this in app's web.xml servlet

Re: RequestFactory: /gwtRequest not found

2011-02-10 Thread $ a r j i t h Pullithodi
Hi, I was trying to follow RequestFactory concept. and I'm still not able to solve this error. (even after adding this servlet mapping). -- Sarjith On Sun, Dec 26, 2010 at 10:03 AM, Matthew Hill matt2...@gmail.com wrote: Fixed by adding a servlet handler for

Re: RequestFactory: /gwtRequest not found

2011-02-10 Thread $ a r j i t h Pullithodi
On Fri, Feb 11, 2011 at 11:41 AM, Y2i yur...@gmail.com wrote: There should be something like this in app's web.xml servlet servlet-namerequestFactory/servlet-name servlet-classcom.google.gwt.requestfactory.server.RequestFactoryServlet/servlet-class /servlet servlet-mapping

Re: RequestFactory: /gwtRequest not found

2011-02-10 Thread $ a r j i t h Pullithodi
we say getPerson() request should call this method in this service class?. please correct me if I'm wrong anywhere. -- Sarjith On Fri, Feb 11, 2011 at 11:56 AM, $ a r j i t h Pullithodi sarji...@gmail.com wrote: On Fri, Feb 11, 2011 at 11:41 AM, Y2i yur...@gmail.com wrote: There should

[gwt-contrib] Re: Introduces SkipInterfaceValidation annotation. (issue1338807)

2011-02-08 Thread t . broyer
http://gwt-code-reviews.appspot.com/1338807/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: Introduces SkipInterfaceValidation annotation. (issue1338807)

2011-02-07 Thread t . broyer
I think I'll revert the changes to the existing tests (keeping only the changes introducing the new ones exercizing SkipInterfaceValidation), and make a second patch about fixing them. http://gwt-code-reviews.appspot.com/1338807/diff/1/4 File

[gwt-contrib] Introduces SkipInterfaceValidation annotation. (issue1338807)

2011-02-05 Thread t . broyer
Reviewers: bobv, rjrjr, Message: I initially thought about skipping validation of the ProxyFor/ProxyForName/Service/ServiceName if the class is annotated (i.e. don't poison the RFIV, or fail in the RequestFactoryGenerator, if the ProxyFor/etc. annotation is absent), and allowing the annotation

Re: Bind form values when sending to server

2011-02-03 Thread $ a r j i t h Pullithodi
Thanks buddy... donno this gonna work in my scenario, but had decided to give it a try. Thanks a lot. please do share if anyone has a better idea or any alternative for this, so that we don't miss anything. Thanks once again...!! -- Sarjith On Thu, Feb 3, 2011 at 2:42 PM, Thomas Broyer

Bind form values when sending to server

2011-02-02 Thread $ a r j i t h Pullithodi
Hi All, I'm starting with GWT module for a functionality say 'addPerson'. so ui will have around 10-15 user inputs (in a single form) like firstname, lastname, address, phone, etc. From a short period of experience in gwt, I am sending these data to server using RPC call like

[gwt-contrib] Re: Overhaul Editor framework traversal logic to use a visitor pattern. (issue1340802)

2011-02-01 Thread t . broyer
http://gwt-code-reviews.appspot.com/1340802/diff/1/4 File user/src/com/google/gwt/editor/client/EditorVisitor.java (right): http://gwt-code-reviews.appspot.com/1340802/diff/1/4#newcode27 user/src/com/google/gwt/editor/client/EditorVisitor.java:27: public T void endVisit(EditorContextT ctx

SimplePager - changing page callback

2011-01-27 Thread Stephan T
I'd like to perform some operations when a user, by clicking left or right arrow in a SimplePager, changes page in a CellTable. Can't find any suitable callback-methods or methods to override on SimplePager or CellTable. Any suggestions? -- You received this message because you are subscribed

GWT SDK 2.1.1 eclipse plugin crashes Helios

2011-01-24 Thread T S
or workarounds? I'm using the helios GWT provided from: http://dl.google.com/eclipse/plugin/3.6 T S -- You received this message because you are subscribed to the Google Groups Google Web Toolkit group. To post to this group, send email to google-web-toolkit@googlegroups.com. To unsubscribe from

[gwt-contrib] Issue 4916: optgroups in select elements cause issue in WebKit-based browsers (issue1313801)

2011-01-21 Thread t . broyer
Reviewers: cramsdale, jgw, Description: http://code.google.com/p/google-web-toolkit/issues/detail?id=4916 Remove old workaround code from DOMImplSafari for something that's been fixed in Safari for a while makes it hard to work with optgroups in select elements. Assigning arbitrarily to Chris

[gwt-contrib] Re: Issue 5563: Make it easier to implement PlaceHistoryMapper directly (issue1293801)

2011-01-21 Thread t . broyer
http://gwt-code-reviews.appspot.com/1293801/diff/1/2 File user/src/com/google/gwt/place/rebind/PlaceHistoryGeneratorContext.java (right): http://gwt-code-reviews.appspot.com/1293801/diff/1/2#newcode41 user/src/com/google/gwt/place/rebind/PlaceHistoryGeneratorContext.java:41: static

[gwt-contrib] AbstractPlaceHistoryMapper doesn't process tokens correctly wrt empty prefixes (issue1316801)

2011-01-21 Thread t . broyer
Reviewers: rjrjr, Description: in r8817, we landed a patch that allows @Prefix(), i.e. a non-prefixed PlaceTokenizer, and produces a separator-less history token as a result. The token parsing however wasn't updated, and such a token would always go the fallback route to the default place,

[gwt-contrib] Issue 5599: HasDataEditor doesn't handle removes from the list (never calls HasData#setRowCount) (issue1297801)

2011-01-17 Thread t . broyer
Reviewers: bobv, rjrjr, jlabanca, Description: http://code.google.com/p/google-web-toolkit/issues/detail?id=5599 Bob suggested that It should check to see if the size of the exported list has changed whenever create() or dispose() is called.

[gwt-contrib] Issue 5872: RequestFactory: onFailure or onViolations on one invocation prevents following Receivers (issue1290801)

2011-01-14 Thread t . broyer
Reviewers: bobv, rjrjr, Description: http://code.google.com/p/google-web-toolkit/issues/detail?id=5872 This is a first step in the right direction, but I suppose you'd want to go farther. Noticeably: - it's using UmbrellaException from c.g.g.event.shared. Maybe it should define its own

[gwt-contrib] Issue 5563: Make it easier to implement PlaceHistoryMapper directly (issue1293801)

2011-01-14 Thread t . broyer
Reviewers: , Description: http://code.google.com/p/google-web-toolkit/issues/detail?id=5563 Following my proposal (copied here from the issue tracker): The fix should IMO be (and I believe it was in my initial patch that introduced the handler/mapper dichotomy) to ignore classes in the

[gwt-contrib] Issue 5752: ListEditor.setValue fails once the underlying AutoBean has been frozen (issue1284801)

2011-01-13 Thread t . broyer
Reviewers: bobv, rjrjr, Description: Copied from the issue tracker: The real issue seems to be in how com.google.gwt.requestfactory.client.impl.RequestFactoryEditorDelegate determines whether it shouldFlush() or not: it only deals with am I in 'read-only' mode or not (if request==null, edited

[gwt-contrib] Issue 5549: fix is/has support for boolean properties (issue1272801)

2011-01-09 Thread t . broyer
Reviewers: bobv, Description: There are 3 issues: - AutoBean server code incorrectly assume the getter prefix is always get (or rather, 3-chars long) - same thing in the RequestFactory generator (RequestFactoryModel) - and in RequestFactory server code (ReflectiveServiceLayer) I've also

[gwt-contrib] Re: Issue 5549: fix is/has support for boolean properties (issue1272801)

2011-01-09 Thread t . broyer
http://gwt-code-reviews.appspot.com/1272801/diff/1/3 File user/src/com/google/gwt/autobean/server/BeanMethod.java (right): http://gwt-code-reviews.appspot.com/1272801/diff/1/3#newcode52 user/src/com/google/gwt/autobean/server/BeanMethod.java:52: String inferName(Method method) { Similar to

[gwt-contrib] Re: Adding support for setting column widths in CellTable, and for allowing CellTable to use fixed t... (issue1263801)

2011-01-06 Thread t . broyer
In a word: I like it (just gave a cursory gance at the CellTable and CellTableBasic.css changes though) http://gwt-code-reviews.appspot.com/1263801/diff/1/12 File user/src/com/google/gwt/user/cellview/client/CellTable.java (right):

[gwt-contrib] Issue 5795: Editor framework does not descend into an OptionalFieldEditor's sub-editor (issue1249801)

2010-12-27 Thread t . broyer
Reviewers: bobv, Description: See the bug report and associated thread in GWT-Contrib for details. http://code.google.com/p/google-web-toolkit/issues/detail?id=5795 I verified that it didn't break any other test in the EditorSuite. Please review this at

[gwt-contrib] Re: Copying the DOMImplIE6 setOpacity implementation into DOMImplIE8 because IE8 does not support th... (issue1211801)

2010-12-13 Thread t . broyer
http://gwt-code-reviews.appspot.com/1211801/diff/3001/4001 File user/src/com/google/gwt/dom/client/DOMImplIE8.java (right): http://gwt-code-reviews.appspot.com/1211801/diff/3001/4001#newcode39 user/src/com/google/gwt/dom/client/DOMImplIE8.java:39: var ua = navigator.userAgent.toLowerCase(); Is

Showing and hiding a ButtonCell in a CellTable

2010-12-02 Thread Stephan T
In a CellTable I'm using a ButtonCell column. The button for each row should show only when a condition is true. How do I achieve that? This is my code now: ColumnPerson, String myColumn = new ColumnPerson, String(new ButtonCell()) { @Override public String getValue(Person

Style a ClickableTextCell

2010-12-01 Thread Stephan T
I'm using the new CellTable from GWT 2.1 and have a ClickableTextCell. I want the clickable text to have the same style as an a-element and the cursor should be a pointer when hovering the clickable text. How do I achieve this? -- You received this message because you are subscribed to the

Showing and hiding a ButtonCell in a CellTable

2010-12-01 Thread Stephan T
I have a CellTable and a ButtonCell column. The button for each row should show only when a condition is true. How do I achieve that? This is my code now: ColumnPerson, String myColumn = new ColumnPerson, String(new ButtonCell()) { @Override public String getValue(Person

[gwt-contrib] Issue 5663: DoubleParser rounds values to integers (issue1170801)

2010-12-01 Thread t . broyer
Reviewers: rjrjr, jat, Description: This is critical for us; could you please include it in 2.1.1? I'm not sure about the test (re. i18n involved through NumberFormat) but I'm sure about the fix! Please review this at http://gwt-code-reviews.appspot.com/1170801/show Affected files:

[gwt-contrib] Re: Ensure RequestFactory is UTF-8 clean. (issue1158801)

2010-11-29 Thread t . broyer
LGTM (note that it also fixes issue 5238, which is a dupe of 5474, AIUI) http://gwt-code-reviews.appspot.com/1158801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Issue 5474: RequestFactory response encoding (issue1154801)

2010-11-27 Thread t . broyer
Reviewers: bobv, Description: Fix for RequestFactoryServlet and unit test. Also fixed testUserInfo in RequestFactoryTest, which was not asynchronous (and therefore, I believe, wasn't actually testing anything). The tests use finishTest and not finishTestAndReset because they don't use

[gwt-contrib] Re: Issues 5479, 5507 and 5571 (and a few other editor-related tweaks) (issue1099801)

2010-11-19 Thread t . broyer
http://gwt-code-reviews.appspot.com/1099801/diff/26001/27009 File user/src/com/google/gwt/user/client/ui/DateLabel.java (right): http://gwt-code-reviews.appspot.com/1099801/diff/26001/27009#newcode25 user/src/com/google/gwt/user/client/ui/DateLabel.java:25: * A ValueLabel that uses {...@link

[gwt-contrib] Re: Add Locator API to allow arbitrary domain types to be used with RequestFactory. (issue1130801)

2010-11-19 Thread t . broyer
http://gwt-code-reviews.appspot.com/1130801/diff/1/10 File user/src/com/google/gwt/requestfactory/server/LocatorServiceLayer.java (right): http://gwt-code-reviews.appspot.com/1130801/diff/1/10#newcode146 user/src/com/google/gwt/requestfactory/server/LocatorServiceLayer.java:146: Locator?, ?

[gwt-contrib] Re: Issues 5479, 5507 and 5571 (and a few other editor-related tweaks) (issue1099801)

2010-11-18 Thread t . broyer
http://gwt-code-reviews.appspot.com/1099801/diff/26001/27004 File user/src/com/google/gwt/uibinder/elementparsers/DateLabelParser.java (right): http://gwt-code-reviews.appspot.com/1099801/diff/26001/27004#newcode36 user/src/com/google/gwt/uibinder/elementparsers/DateLabelParser.java:36: static

[gwt-contrib] Re: Issues 5479, 5507 and 5571 (and a few other editor-related tweaks) (issue1099801)

2010-11-15 Thread t . broyer
is ValueLabelDouble (which accepts a Renderer? super Double argument in its ctor) but the ownerFieldClass is the JGenericType ValueLabelT (expecting a Renderer? super T). I tried changing the ownerFieldClass to uiField.getType() (when uiField is not null *and* its getType().getRawType() is more specific than

[gwt-contrib] Re: Issues 5479, 5507 and 5571 (and a few other editor-related tweaks) (issue1099801)

2010-11-13 Thread t . broyer
http://gwt-code-reviews.appspot.com/1099801/diff/1/8 File user/src/com/google/gwt/user/client/ui/NumberLabel.java (right): http://gwt-code-reviews.appspot.com/1099801/diff/1/8#newcode34 user/src/com/google/gwt/user/client/ui/NumberLabel.java:34: } On 2010/11/11 19:06:31, jat wrote: On

[gwt-contrib] Issues 5479, 5507 and 5571 (and a few other editor-related tweaks) (issue1099801)

2010-11-11 Thread t . broyer
Reviewers: bobv, Description: This patch: - reuses editors in every IsEditor widget (issue 5479) - adds a ValueLabel widget, and DateLabel and NumberLabel subclasses (issue 5507), introduces a NumberFormatRenderer (similar to the existing DateTimeFormatRenderer) for use in NumberLabel (this

[gwt-contrib] Re: Issues 5479, 5507 and 5571 (and a few other editor-related tweaks) (issue1099801)

2010-11-11 Thread t . broyer
Thanks for the quick feedback! http://gwt-code-reviews.appspot.com/1099801/diff/1/8 File user/src/com/google/gwt/user/client/ui/NumberLabel.java (right): http://gwt-code-reviews.appspot.com/1099801/diff/1/8#newcode22 user/src/com/google/gwt/user/client/ui/NumberLabel.java:22: * A ValueLabel

[gwt-contrib] Re: First-pass for adding HTML5's Canvas. (issue1082801)

2010-11-10 Thread t . broyer
http://gwt-code-reviews.appspot.com/1082801/diff/1/4 File user/src/com/google/gwt/html5/canvas/client/CanvasElement.java (right): http://gwt-code-reviews.appspot.com/1082801/diff/1/4#newcode42 user/src/com/google/gwt/html5/canvas/client/CanvasElement.java:42: public final native Context

[gwt-contrib] Re: First-pass for adding HTML5's Canvas. (issue1082801)

2010-11-07 Thread t . broyer
http://gwt-code-reviews.appspot.com/1082801/diff/1/10 File user/src/com/google/gwt/html5/canvas/client/CssColor.java (right): http://gwt-code-reviews.appspot.com/1082801/diff/1/10#newcode23 user/src/com/google/gwt/html5/canvas/client/CssColor.java:23: public class CssColor implements

[gwt-contrib] Re: First-pass for adding HTML5's Canvas. (issue1082801)

2010-11-07 Thread t . broyer
http://gwt-code-reviews.appspot.com/1082801/diff/1/7 File user/src/com/google/gwt/html5/canvas/client/CanvasPixelArray.java (right): http://gwt-code-reviews.appspot.com/1082801/diff/1/7#newcode41 user/src/com/google/gwt/html5/canvas/client/CanvasPixelArray.java:41: return this[i]; On 2010/11/07

[gwt-contrib] Re: First-pass for adding HTML5's Canvas. (issue1082801)

2010-11-05 Thread t . broyer
http://gwt-code-reviews.appspot.com/1082801/diff/1/3 File user/src/com/google/gwt/html5/canvas/client/Canvas.java (right): http://gwt-code-reviews.appspot.com/1082801/diff/1/3#newcode52 user/src/com/google/gwt/html5/canvas/client/Canvas.java:52: * @return the height, in pixels Are they really

Re: Can't compile in eclipe - unhandled event loop exception

2010-11-04 Thread Stephan T
why you'd be running into problems against an Eclipse 3.5 version of an older incarnation of STS. Did you have the Google Plugin for Eclipse installed previously? Was this an upgrade? On Wed, Nov 3, 2010 at 5:06 AM, Stephan T stephan.tern...@gmail.com wrote: Sometimes I also get this error

Re: Can't compile in eclipe - unhandled event loop exception

2010-11-03 Thread Stephan T
of the plugins in there that start with com.google.gdt.*? Also, do you see any errors in your Error Log (Window - Show View - Error Log)? On Tue, Nov 2, 2010 at 11:12 AM, Stephan T stephan.tern...@gmail.comwrote: I can't compile my gwt application in Eclipse. Works in Maven outside Eclipse though

Re: Can't compile in eclipe - unhandled event loop exception

2010-11-03 Thread Stephan T
) at java.lang.ClassLoader.loadClass(ClassLoader.java:248) ... 37 more On 3 Nov, 09:52, Stephan T stephan.tern...@gmail.com wrote: Yes, it's the Unhandledeventloopexception that shows up in the Error log. In my plugin folder: com.google.gdt.eclipse.core_1.4.0.v201010280047.jar

[gwt-contrib] Re: Loosens up JSO restrictions in the TypeOracle to allow multiple JSO (issue1076801)

2010-11-03 Thread t . broyer
http://gwt-code-reviews.appspot.com/1076801/diff/1/2 File dev/core/src/com/google/gwt/core/ext/typeinfo/TypeOracle.java (right): http://gwt-code-reviews.appspot.com/1076801/diff/1/2#newcode676 dev/core/src/com/google/gwt/core/ext/typeinfo/TypeOracle.java:676: // TODO(jgw): Not sure if

Can't compile in eclipe - unhandled event loop exception

2010-11-02 Thread Stephan T
I can't compile my gwt application in Eclipse. Works in Maven outside Eclipse though. I'm getting an error related to IPixelConverter (see below for full stacktrace). The version of Eclipse is 3.5.1 (SpringSource Tool Suite2.3.0) and version 2.1.0 of GWT Plugin installed from the 3.5 update site.

[gwt-contrib] Re: Overhaul the RequestFactory server code. (issue1062801)

2010-10-29 Thread t . broyer
http://gwt-code-reviews.appspot.com/1062801/diff/1/75 File user/src/com/google/gwt/requestfactory/server/ReflectiveServiceLayer.java (right): http://gwt-code-reviews.appspot.com/1062801/diff/1/75#newcode77 user/src/com/google/gwt/requestfactory/server/ReflectiveServiceLayer.java:77: return

[gwt-contrib] Re: Overhaul the RequestFactory server code. (issue1062801)

2010-10-29 Thread t . broyer
http://gwt-code-reviews.appspot.com/1062801/diff/1/78 File user/src/com/google/gwt/requestfactory/server/RequestFactoryInterfaceValidator.java (right): http://gwt-code-reviews.appspot.com/1062801/diff/1/78#newcode947

Adding an additional module to the start project

2010-10-22 Thread Kasper T.
Hi, I am trying to figure out how to organize my project so i figured I would try to make a simple project with multiple modules to see how it works. I got the starter project running just fine and i can get any modifications to the existing module to work as well. (I can't find any tutorial

[gwt-contrib] Re: Making Composite implement RequiresResize and ProvidesResize so that it can pass resize informat... (issue1041801)

2010-10-21 Thread t . broyer
Isn't that duplicating the functionnality of the ResizeComposite? http://google-web-toolkit.googlecode.com/svn/javadoc/2.1/com/google/gwt/user/client/ui/ResizeComposite.html Should it then be deprecated? At least it should itself implement ResizeComposite, as it would now inherit the behavior

Re: Why can't see progress bar?

2010-10-18 Thread Hendrik T
Not sure, but maybe you forgot to add the style? It sounds like you are using the ProgressBar from the incubator. It needs a style to be displayed properly. A working css style definition can be found in the java doc, iirc. -- You received this message because you are subscribed to the Google

[gwt-contrib] Make sure redundant calls to an Activity's display setWidget are OK (issue999802)

2010-10-16 Thread t . broyer
Reviewers: rjrjr, Description: See one use-case in point 3 of http://groups.google.fr/group/google-web-toolkit-contributors/msg/af8578d95692aff5 Another use case: edit and details of a proxy is based on the proxy itself (it has an isReadOnly() property, and if it isn't read-only it's always in

[gwt-contrib] Re: Let ToggleButton implement HasValueBoolean (issue887804)

2010-10-08 Thread t . broyer
On 2010/10/08 21:55:30, jgw wrote: On 2010/09/23 14:32:37, tbroyer wrote: LGTM. Committed at r8982. Thanks! http://gwt-code-reviews.appspot.com/887804/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: Changes in com.google.gwt.app land to catch up to the RequestFactory (issue946802)

2010-10-04 Thread t . broyer
http://gwt-code-reviews.appspot.com/946802/diff/8001/9005 File user/src/com/google/gwt/app/place/CreateAndEditProxy.java (right): http://gwt-code-reviews.appspot.com/946802/diff/8001/9005#newcode54 user/src/com/google/gwt/app/place/CreateAndEditProxy.java:54: display.setWidget(null); Er, really

[gwt-contrib] Re: Changes required to make the Scaffold app look like the mocks. Added null checks AbstractProxyLi... (issue925801)

2010-09-28 Thread t . broyer
On 2010/09/27 23:49:02, rjrjr wrote: Point taken on the view contention front. I think Thomas is right that setting and clearing the view delegate should happen in start() and stop(), not in the constructor. Thanks, Thomas. You're welcome ;-) Sounds like the javadoc should say so. +1

[gwt-contrib] Re: Changes required to make the Scaffold app look like the mocks. Added null checks AbstractProxyLi... (issue925801)

2010-09-27 Thread t . broyer
http://gwt-code-reviews.appspot.com/925801/diff/1/2 File user/src/com/google/gwt/app/place/AbstractProxyListActivity.java (right): http://gwt-code-reviews.appspot.com/925801/diff/1/2#newcode256 user/src/com/google/gwt/app/place/AbstractProxyListActivity.java:256: if (view == null) { Isn't the

[gwt-contrib] Re: Fixing a bug with images used in CellTree, CellBrowser, and IconCellDecorator in browsers that b... (issue912801)

2010-09-23 Thread t . broyer
Looks like there's a regression in IconCellDecorator (re. valign) http://gwt-code-reviews.appspot.com/912801/diff/1/2 File user/src/com/google/gwt/cell/client/IconCellDecorator.java (right): http://gwt-code-reviews.appspot.com/912801/diff/1/2#newcode53

[gwt-contrib] Let ToggleButton implement HasValueBoolean (issue887804)

2010-09-23 Thread t . broyer
Reviewers: jgw, Description: also fixes issue 5076 (add ToggleButton(upText,downText,handler) constructor) and a few nits in the CheckBox javadoc (referring to text box, and bad doc re. null value handling in setValue) Assigning to Joel as per

[gwt-contrib] Re: Replaces the public and obnoxious String EntityProxy#getId() with the (issue902801)

2010-09-22 Thread t . broyer
http://gwt-code-reviews.appspot.com/902801/diff/20001/21013 File user/src/com/google/gwt/app/place/AbstractProxyEditActivity.java (right): http://gwt-code-reviews.appspot.com/902801/diff/20001/21013#newcode147 user/src/com/google/gwt/app/place/AbstractProxyEditActivity.java:147: stableId =

[gwt-contrib] Re: Makes IsWidget a first class concept, and makes it convenient to (issue864801)

2010-09-13 Thread t . broyer
LGTM (FWIW, and speaking only about the API) http://gwt-code-reviews.appspot.com/864801/diff/3001/4034 File user/test/com/google/gwt/user/client/ui/SimplePanelTest.java (right): http://gwt-code-reviews.appspot.com/864801/diff/3001/4034#newcode35

[gwt-contrib] Re: Makes IsWidget a first class concept, and makes it convenient to (issue864801)

2010-09-12 Thread t . broyer
Are all these ForIsWidget interfaces (and method overloads) really needed? (and actually useful?) For completeness, TabLayoutPanel and TabPanel don't have IsWidget overloads for their add, getTabWidget and selectTab methods. (if you ask me, I'd only keep HasOneWidget, without the getWidget

[gwt-contrib] Adds getInheritableMethods to JClassType (issue863802)

2010-09-11 Thread t . broyer
Reviewers: bruce, scottb, Description: Similar to getOverridableMethods but includes final methods as well. This would make it easier to fix ROO-1297 and in general work with factories (such as the code generator for ActivityMapper I proposed in the Activity design Wave)

[gwt-contrib] Re: Adds getInheritableMethods to JClassType (issue863802)

2010-09-11 Thread t . broyer
I think I've spotted the reason for the test failure: it's actually a bug in the current getOverridableMethods, and a bug in JEnumTypeTest (it should have really expected 4 methods instead of 7). Reported as issue 5270: http://code.google.com/p/google-web-toolkit/issues/detail?id=5270

[gwt-contrib] Fixes issue 5270 (issue839803)

2010-09-11 Thread t . broyer
Reviewers: bruce, scottb, Description: http://code.google.com/p/google-web-toolkit/issues/detail?id=5270 Description copied from the issue: In c.g.g.core.ext.typeinfo.AbstractMembers, final methods are simply ignored instead of being treated as possible overrides of already seen methods. A

[gwt-contrib] Alternate PlaceHistoryHandler design/API (issue845802)

2010-09-05 Thread t . broyer
Reviewers: rjrjr, Description: Following up on https://jira.springsource.org/browse/ROO-1276, here's the proposed alternate design, separating the keep history and places in sync and map places to/from history tokens concerns into a concrete PlaceHistoryHandler class and a PlaceHistoryMapper

[gwt-contrib] Re: Fix the escaping done by UrlBuilder. (issue754803)

2010-09-02 Thread t . broyer
I don't quite understand what this change in RemoteServiceServlet has to do with UrlBuilder... http://gwt-code-reviews.appspot.com/754803/diff/21001/22002 File user/src/com/google/gwt/user/server/rpc/RemoteServiceServlet.java (right):

[gwt-contrib] Re: Fix the escaping done by UrlBuilder. (issue754803)

2010-09-02 Thread t . broyer
In brief: URIs (RFC 3986) is a mess, and back to URL.encode for the path in UrlBuilder? (with a second pass though) http://gwt-code-reviews.appspot.com/754803/diff/21001/22002 File user/src/com/google/gwt/user/server/rpc/RemoteServiceServlet.java (right):

[gwt-contrib] Allow a fallback/catch-all and prefix-less PlaceTokenizer makes Historian pluggable (issue824801)

2010-09-01 Thread t . broyer
Reviewers: rjrjr, Description: Fixes https://jira.springsource.org/browse/ROO-1276 First, the anonymous-class implementation of Historian is moved into a public static DefaultHistorian nested class that's GWT.create()d to allow pluggability via a replace-with rule in a gwt.xml file (similar to

[gwt-contrib] Addresses various issues with PlaceHistoryHandlerGenerator (issue827801)

2010-09-01 Thread t . broyer
Reviewers: , Description: This is a major refactor of PlaceHistoryHandlerGenerator/PlaceHistoryGeneratorContext to address various issues with the current implementation: - place types should be ordered from most-derived to least-derived in the generated getPrefixAndToken (see

[gwt-contrib] Re: Fix the escaping done by UrlBuilder. (issue754803)

2010-08-24 Thread t . broyer
http://gwt-code-reviews.appspot.com/754803/diff/1/2 File user/src/com/google/gwt/http/client/UrlBuilder.java (right): http://gwt-code-reviews.appspot.com/754803/diff/1/2#newcode69 user/src/com/google/gwt/http/client/UrlBuilder.java:69: for (String segment : path.split(/)) { It'd probably be

[gwt-contrib] Re: Added all safehtml packages. (issue771801)

2010-08-23 Thread t . broyer
http://gwt-code-reviews.appspot.com/771801/diff/28002/6006 File user/src/com/google/gwt/safehtml/shared/OnlyToBeUsedInGeneratedCodeStringBlessedAsSafeHtml.java (right): http://gwt-code-reviews.appspot.com/771801/diff/28002/6006#newcode32

[gwt-contrib] Re: Added all safehtml packages. (issue771801)

2010-08-20 Thread t . broyer
http://gwt-code-reviews.appspot.com/771801/diff/11002/15007 File user/src/com/google/gwt/safehtml/shared/OnlyToBeUsedInGeneratedCodeStringBlessedAsSafeHtml.java (right): http://gwt-code-reviews.appspot.com/771801/diff/11002/15007#newcode38

[gwt-contrib] Re: Added all safehtml packages. (issue771801)

2010-08-18 Thread t . broyer
On 2010/08/17 23:23:39, xtof wrote: On 2010/08/17 23:05:06, tbroyer wrote: Looking at the code more closely it would merely fail by overly rejecting tags that are whitelisted: i.e. b foo=ishould be bold would be sanitized to lt;b foo=ishould be bold and the end part would be italicized

[gwt-contrib] Re: Continuation of r8542 to actually properly enables double click for (issue774801)

2010-08-17 Thread t . broyer
LGTM, though I'm puzzled by Hyperlink implementing HasDoubleClickHandlers, given that addClickHandler is deprecated on that class. http://gwt-code-reviews.appspot.com/774801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: Added all safehtml packages. (issue771801)

2010-08-17 Thread t . broyer
On 2010/08/17 20:19:04, xtof wrote: On 2010/08/17 18:39:28, jat wrote: On 2010/08/16 23:39:47, tbroyer wrote: The HtmlSanitizer is a good idea, but the implementation is very weak [2]. Note that the API is what is important, and SimpleHtmlSanitizer is just that, a simple

Re: Building two war-files with Maven using different web.xml

2010-08-16 Thread Stephan T
dane.molo...@gmail.com wrote: Have you thought about using themaven-assembly-plugin to generate a secondary end-state? Just an idea. On Aug 12, 6:23 am, Stephan T stephan.tern...@gmail.com wrote: I need to build two war files with different web.xml files becuase of different servlet filter

[gwt-contrib] Re: Added all safehtml packages. (issue771801)

2010-08-16 Thread t . broyer
I really like the interface/generator idea but not really the implementation (XML parsing –even though I could live with this–, generating XML –the HTML5 serialization algorithm [1] is fortunately easy enough and interoperable–) The HtmlSanitizer is a good idea, but the implementation is very

Using UI Binder

2010-08-13 Thread $ a r j i t h Pullithodi
HI All, I am a beginner in GWT, I was trying to develop an application through UI Binder way. but it found really difficult to integrate designers (html) code to gwt widgets. is it possible to use g:widgets inside html tag and vice versa?. I can't use complete html as ui binder xml since its

[gwt-contrib] Re: History integration for the RequestFactory apps. (issue717801)

2010-08-13 Thread t . broyer
I only quickly read through the list of added files and some of the files' content, but I can say I really like it the way it is (with PlaceTokenizer instead of the static fromToken method described in the Wave) There's a nit though, which should be probably fixed rather sooner than later ;-)

Re: Instantiate composite on the server side

2010-08-06 Thread Stephan T
. Your composite would then delegate to the POJO, which could be reused server-side. And if you wish to push further towards great design, embrace MVP ! On 5 août, 14:42, Stephan T stephan.tern...@gmail.com wrote: For several reasons I need to instansiate a Composite on the server side

Building two war-files with Maven using different web.xml

2010-08-06 Thread Stephan T
I'm using the GWT-plugin for Maven to build my application. I want to build two war-files where the only difference is that I want to use a different web.xml. How do I achieve that? Here's my pom.xml: ?xml version=1.0 encoding=UTF-8? project xmlns=http://maven.apache.org/POM/4.0.0;

Instantiate composite on the server side

2010-08-05 Thread Stephan T
For several reasons I need to instansiate a Composite on the server side that is in the client package. Is it possible? Now I think I'm getting class not found exception when trying... -- You received this message because you are subscribed to the Google Groups Google Web Toolkit group. To post

[gwt-contrib] Re: Puts PlaceController in the business of showing the user (issue698801)

2010-07-16 Thread t . broyer
LGTM (awesome! next step: integration with History ;-) ) http://gwt-code-reviews.appspot.com/698801/diff/8001/9002 File user/src/com/google/gwt/app/place/AbstractRecordEditActivity.java (right): http://gwt-code-reviews.appspot.com/698801/diff/8001/9002#newcode62

[gwt-contrib] Re: Adds Java classes for JSON serialization/deserialization to complement (issue696801)

2010-07-15 Thread t . broyer
How is this better than existing JSON libraries, such as Google GSON? http://code.google.com/p/google-gson/ Moreover, GSON's use of reflection (JsonDeserializatonVisitor ObjectNavigator) might make it possible to use it for RequestFactory (both servlet, and client-side code generation) with

[gwt-contrib] Re: Fix issue 4781: allow HTML= in UiBinder for setHTML() (issue642801)

2010-07-02 Thread t . broyer
http://gwt-code-reviews.appspot.com/642801/diff/1/3 File user/test/com/google/gwt/uibinder/rebind/model/OwnerFieldClassTest.java (right): http://gwt-code-reviews.appspot.com/642801/diff/1/3#newcode62 user/test/com/google/gwt/uibinder/rebind/model/OwnerFieldClassTest.java:62:

[gwt-contrib] Re: Adds EventTarget, add/removeEventListener() et al to gwt dom. (issue623803)

2010-07-02 Thread t . broyer
LGTM overall. As a side note, where is the c.g.g.dom.client.EventListener defined? Could it be made generic? I like how the WebSocket proposal added a base Event class mimicking the DOM Event interface, with NativeEvent (and other WebSocket-specific events) extending it. EventListener could

[gwt-contrib] Re: Add WebSocket API (issue646803)

2010-07-01 Thread t . broyer
In a few words: everything's in c.g.g.dom, bringing a dependency from c.g.g.dom to c.g.g.event and breaking the assumption that an EventHandler is coupled with a GwtEvent (rather than a JSO-derivative), and it starts to refactor event handling at the low-level, while Joel is working on it (see

[gwt-contrib] Re: Rolling back r8329 because it hands in IE8. (issue630802)

2010-06-29 Thread t . broyer
On 2010/06/29 21:56:58, jlabanca wrote: I think all of the doTestParseUnescaped loops in JSONTest are causing the tests to timeout. We'll have to break these tests down so they don't timeout. How about taking advantage of being in a GWTTestCase and use a Scheduler.RepeatingCommand?

[gwt-contrib] Fix issue 5065: Float.parseFloat and Double.parseDouble are too strict validating input (issue647802)

2010-06-28 Thread t . broyer
Reviewers: , Description: Make Float.parseFloat/valueOf and Double.parseDouble/valueOf accept strings with a float type suffix, such as 1.0f or 1.0d. Please review this at http://gwt-code-reviews.appspot.com/647802/show Affected files: user/super/com/google/gwt/emul/java/lang/Number.java

[gwt-contrib] Re: Fix external issue 5052 - JSONParser.parse exceptions with some unicode characters (issue659801)

2010-06-28 Thread t . broyer
http://gwt-code-reviews.appspot.com/659801/diff/9001/10001 File user/src/com/google/gwt/core/client/JsonUtils.java (right): http://gwt-code-reviews.appspot.com/659801/diff/9001/10001#newcode89 user/src/com/google/gwt/core/client/JsonUtils.java:89: out[0xad] = '\\u00ad'; // Soft hyphen On

[gwt-contrib] Re: Fix external issue 5052 - JSONParser.parse exceptions with some unicode characters (issue659801)

2010-06-25 Thread t . broyer
http://gwt-code-reviews.appspot.com/659801/diff/9001/10001 File user/src/com/google/gwt/core/client/JsonUtils.java (right): http://gwt-code-reviews.appspot.com/659801/diff/9001/10001#newcode76 user/src/com/google/gwt/core/client/JsonUtils.java:76: return typeof JSON == object typeof JSON.parse

[gwt-contrib] Re: Fix external issue 5052 - JSONParser.parse exceptions with some unicode characters (issue659801)

2010-06-25 Thread t . broyer
Conclusion: Safari 5 is half-baked :-( http://gwt-code-reviews.appspot.com/659801/diff/9001/10001 File user/src/com/google/gwt/core/client/JsonUtils.java (right): http://gwt-code-reviews.appspot.com/659801/diff/9001/10001#newcode76 user/src/com/google/gwt/core/client/JsonUtils.java:76: return

[gwt-contrib] Fix issue 5061: backslashes are not escaped by UiBinder in HasHTML and HasText (issue620804)

2010-06-25 Thread t . broyer
Reviewers: Ray Ryan, Description: UiBinderWriter#escapeTextForJavaStringLiteral correctly escapes \n and , but forgets \. Also, I couldn't see a reason for using replaceAll instead of replace. Please review this at http://gwt-code-reviews.appspot.com/620804/show Affected files:

[gwt-contrib] Re: Fix external issue 5052 - JSONParser.parse exceptions with some unicode characters (issue659801)

2010-06-24 Thread t . broyer
by apostrophes while JSON mandates quotes, numbers in JSON have a stricter syntax than JS number literals, whitespace in JSON is limited to \u0020\t\r\n whereas JavaScript allows \v\f\uFEFF and any character from the Unicode Zs category, etc.) http://gwt-code-reviews.appspot.com/659801/show -- http

[gwt-contrib] Re: Escaping HTML strings from the client as a good practice to avoid XSS vulnerabilities in apps th... (issue619803)

2010-06-22 Thread t . broyer
Sorry to step in without being invited (though I was earlier told that it should be expected when developing in the open), but... (see below) http://gwt-code-reviews.appspot.com/619803/diff/1/2 File user/src/com/google/gwt/user/tools/RpcServerTemplate.javasrc (right):

[gwt-contrib] Fix issue 4781: allow HTML= in UiBinder for setHTML() (issue642801)

2010-06-18 Thread t . broyer
Reviewers: Ray Ryan, Description: Uses the java.beans.Introspector#decapitalize instead of manual decapitalization of setter-method names to attribute names. This means setHTML() can be set using an all-caps HTML= attribute, instead of hTML=. This applies to all all-caps setters (actually,

[gwt-contrib] Fix issue 150: regression re. keyCode/charCode (issue643801)

2010-06-18 Thread t . broyer
Reviewers: jlabanca, Description: As a result of r7692, NativeEvent#getKeyCode no longer returns the charCode for KeyPress events. This leads to better accuracy in handling keyboard-related events, but some widgets still use getKeyCode on KeyPress events, which in some (many? most?) browsers

Disabling DatePicker

2010-06-10 Thread Stephan T
How do I disable the DatePicker widget programatically? Can't find any suitable setEnabled-method... -- You received this message because you are subscribed to the Google Groups Google Web Toolkit group. To post to this group, send email to google-web-tool...@googlegroups.com. To unsubscribe

Label for form-element using UIBindings

2010-05-31 Thread Stephan T
How do I achieve this code this using UI Bindings? label for=someUniqueFieldIdFirst name/label input type=text id=someUniqueFieldId name=firstname/ -- You received this message because you are subscribed to the Google Groups Google Web Toolkit group. To post to this group, send email to

<    1   2   3   4   5   6   7   8   >