Request for change the status bar to reflect the layer selected

[bscott]

Actually, the status bar in the ajax viewer only shows the amount of features selected. It would be great if it showed the name of the selected layer also. If there is many features selected on different layers, the behavior stays the same as actually.

nabble thread: ​http://www.nabble.com/Request-for-change-the-status-bar-to-reflect-the-layer-selected-td21051252.html

[tomfukushima]

Nice! I'm going to show this to a product designer for review.

[tomfukushima]

Sorry, I meant the above comment for ticket #800. This change looks good, I'll try to get it in.

[tomfukushima]

I tested this out and would like to request the following changes before I submit.

1. Check to make sure selLayers is not null before using it (that is, change line "if(selLayers.length ==1)" to "if(selLayers!=null && selLayers.length...".
2. Put up a tooltip when the user floats the mouse over the area that displays the selection count. This is required now for the cases of long layer names which will make it so that some of the text is not visible. I realize that you could select the text area and scroll the text using the mouse, but this is not user friendly. Alternatively, if we could resize the status bar areas, that might be another solution. Of course, there might be another way to solve this too.

Thanks, Tom

[bscott]

Hi Tom, I added the test fir selLayers != null and i added tooltips for it. I have also added tooltips for the first items in the statusbar ( the x,y position), it does not hurt :)

And as you suggested i've made the with a little bigger ( 300 instead of 200 pixels)

[tomfukushima]

It's strange, but after applying this patch nothing shows up in the "selected features" section of the status bar. It is blank. To verify that the patch was applied correctly, I created a patch from my source after applying yours and found that the only differences in the patch were the revisions of the files that the patches were applied against. For en I was at revision 3523, while you were at 3498. For mainframe.templ, me: 3523, you: 3516. For statusbar.templ, me: 3523, you: 3516. Can you see anything that would cause the problem that I'm seeing? BTW, the tooltip comes up for the (x,y position).

[bscott]

I Tom, I have notice something on my pc also. When we add new stuff in the localized "en" file, the modification is not applied directly. I think there is some caching somewhere that cause this. I've tried plenty of thing:

Restart mapguide server : did not work Restart IIS : did not work Reboot the pc : this one worked, ouf!

Bruno

[tomfukushima]

Thanks. Actually, it was because I forgot to replace the en file on my test machine. Sorry, my mistake. It works for me now.

[waltweltonlair]

Fixed in the trunk stream with submission ​https://trac.osgeo.org/mapguide/changeset/3734.

Thanks bscott for providing this patch.