Re: GWT ui:style how big of a bad idea (or not) is it to use it?

2012-08-09 Thread Nuno Godinho de Matos
Thank you all for your replies,

And In general I tend to agree that you should not promote tight coupling 
between your app functionality and the presentation styles.
On the other hand, Ryan Shillington, JQuery is javascript, but as you know 
most Great Jscript libraries jave wrappers for GWT. That is the case with 
jquery, http://code.google.com/p/gwt-jquery/. So yes, you can use JQuery 
from withing GWT.

Another thing is, 
say, for example that in page that you are rendered you have a list of row 
and each of these rows has a cless ODD or EVEN. Say that you want to hide 
away all ODD rows.
Why would it be best to have your GWT code traverse every single row in the 
set an based on your index % 2 decide weather or not collapse it, than have 
a CSS selector in gwt-jquery kill away all ODD rows?
Much less code!
I would normally live very well wich such a depencency, provided that if 
would be making my code use a CSS resource getter to find out the ODD 
classname.

My kindest regards,
Nuno.


On Saturday, August 4, 2012 11:33:09 PM UTC+2, Joseph Lust wrote:

 To start with CSSResource is awesome and I highly recommend using it. It 
 forces you to program *better*. Should you really be trying to pull 
 elements out of the page depending on their CSS names? Seems like a bad day 
 if you ever want to rebrand or dress up your UI. If CSS is the graphical 
 dressing of the site, separate those concerns from the functional aspects. 
 Further, using those CSSResource interfaces keeps you from ever using 
 string literals in your Java code or UiBinders, which is a best practice.

 Additionally, with UI Automation tests our testers are tempted to use ugly 
 selectors based on class names. This is a poor idea because then a class 
 name changes a year later and 400 tests fail, even though functionally the 
 site works just as well as before.

 The real question here is what you want to do with your CSS. If you want 
 to apply rules the site over, then don't use ui:style, but rather a 
 CSSResource interface. This way that interface can be used by many widgets' 
 UiBinders. However bespoke, small CSS tweaks do fit nicely into ui:style 
 and you can always move them out to a CSSResource interface in the future 
 if you want to use them multiple places. Note, from my experience you 
 cannot well handle CSS inheritance/overriding between widgets with 
 ui:style, but you can with standalone CSSResource interfaces.


 Sincerely,
 Joseph


-- 
You received this message because you are subscribed to the Google Groups 
Google Web Toolkit group.
To view this discussion on the web visit 
https://groups.google.com/d/msg/google-web-toolkit/-/WyVpjyTcYf8J.
To post to this group, send email to google-web-toolkit@googlegroups.com.
To unsubscribe from this group, send email to 
google-web-toolkit+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/google-web-toolkit?hl=en.



Re: jQuery sortable list with GWT

2012-08-09 Thread David


