Opened 13 years ago
Closed 12 years ago
#1983 closed defect (fixed)
MapTip command uses hard-coded button position
| Reported by: | jng | Owned by: | |
|---|---|---|---|
| Priority: | low | Milestone: | 2.4 |
| Component: | AJAX Viewer | Version: | |
| Severity: | trivial | Keywords: | |
| Cc: | trevorwekel | External ID: |
Description
The maptip command uses a hard-coded toolbar button position of 14 (to set the enabled/disabled state).
This is obviously bad as adding any toolbar command that displaces the maptip command from its position will cause the maptip command to "toggle" the wrong button.
The fix is to record the indices of the maptip toolbar buttons during initialization and use these indices to locate the toolbar buttons to enable/disable
Attachments (1)
Change History (4)
comment:1 by , 13 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
comment:2 by , 12 years ago
| Cc: | added |
|---|---|
| Resolution: | fixed |
| Status: | closed → reopened |
If the toolbar is not present, the call to GetMapTipButtons() will not exist. A patch to ajaxmappane.templ is attached.
To reproduce, turn off the Map Pane Toolbar in any of the Sheboygan web layouts. Before the patch, the map does not zoom to the correct extents on initial load.
by , 12 years ago
| Attachment: | getmaptip.patch added |
|---|
patch to fix web layout intiial load if no toolbar is present
comment:3 by , 12 years ago
| Resolution: | → fixed |
|---|---|
| Status: | reopened → closed |
Fixed trunk r6573 and 2.4 branch r6574