Bug 155 - [cwc2 1.0.4] Keymap click event not working properly
: [cwc2 1.0.4] Keymap click event not working properly
Status: CLOSED FIXED
: Chameleon
Widget
: 1.0
: PC Windows 2000
: P1 normal
: ---
Assigned To:
:
:
:
:
:
  Show dependency treegraph
 
Reported: 2003-12-04 14:37 by
Modified: 2004-06-24 15:42 (History)


Attachments


Note

You need to log in before you can comment on or make changes to this bug.


Description From 2003-12-04 14:37:04
Charlton Purvis reported this problem to me and I was able to reproduce it 
easily on the IOOS prototype at: http://mapsherpa.com/msapps/ioos/
I've marked this as P1 because this prototype is fairly high profile and will 
be demo-ed to death over the next while and it should work flawlessly.

Here are the steps to reproduce:

* Start with the full extent view.
* Zoom to a region.
* Correct image shows up, and the key map is correctly updated.
* Now click on the key map to move the zoom box to a different area.
* The result is not what you expect.  You have zoomed in further w/i
your original region.
------- Comment #1 From 2003-12-09 21:43:48 -------
reproduced.  Obviously not good.  investigating ...
------- Comment #2 From 2003-12-09 21:48:25 -------
interestingly (that's two interesting bugs in one day!), this works correctly in
my local installation using the demo.html ... investigating further ...
------- Comment #3 From 2003-12-09 22:05:39 -------
The ioos.html template has some problems:

line 56 at the end is a tag: <cwc2 name="CWCJSAPI">, this is invalid and causes
some problems

lines 60 - 79 aren't closed properly, the only valid way (in this version) is
with /> ... the </tagname> style isn't valid (because its not actually parsed as
XML, I've fixed that in version 1.1/2.0)

line 83 ... you start a shared resource, but the previous shared resource
(starting on line 59) isn't closed with a </cwc2> ... this makes the wait image
effectively disappear.

<waitimage tags have the same problem :(

there's a bunch of extra *stuff* on line 96, including two </cwc2> next to each
other.

there is a CWCJSAPI widget but no CWCJSAPI shared resource.

The form start tag is actually inside a cwc2 tag???

This is making me think that something happened to the template when I
downloaded it using Mozilla???  Is the template actually this wrong?

In any case, after fixing the template to work locally, I have isolated this to
a problem with the keymap operating in JSAPI mode.  Looking for actual problem
and fix ...
------- Comment #4 From 2003-12-09 22:08:43 -------
just to answer the above question to myself, the template is in fact okay.  I
downloaded it by entering the URL for the template into mozilla and then saving
the page ... obviously Mozilla does a little 'tweaking' for you :S
------- Comment #5 From 2003-12-09 22:22:33 -------
There is a (couple of) typos in the CWCJSAPI-specific code for the KeyMap widget
that cause this problem.  Unfortunately, fixing them did not make it work
correctly.  The map no longer zooms in when you click the keymap, but now it
doesn't recenter correctly (i.e. at all).  The various elements of the page all
get updated, but nothing moves.  I expect there is something simple to fix here,
but I can't fix it tonight, will finish it tomorrow.
------- Comment #6 From 2003-12-10 10:04:26 -------
There is/was an extensive problem in the keymap widget related to jsapi mode
that never surfaced because we test the same keymap settings in the widget as is
defined in the template.map, so it never became obvious that the keymap was not
working correctly in jsapi mode.  I've made some substantial changes to how the
keymap works so that it will work in jsapi mode (and non-jsapi mode too)

The changes have been committed to cvs and installed on xcalibur in a new cwc2
snapshot release that will become 1.0.5 after testing and a couple of other bug
fixes (like the time filter thing).

Dean, you can test at:

http://www2.dmsolutions.ca/msapps/cwc2-snap-20031210/cwc2.php?template=http://www2.dmsolutions.ca/msapps/ioos/ioos.html&context=http://www2.dmsolutions.ca/msapps/ioos/ioos_prototype.xml
------- Comment #7 From 2003-12-10 10:23:45 -------
Works for me. I've sent the URL to Charlton to test since he originally 
reported the bug.
------- Comment #8 From 2003-12-10 10:38:19 -------
Ok'd by Charlton too.
------- Comment #9 From 2004-06-24 15:42:37 -------
closing bug, since verified.