On Sunday, August 5, 2012 6:40:13 PM UTC-4, CSchulz wrote:

 GWT doesn't have any native support for this sort of thing, that's why 
 I'm trying to drop in some basic jquery code.

 gwt-dnd ( http://code.google.com/p/gwt-dnd/) works pretty well for this 
 sort of thing.


-- 
You received this message because you are subscribed to the Google Groups 
Google Web Toolkit group.
To view this discussion on the web visit 
https://groups.google.com/d/msg/google-web-toolkit/-/DgbiY4DS_pkJ.
To post to this group, send email to google-web-toolkit@googlegroups.com.
To unsubscribe from this group, send email to 
google-web-toolkit+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/google-web-toolkit?hl=en.



Re: facebook authentication

2012-08-09 Thread Sydney
https://github.com/fernandezpablo85/scribe-java/

On Tuesday, August 7, 2012 4:37:44 PM UTC+2, aamonten wrote:

 Hi,
 developing an app and would like to use facebook api to do user 
 authentication. Does any has a pointer about how to do this integrated with 
 GWT?

 thanks
 Alejandro


-- 
You received this message because you are subscribed to the Google Groups 
Google Web Toolkit group.
To view this discussion on the web visit 
https://groups.google.com/d/msg/google-web-toolkit/-/5lpNJMCAm0YJ.
To post to this group, send email to google-web-toolkit@googlegroups.com.
To unsubscribe from this group, send email to 
google-web-toolkit+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/google-web-toolkit?hl=en.



Re: GWT future

2012-08-09 Thread Sydney
gwtp should release a 0.8 or 1.0 version soon. 
See https://groups.google.com/d/topic/gwt-platform/h-4KPGKFIoM/discussion

On Wednesday, August 8, 2012 10:23:04 PM UTC+2, David wrote:

 That would be gwt-platform as in gwtP  
 http://code.google.com/p/gwt-platform.

 I'm not looking for comparisons to other MVP solutions.  I'm just 
 wondering if gwtP is something that will continue to be developed.
 thanks.  



 On Wednesday, August 8, 2012 1:20:19 PM UTC-4, Stevko wrote:

 David,
 Its a BUY-long  on my list. 
 Seeing the huge line of peeps waiting for the history  future session 
 was a sure signal that there is a lot of interest (outside of google).
 The volume of GWT recruiter hits on my LinkedIn profile means there is 
 new work being done,
 Moving to an external F/OSS respository and master commiters will 
 increase the velocity of the framework rather than waiting months  years 
 for google to vet the commits to protect the stability of their internal 
 systems.  (see my signature for my philosophy) 
 Not to say its a sure bet. Growing up is hard to do...



 On Wed, Aug 8, 2012 at 7:56 AM, David lev...@gmail.com wrote:

 Anyone have insight as to the future of gwt-platform?




 -- 
 -- A. Stevko
 ===
 If everything seems under control, you're just not going fast enough. 
 M. Andretti





  

-- 
You received this message because you are subscribed to the Google Groups 
Google Web Toolkit group.
To view this discussion on the web visit 
https://groups.google.com/d/msg/google-web-toolkit/-/Nen0TW0U0G8J.
To post to this group, send email to google-web-toolkit@googlegroups.com.
To unsubscribe from this group, send email to 
google-web-toolkit+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/google-web-toolkit?hl=en.



Re: GWT ui:style how big of a bad idea (or not) is it to use it?

2012-08-09 Thread Joseph Lust


 you have a list of row and each of these rows has a cless ODD or EVEN. Say 
 that you want to hide away all ODD rows.
 Why would it be best to have your GWT code traverse every single row in 
 the set an based on your index % 2 decide weather or not collapse it, than 
 have a CSS selector in gwt-jquery kill away all ODD rows?


Set a class 'odd' and 'even' and place those on each row when you make 
them. Then just use a CSS selector to show or hide all 'odd' rows by 
element.addClassName('hideOdd') on the parent container. Just a single 
operation in GWT.


Sincerely,
Joseph

-- 
You received this message because you are subscribed to the Google Groups 
Google Web Toolkit group.
To view this discussion on the web visit 
https://groups.google.com/d/msg/google-web-toolkit/-/YVauwMIIAaQJ.
To post to this group, send email to google-web-toolkit@googlegroups.com.
To unsubscribe from this group, send email to 
google-web-toolkit+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/google-web-toolkit?hl=en.



Re: Examples and implementation of Dynamic String Internationalization for UIBinder and its classes.

2012-08-09 Thread Joseph Lust
Nataraj,

I know you don't want to redeploy you application each time a string 
changes, but calling an external service can also complicate things. I say 
this from experience as a group at my company did a very large (200K line, 
400 screen) GWT app last year which had to work in many languages. They did 
not use the GWT i18n tools because the company wanted to use a service like 
you describe. However, the result was a much slower application because 
various parts of the GWT code had to wait for the i18n files to be loaded 
from the company's (very slow) i18n service before it could render anything 
on a given page or screen. In hindsight, they wish they had used the GWT 
i18n facilities since such strings rarely are ever changed in production.


Hope that helps.

Sincerely,
Joseph

-- 
You received this message because you are subscribed to the Google Groups 
Google Web Toolkit group.
To view this discussion on the web visit 
https://groups.google.com/d/msg/google-web-toolkit/-/6r9JasqdCJ8J.
To post to this group, send email to google-web-toolkit@googlegroups.com.
To unsubscribe from this group, send email to 
google-web-toolkit+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/google-web-toolkit?hl=en.



GWT Multi-Module Server Classpath Problem

2012-08-09 Thread Geoffrey Wiseman
I have a 2yo GWT project that's getting a little big for a single module, 
and we're bringing in a new project and we'd like to share some code.

Accordingly, I've split the project into a 'common' codebase and a 
project-specific codebase. That split isn't perfect yet, but I can't seem 
to get Eclipse to launch the project. If I launch the project through mvn 
gwt:run, it launches successfully, but if I launch it through Eclipse (Run 
As  Web Application) the server part fails to start up when a filter can't 
find the SLF4J LoggerFactory.

If I do some introspection on the classpath from the servlet, it seems like 
all of the JARs in the project aren't on the classpath -- only the classes 
and dependent projects are there.

The launch configuration classpath seems ok, and the command-line startup 
seems to work. It doesn't seem to matter if I use m2e (with its Maven 
Dependencies container) or mvn eclipse:eclipse, which adds the JARs to 
the project .classpath directly. I'm running low on ideas.

