I added some code to have a callback from GWT to javascript, to make sure the GWT visualization is loaded before using it.
On 2008/11/10 13:56:48, zundel wrote: > LGTM. > > I am not all that taken with the name 'MyVisualization', I would prefer > something like "CustomVisualization" instead. No matter what we decide on that, > we need to add the eclipse setup config and launch scripts. Lets discuss for a > followon patch. > I renamed everything from "My" to "Custom" > http://galgwt-reviews.appspot.com/202/diff/1/5 > File > samples/myvisualization/src/com/google/gwt/visualization/sample/myvisualization/client/MyVisualization.java > (right): > > http://galgwt-reviews.appspot.com/202/diff/1/5#newcode65 > Line 65: grid.getElement().setAttribute("style", "background-color:" + > options.getBackgroundColor()); > Using the getStyle() method, you can set DOM properties one at a time. > grid.getElement().getStyle().setProperty("backgroundColor", > options.getBackgroundColor()); done. http://galgwt-reviews.appspot.com/202 --~--~---------~--~----~------------~-------~--~----~ http://groups.google.com/group/Google-Web-Toolkit-Contributors -~----------~----~----~----~------~----~------~--~---
