On Tue, Sep 9, 2008 at 3:54 PM, Eric Ayers <[EMAIL PROTECTED]> wrote:

> Hello Miguel,
>
> I've combined the conversion of several classes into one patch because of
> the overlap.
>
>  - PolylineOptions
>  - PolygonOptions
>  - Fixed constructor name in PolyStyleOptions.java
>  - Fixed references to above.
>  - Changed some references to PolylineOptions to EncodedPolyline... that
> was weird!
>
> M      maps/maps/test/com/google/gwt/maps/client/overlay/PolygonTest.java
>
LG


>
> M      maps/maps/src/com/google/gwt/maps/client/overlay/PolygonOptions.java
>
LG - consider adding a newInstance(boolean clickable) method.


>
> A
> maps/maps/src/com/google/gwt/maps/client/overlay/PolylineOptions.java
>
LG - This is one place where we could provide an additional newInstance
overload to specify clickable and geodesic.


>
> M
> maps/maps/src/com/google/gwt/maps/client/overlay/EncodedPolyline.java
>
LG


>
> M
> maps/maps/src/com/google/gwt/maps/client/overlay/PolyStyleOptions.java
>
LG


> M      maps/maps/src/com/google/gwt/maps/client/overlay/Polygon.java
>
LG


> M      maps/maps/src/com/google/gwt/maps/client/overlay/Polyline.java
>
LG


> D
> maps/maps/src/com/google/gwt/maps/client/impl/PolyStyleOptionsImpl.java
>
LG


> M
> maps/maps/src/com/google/gwt/maps/client/impl/PolylineFactoryImpl.java
>
LG - Is this an overlay conversion candidate?

D
> maps/maps/src/com/google/gwt/maps/client/impl/PolygonOptionsImpl.java
>
LG


> D
> maps/maps/src/com/google/gwt/maps/client/impl/PolylineOptionsImpl.java
>
LG


> M
> maps/samples/hellomaps/src/com/google/gwt/maps/sample/hellomaps/client/DrawingOverlayDemo.java
>
LG

M
> maps/samples/hellomaps/src/com/google/gwt/maps/sample/hellomaps/client/OverlayDemo.java
>
LG


>
>
> --
> Eric Z. Ayers - GWT Team - Atlanta, GA USA
> http://code.google.com/webtoolkit/
>



-- 
Miguel

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

Reply via email to