So:

   1. Anyone experience this or something like it and know what I should 
   look for?
   2. How does the server get its classpath -- from the launch 
   configuration classpath or something different?
   3. Is anyone using multiple eclipse projects with the Google Eclipse 
   Plugin successfully? (from other threads, it sounded like there were people 
   who had it working, but ...)

Any other thoughts or suggestions welcome.

-- 
You received this message because you are subscribed to the Google Groups 
Google Web Toolkit group.
To view this discussion on the web visit 
https://groups.google.com/d/msg/google-web-toolkit/-/ty-qiodo-4wJ.
To post to this group, send email to google-web-toolkit@googlegroups.com.
To unsubscribe from this group, send email to 
google-web-toolkit+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/google-web-toolkit?hl=en.



Problems sorting Column in CellTable

2012-08-09 Thread Pete
Hi all,

I am using GWT 2.5 to create a CellTable with a sortable Date column. 
 All goes well until I click on the sortable column's header, but its not 
responsive.

Details here:
http://stackoverflow.com/questions/11883811/header-of-sortable-celltable-column-not-responsive

Any help is greatly appreciated.

Thanks!

Pete

-- 
You received this message because you are subscribed to the Google Groups 
Google Web Toolkit group.
To view this discussion on the web visit 
https://groups.google.com/d/msg/google-web-toolkit/-/JXeEL1p4d4QJ.
To post to this group, send email to google-web-toolkit@googlegroups.com.
To unsubscribe from this group, send email to 
google-web-toolkit+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/google-web-toolkit?hl=en.



GWT and automated testing tools

2012-08-09 Thread Ocean_Living
I'm having some performance issues using Rational Functional Tester as an 
automated testing tool when testing a web app built on GWT.

Can anyone recommend an automated tool that works and plays well with GWT?

Thanks!

-- 
You received this message because you are subscribed to the Google Groups 
Google Web Toolkit group.
To view this discussion on the web visit 
https://groups.google.com/d/msg/google-web-toolkit/-/g3DtPqxfvJwJ.
To post to this group, send email to google-web-toolkit@googlegroups.com.
To unsubscribe from this group, send email to 
google-web-toolkit+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/google-web-toolkit?hl=en.



GWT 2.0.0 DevMode: Strange warning when RPC with some object as parameter

2012-08-09 Thread Vitaliy K
Hi all,

when I updated my project from GWT 2.0.0 to 2.4.0, noticed that in window 
GWT Development Mode at Jetty tab there is a warning:

 [WARN] Server class 'junit.framework.AssertionFailedError' could not be 
 found in the web app, but was found on the system classpath
 [WARN] Adding classpath entry 
 'file:/C:/eclipse.Indigo/plugins/org.junit_4.8.2.v4_8_2_v20110321-1705/junit.jar'
  
 to the web app classpath for this session

I found it appears when RPC method that takes an object, such as 
java.util.Date (not String and not null) or some my class.

How to get rid of this warning? I do not think is a good idea to copy 
junit.jar into the directory war\WEB-INF\lib.

(OS: Windows 7; IDE: Eclipse Indigo)

Thank you in advance

-- 
You received this message because you are subscribed to the Google Groups 
Google Web Toolkit group.
To view this discussion on the web visit 
https://groups.google.com/d/msg/google-web-toolkit/-/0839-QvvhCQJ.
To post to this group, send email to google-web-toolkit@googlegroups.com.
To unsubscribe from this group, send email to 
google-web-toolkit+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/google-web-toolkit?hl=en.



DialogBox won't get CSS attribute box-shadow

