On Thu, 2009-07-02 at 16:29 +0200, Andrea Aime wrote: > Hi, > following http://jira.codehaus.org/browse/GEOS-3185 > I would like to draw a plan to revert back KML to refresh > mode and have people ask explicitly that they do want superoverlays. > > The first step would be to have the kml reflector to default > to refresh mode as opposed to super-overlay mode (and adjust > the tests accordingly). > > The second would be to make the superoverlay mode visible anyways, > somehow. On 1.7.x I guess we can add it as a further link in the > preview? > http://geo.openplans.org:8080/geoserver/mapPreview.do > Imagine a "KML (superovelay)" link among the others. There's > enough space to just add it no? > > Things are different on trunk, the preview page looks like > attached and it's already quite busy. > I would like to add the superoverlay into the the "all formats" > drop down box as KML (superoverlay), however, > this requires the superoverlay to be turned into a valid WMS > output format, as the list is populated by scanning the map producers. > > I guess it's doable if we create a subclass of KMLMapProducer that > does use the mime type: > > application/vnd.google-earth.kml+xml; mode=superoverlay > > (longest mime ever) and that forces is the superoverlay mode. > > Otherwise I can decouple somehow the map producers and the > drop down UI generation, with every producer having one or > more descriptor bean that can be used to build a UI entry. > > Opinions? > Cheers > Andrea
My opinion is that, while I'd like to see these options decoupled (just for aesthetic reasons) it would be one more thing for developers of WMS format extensions to remember, and which wouldn't break noisily if they forgot it. So, of the options provided, +1 on adding the custom format, with a little knot in my stomach while I vote. A third option might be implementing the extension point for the UI, but automatically pulling in all formats with no extra options. This would ensure all formats show up at least once, and if there are particular combinations of options that are interesting developers are free to highlight them. It seems like a lot of work for just this one link, however. Also, unrelated to the original question: the AtomPub entry in the formats list should be Atom (AtomPub is a protocol, Atom is a format). -- David Winslow OpenGeo - http://opengeo.org/ ------------------------------------------------------------------------------ _______________________________________________ Geoserver-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/geoserver-devel
