Opened 16 years ago

Closed 12 years ago

#469 closed defect (fixed)

Fusion Tiled Maps Issues

Reported by: jbirch Owned by:
Priority: medium Milestone:
Component: Fusion Version: 2.1.0
Severity: major Keywords: fusion, tiled
Cc: pagameba, madair External ID: 1054718, 1296474.01

Description (last modified by jbirch)

Tiled maps in Fusion do not appear to be fully supported yet.

To test these, modify Aqua's ApplicationDefinition.xml, changing to the map to MapsTiled\Sheyboygan, and setting SingleTile to false. All of these were tested in FireFox.

  1. Zoom slider does not appear to initialize. Stays on closest zoom in.
  1. Selection graphic is offset up and to the right by several pixels (Firefox 2)
  1. When panning and zooming, sometimes I'll get an error in MapTip.js at line 203, erroring out because this.oMapTipPosition.y is null. I'm not sure what's happening here; whether I'm placing the mouse over a div that's floating over the map, or if it's trying to query while the map div isn't active?
  1. The Legend panel does not "work" with tiled layers properly. It will need some kind of special treatment to only display a check-box at the parent level, etc.
  1. Specifying a tile size smaller than 300x300 in the MapGuide server config makes small outlines appear around the images. Perhaps the OL code is hard-coded to 300x300?
  1. For some reason, on the Demo server I'm getting 401 responses for some tiles, followed by 200 responses. Huh?

http://mapguidebeta.osgeo.org/mapguide/fusion/templates/mapguide/slate_tiled/index.html

The performance of tiled maps in Fusion is pretty decent though. I'd love to be able to use this stuff!

Attachments (1)

TiledMapSupport.patch (17.5 KB ) - added by chrisclaydon 14 years ago.

Download all attachments as: .zip

Change History (11)

comment:1 by jbirch, 16 years ago

Cc: pspencer madair added
Description: modified (diff)

Added number 4 and 5 to the list.

comment:2 by jbirch, 16 years ago

Cc: pagameba added; pspencer removed

comment:3 by jbirch, 16 years ago

Description: modified (diff)

Added authentication issue.

comment:4 by chrisclaydon, 16 years ago

External ID: 1054718

comment:5 by zspitzer, 16 years ago

In regards to item 5, the mapguide server config isn't exposed to the tile client

It would be nice to have a GETILECONFIG which returns that and maybe the calculated tile grid, that also would reduce the amount of client knowledge

something along the lines of PROJECTION: EPSG:4236 BOUNDS: ??? FORMAT: PNG8 TILESIZE : 300x300 SO 4x4 3000000 S1 16x16 12000000 S2 32x32 88000000

comment:6 by pagameba, 16 years ago

On the issue of tile size, it is set by default to 300 x 300 in OpenLayers.Layer.MapGuide but I believe it could be overridden with an option in the Extension of the Map tag in the ApplicationDefinition. Not now, of course, but we could add that as an option :)

by chrisclaydon, 14 years ago

Attachment: TiledMapSupport.patch added

comment:7 by chrisclaydon, 14 years ago

Please review the attached patch, which adds support to the trunk code stream for tiled MG maps. In a nutshell, we add an additional OpenLayers.Layer.MapGuide layer to the OL map that is responsible for rendering the dynamic overlay layer.

Updates have been made to Legend.js to allow it to display checkboxes appropriately for static/dynamic layers, and to LoadMap.php, MapGuide.js, and Layers.js to provide it with the information necessary to do this.

The 'zoom slider' widget, Navigator.js has been updated to address an issue with the zoom indices.

Selection works in versions of MapGuide built with the patch included in ticket:

http://trac.osgeo.org/mapguide/ticket/1241

(Or versions created before the bug was introduced).

comment:8 by chrisclaydon, 14 years ago

External ID: 10547181054718, 1296474.01

The following submission fixes most of these issues in the trunk code stream, except for the non-default tile size issue:

http://trac.osgeo.org/fusion/changeset/2041

comment:9 by jbirch, 14 years ago

Version: 2.0.02.1.0

Chris, I'd be OK with you closing this; I can add an enhancement ticket for support of non-default tile sizes. Unless you're tracking this as part of your internal tickets as well.

comment:10 by jng, 12 years ago

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.