2012-08-09 Thread Dominic Warzok
Hi everyone, 

I have a DialogBox an want to add a box-shadow to it. 

So at the end it should look like endversion.png.

But when I write my DialogBox class and use the method 
setStylePrimaryName() or setStyleName() the given String would be 
applied to the DialogBox.

CSS attributes like border-radius or background-color are shown in 
FireBug but the attribute box-shadow is missing. 

Any sugesstions ?  

-- 
You received this message because you are subscribed to the Google Groups 
Google Web Toolkit group.
To view this discussion on the web visit 
https://groups.google.com/d/msg/google-web-toolkit/-/tPxuqfWgpr0J.
To post to this group, send email to google-web-toolkit@googlegroups.com.
To unsubscribe from this group, send email to 
google-web-toolkit+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/google-web-toolkit?hl=en.

attachment: endversion.PNG

Re: DevMode not working in Chrome after update

2012-08-09 Thread Jason E
Same here with Version 21.0.1180.57 Come on, Google!

On Tuesday, July 17, 2012 7:16:03 AM UTC-7, Andy wrote:

 I updated Chrome this morning and now when I try to use DevMode I get an 
 error Could not load GWT DevMode Plugin and an offer to Download the GWT 
 Developer Plugin. When I try to download it (it's already installed), it 
 says Extensions, apps, and user scripts can only be added from the Chrome 
 Web Store. and there's no GWT plugin in the store.

-- 
You received this message because you are subscribed to the Google Groups 
Google Web Toolkit group.
To view this discussion on the web visit 
https://groups.google.com/d/msg/google-web-toolkit/-/jc7Pcbel2rMJ.
To post to this group, send email to google-web-toolkit@googlegroups.com.
To unsubscribe from this group, send email to 
google-web-toolkit+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/google-web-toolkit?hl=en.



Re: Tree View

2012-08-09 Thread Justin Das
please specify is it  the smartgwt tree grid or gwt cell tree?

On Thu, Aug 9, 2012 at 2:12 AM, Thad thad.humphr...@gmail.com wrote:

 If you mean a CellTree, you're gonna need to generate a List for each
 level of your tree. See the second example in
 http://google-web-toolkit.googlecode.com/svn/javadoc/2.4/com/google/gwt/user/cellview/client/CellTree.html


 On Wednesday, August 8, 2012 2:18:30 AM UTC-4, Praveen Jesudoss wrote:

 HI,

 I want to show tree view in my application. From db i retrieved data like
 level,id,name,parent id. I stored this in array list. now i want to show
 the data in tree view. i am not able to get correct parent node and child.

 Please help me to achieve this.

 Thanks and Regards,
 Praveen J

  --
 You received this message because you are subscribed to the Google Groups
 Google Web Toolkit group.
 To view this discussion on the web visit
 https://groups.google.com/d/msg/google-web-toolkit/-/39lqZsRvRCsJ.

 To post to this group, send email to google-web-toolkit@googlegroups.com.
 To unsubscribe from this group, send email to
 google-web-toolkit+unsubscr...@googlegroups.com.
 For more options, visit this group at
 http://groups.google.com/group/google-web-toolkit?hl=en.


-- 
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 this group, send email to 
google-web-toolkit+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/google-web-toolkit?hl=en.



Re: DevMode not working in Chrome after update

2012-08-09 Thread Brendan O'Fallon
Same issue here - no GWT Developer extension in the App Store, and it won't 
install otherwise. This happens on both my Mac (OS X 10.7) and Linux 
machines (Fedora 16). Also, developer plugin apparently only supported in 
Firefox verisons 3-10.0, but current version is 14.0. Do I really have to 
regress four versions of Firefox just to install the plugin?

On Tuesday, July 17, 2012 8:16:03 AM UTC-6, Andy wrote:

 I updated Chrome this morning and now when I try to use DevMode I get an 
 error Could not load GWT DevMode Plugin and an offer to Download the GWT 
 Developer Plugin. When I try to download it (it's already installed), it 
 says Extensions, apps, and user scripts can only be added from the Chrome 
 Web Store. and there's no GWT plugin in the store.

-- 
You received this message because you are subscribed to the Google Groups 
Google Web Toolkit group.
To view this discussion on the web visit 
https://groups.google.com/d/msg/google-web-toolkit/-/C4RDO7TQk6IJ.
To post to this group, send email to google-web-toolkit@googlegroups.com.
To unsubscribe from this group, send email to 
google-web-toolkit+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/google-web-toolkit?hl=en.



Can't get touch handlers to fire?

2012-08-09 Thread mark
Hi,

I can't get touch handlers to fire:

RootPanel rp = RootPanel.get(dummy);
rp.sinkEvents(Event.TOUCHEVENTS);
rp.addHandler(new TouchStartHandler() {
@Override
public void onTouchStart(TouchStartEvent event) {
// touch down!
}
}, TouchStartEvent.getType());

What am I missing?

Thanks

-- 
You received this message because you are subscribed to the Google Groups 
Google Web Toolkit group.
To view this discussion on the web visit 
https://groups.google.com/d/msg/google-web-toolkit/-/ytcG1GLoQUIJ.
To post to this group, send email to google-web-toolkit@googlegroups.com.
To unsubscribe from this group, send email to 
google-web-toolkit+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/google-web-toolkit?hl=en.



Re: Can't get touch handlers to fire?

2012-08-09 Thread Shawn Brown
 I can't get touch handlers to fire:

see this 
http://stackoverflow.com/questions/11875484/how-to-use-touch-handlers-does-gwt-support-them/11875997#11875997

-- 
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 this group, send email to 
google-web-toolkit+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/google-web-toolkit?hl=en.



Re: GWT and automated testing tools

2012-08-09 Thread gong min
Selenium is suggested in a book Essential GWT Building for web with Google
Web Toolkit 2.
I never touched. Can anyone give some samples if it is a good solution for
GWT auto test? Thanks.

2012/8/9 Ocean_Living winteried...@gmail.com

 I'm having some performance issues using Rational Functional Tester as an
 automated testing tool when testing a web app built on GWT.

 Can anyone recommend an automated tool that works and plays well with GWT?

 Thanks!

 --
 You received this message because you are subscribed to the Google Groups
 Google Web Toolkit group.
 To view this discussion on the web visit
 https://groups.google.com/d/msg/google-web-toolkit/-/g3DtPqxfvJwJ.
 To post to this group, send email to google-web-toolkit@googlegroups.com.
 To unsubscribe from this group, send email to
 google-web-toolkit+unsubscr...@googlegroups.com.
 For more options, visit this group at
 http://groups.google.com/group/google-web-toolkit?hl=en.




-- 
Gong Min
gongmin...@gmail.com

-- 
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 this group, send email to 
google-web-toolkit+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/google-web-toolkit?hl=en.



YUIGWT - YUI 3 in GWT

2012-08-09 Thread Sebastián Gurin
I just started a YUI 3 http://yuilibrary.comJava GWT API. It is a GWT 
library for using YUI 3.

Project home page: http://code.google.com/p/yuigwt/ 
http://code.google.com/p/yuigwt/

Example Gallery with online Java sources :http://cancerbero.vacau.com/yuigwt/

It is 99% overlay types, and the syntax try to emulate YUI JavaScript 
syntax statements as much as possible. (see 
JavaScript YUI code compared to its YUIGWT Java code equivalent: 
http://code.google.com/p/yuigwt/wiki/equivalentCodeCompared ) 

The project is very very new, but a general pattern for migrating all the 
YUI JavaScript API to Java was found, using 100% GWT Overlay types for a 
zero overhead Java API. A lot of components are already ready to be used, 
check the example gallery that will reflect current progress. 

I must say, I must to give thanks to eclipse java templates for writing GWT 
overlay methods; it is speeding my code a lot. 

Any suggestions or ideas are most welcome. 

Regards, 


-- 
You received this message because you are subscribed to the Google Groups 
Google Web Toolkit group.
To view this discussion on the web visit 
https://groups.google.com/d/msg/google-web-toolkit/-/fVGkaVLc8ZEJ.
To post to this group, send email to google-web-toolkit@googlegroups.com.
To unsubscribe from this group, send email to 
google-web-toolkit+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/google-web-toolkit?hl=en.



[gwt-contrib] Re: Added SetSelectionModel interface and made Single- and MultiSelectionModels (issue1798806)

2012-08-09 Thread tuckerpmt

On 2012/08/08 22:25:10, skybrian wrote:

Any reason clear() can't be moved into SelectionModel and not make
SingleSelectionModel implement SetSelectionModel?  It seems misleading
to make SingleSelectionModel extend SetSelectionModel when it only deals
with one element.

http://gwt-code-reviews.appspot.com/1798806/

--
http://groups.google.com/group/Google-Web-Toolkit-Contributors


[gwt-contrib] Re: Implemented a subset of the bootstrap configuration which allows for Validator customization. (issue1807803)

2012-08-09 Thread nchalko

minor nit.
LGTM


http://gwt-code-reviews.appspot.com/1807803/diff/1/user/src/com/google/gwt/validation/client/impl/BaseGwtConfiguration.java
File
user/src/com/google/gwt/validation/client/impl/BaseGwtConfiguration.java
(right):

http://gwt-code-reviews.appspot.com/1807803/diff/1/user/src/com/google/gwt/validation/client/impl/BaseGwtConfiguration.java#newcode52
user/src/com/google/gwt/validation/client/impl/BaseGwtConfiguration.java:52:
this.properties = new HashMapString, String();
move this to the declaration.

http://gwt-code-reviews.appspot.com/1807803/

--
http://groups.google.com/group/Google-Web-Toolkit-Contributors


[gwt-contrib] Re: Added SetSelectionModel interface and made Single- and MultiSelectionModels (issue1798806)

2012-08-09 Thread skybrian

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 SelectionModel can
contain multiple items; if that's unclear, perhaps it needs to be
clarified somehow?


http://gwt-code-reviews.appspot.com/1798806/

--
http://groups.google.com/group/Google-Web-Toolkit-Contributors


[gwt-contrib] Re: Move GAE Auth functionality from Expenses over the MobileWebApp sample. (issue1806803)

2012-08-09 Thread rchandia


http://gwt-code-reviews.appspot.com/1806803/diff/1/samples/mobilewebapp/pom.xml
File samples/mobilewebapp/pom.xml (right):

http://gwt-code-reviews.appspot.com/1806803/diff/1/samples/mobilewebapp/pom.xml#newcode183
samples/mobilewebapp/pom.xml:183: target1.6/target
Change tabs to spaces

http://gwt-code-reviews.appspot.com/1806803/diff/1/samples/mobilewebapp/pom.xml#newcode200
samples/mobilewebapp/pom.xml:200: version2.3.0-1/version
While you are at it, can we update this to a newer version?

http://gwt-code-reviews.appspot.com/1806803/diff/1/samples/mobilewebapp/src/main/java/com/google/gwt/sample/gaerequest/client/GaeAuthenticationFailureEvent.java
File
samples/mobilewebapp/src/main/java/com/google/gwt/sample/gaerequest/client/GaeAuthenticationFailureEvent.java
(right):

http://gwt-code-reviews.appspot.com/1806803/diff/1/samples/mobilewebapp/src/main/java/com/google/gwt/sample/gaerequest/client/GaeAuthenticationFailureEvent.java#newcode2
samples/mobilewebapp/src/main/java/com/google/gwt/sample/gaerequest/client/GaeAuthenticationFailureEvent.java:2:
* Copyright 2010 Google Inc.
Why the change in year?

http://gwt-code-reviews.appspot.com/1806803/diff/1/samples/mobilewebapp/src/main/java/com/google/gwt/sample/gaerequest/client/ReloadOnAuthenticationFailure.java
File
samples/mobilewebapp/src/main/java/com/google/gwt/sample/gaerequest/client/ReloadOnAuthenticationFailure.java
(right):

http://gwt-code-reviews.appspot.com/1806803/diff/1/samples/mobilewebapp/src/main/java/com/google/gwt/sample/gaerequest/client/ReloadOnAuthenticationFailure.java#newcode29
samples/mobilewebapp/src/main/java/com/google/gwt/sample/gaerequest/client/ReloadOnAuthenticationFailure.java:29:
public HandlerRegistration register(EventBus eventBus) {
Not in alphabetical order. Also, other than the order, changes in lines
26 to 32 seem to be unnecessary.

http://gwt-code-reviews.appspot.com/1806803/

--
http://groups.google.com/group/Google-Web-Toolkit-Contributors


[gwt-contrib] Re: Move GAE Auth functionality from Expenses over the MobileWebApp sample. (issue1806803)

2012-08-09 Thread drfibonacci

If you implement login and logout link in a JSP, you can remove all the
user service wrapper stuff.


http://gwt-code-reviews.appspot.com/1806803/diff/1/samples/mobilewebapp/src/main/java/com/google/gwt/sample/gaerequest/server/GaeAuthFilter.java
File
samples/mobilewebapp/src/main/java/com/google/gwt/sample/gaerequest/server/GaeAuthFilter.java
(right):

http://gwt-code-reviews.appspot.com/1806803/diff/1/samples/mobilewebapp/src/main/java/com/google/gwt/sample/gaerequest/server/GaeAuthFilter.java#newcode64
samples/mobilewebapp/src/main/java/com/google/gwt/sample/gaerequest/server/GaeAuthFilter.java:64:
private static String generateHomePageURL(String requestURL, String
queryString) {
A comment would be nice. What is this doing and why? So we don't lose
the rationale involving /gwtRequest

http://gwt-code-reviews.appspot.com/1806803/

--
http://groups.google.com/group/Google-Web-Toolkit-Contributors


[gwt-contrib] Changed package structure with the intention of keeping the client package clean of all classe... (issue1807804)

2012-08-09 Thread idol

Reviewers: Nick Chalko, cromwellian,

Description:
Changed package structure with the intention of keeping the client
package clean of all classes which the user should not see
(implementation-specific).

[JSR 303 TCK Result] 191 of 204 (93.63%) Pass with 12 Failures and 0
Errors.


Please review this at http://gwt-code-reviews.appspot.com/1807804/

Affected files:
  D user/src/com/google/gwt/validation/client/Group.java
  D user/src/com/google/gwt/validation/client/GroupChain.java
  D user/src/com/google/gwt/validation/client/GroupChainGenerator.java
  D user/src/com/google/gwt/validation/client/GroupValidator.java
  D user/src/com/google/gwt/validation/client/ValidationGroupsMetadata.java
  M  
user/src/com/google/gwt/validation/client/impl/AbstractGwtSpecificValidator.java

  M user/src/com/google/gwt/validation/client/impl/AbstractGwtValidator.java
  D user/src/com/google/gwt/validation/client/impl/BeanMetadata.java
  M user/src/com/google/gwt/validation/client/impl/ConstraintFinderImpl.java
  M  
user/src/com/google/gwt/validation/client/impl/ConstraintValidatorContextImpl.java

  A user/src/com/google/gwt/validation/client/impl/Group.java
  A user/src/com/google/gwt/validation/client/impl/GroupChain.java
  A user/src/com/google/gwt/validation/client/impl/GroupChainGenerator.java
  A user/src/com/google/gwt/validation/client/impl/GroupValidator.java
  M user/src/com/google/gwt/validation/client/impl/GwtBeanDescriptor.java
  M  
user/src/com/google/gwt/validation/client/impl/GwtBeanDescriptorImpl.java

  M user/src/com/google/gwt/validation/client/impl/GwtSpecificValidator.java
  D user/src/com/google/gwt/validation/client/impl/MessageAndPath.java
  M  
user/src/com/google/gwt/validation/client/impl/PropertyDescriptorImpl.java
  A  
user/src/com/google/gwt/validation/client/impl/metadata/BeanMetadata.java
  A  
user/src/com/google/gwt/validation/client/impl/metadata/MessageAndPath.java
  A  
user/src/com/google/gwt/validation/client/impl/metadata/ValidationGroupsMetadata.java
  M  
user/src/com/google/gwt/validation/rebind/GwtSpecificValidatorCreator.java

  M user/src/com/google/gwt/validation/rebind/ValidatorCreator.java
  M user/test/com/google/gwt/validation/ValidationClientJreSuite.java
  D  
user/test/com/google/gwt/validation/client/ValidationGroupsMetadataTest.java
  A  
user/test/com/google/gwt/validation/client/impl/metadata/ValidationGroupsMetadataTest.java



--
http://groups.google.com/group/Google-Web-Toolkit-Contributors


[gwt-contrib] Re: Changed package structure with the intention of keeping the client package clean of all classe... (issue1807804)

2012-08-09 Thread nchalko

lgtm

http://gwt-code-reviews.appspot.com/1807804/

--
http://groups.google.com/group/Google-Web-Toolkit-Contributors