[mapguide-internals] RFC 50 - Hide Legend Styles

[Kenneth Skovhede, GEOGRAF A/S]

I have now added the enhancements as a new RFC:
http://trac.osgeo.org/mapguide/wiki/MapGuideRfc54

I put on Bruce as the PSC member, but perhaps it should be Jason?

Regards, Kenneth Skovhede, GEOGRAF A/S



Kenneth Skovhede, GEOGRAF A/S skrev:
> Yes, I can create a new RFC for further enhancements to the legend.
> Should I wait until RFC 50 has been voted on?
>
> I still think that the changes involved in adding two extra parameters 
> are very limited,
> but as I won't be doing the actual changes, I can't object to your 
> resource estimates.
> Regards, Kenneth Skovhede, GEOGRAF A/S
>   
>
>
> Bruce Dechant skrev:
>> I really like the additional suggestions that have been brought up.
>>
>> Rather than tack them on to RFC 50 could we create a new RFC to capture these?
>>
>> I am suggesting this only because of resource issues and that if all of this is added to RFC 50 it won't all get done. What are your thought on doing it this way?
>>
>> Kenneth - do you still volunteer to create the RFC? :)
>>
>> Thanks,
>> Bruce
>>
>> -----Original Message-----
>> From: mapguide-internals-bounces at lists.osgeo.org [mailto:mapguide-internals-bounces at lists.osgeo.org] On Behalf Of Kenneth Skovhede, GEOGRAF A/S
>> Sent: Wednesday, July 23, 2008 1:35 AM
>> To: MapGuide Internals Mail List
>> Subject: Re: [mapguide-internals] RFC 50 - Hide Legend Styles
>>
>> I am fairly booked. As my master thesis is due in december, I
>> won't have much time for something like this until next year.
>>
>> As I see it, the proposed changes (including Jason's), are only
>> adding properties, not functionality in the server.
>>
>> I don't think GetLegendImage should return the alternate image,
>> because in many places, the server does not have internet access.
>> Instead, the client that builds the legend should read this info as it
>> itterates over the layers, and return that image _instead_ of calling
>> GetLegendImage.
>>
>> If it is implemented as that, the code change on the server should
>> be very limited.
>>
>> There is some client code work then, but that is not required to
>> complete´the RFC, and all clients can ignore the extra info until
>> a later time.
>>
>> Regards, Kenneth Skovhede, GEOGRAF A/S
>>
>>
>>
>> Jason Birch skrev:
>>   
>>> I can hear the ADSK folks cringing at the scope creep.  Kenneth, are you
>>> able to help out with the coding on these changes?
>>>
>>> I believe that these changes would be best made in the map definition
>>> schema.  Not that there's anything wrong with modifying two schemas as
>>> part of a single RFC if the same code needs to be touched for both. :)
>>>
>>> I wonder how sensitive the server code is to schema changes in this
>>> area.  I'm thinking that the enableVisibilityChange item would likely
>>> not be all that invasive?
>>>
>>> The alternativeImage item would have all kinds in interactions with
>>> things like GetLegendImage though.  And if work is going to be done on
>>> alternative images, it would be a good to ensure that this doesn't get
>>> in the way of ticket #22 (using CSS sprites for legend image loads):
>>> http://trac.osgeo.org/mapguide/ticket/22  Fusion supports sprites in the
>>> MapServer legend now (in sandbox), so I'm hoping that MapGuide
>>> eventually goes there as well.  The number of files being loaded on map
>>> initiation is one of the core factors in perceived performance now.
>>>
>>> Maybe some of these interactions could be avoided by limiting the
>>> alternativeImage tag to map groups (not layers) for now?
>>>
>>> Jason
>>>
>>> -----Original Message-----
>>> From: Kenneth Skovhede, GEOGRAF A/S
>>> Subject: Re: [mapguide-internals] RFC 50 - Hide Legend Styles
>>>
>>> I would like this to part of RFC 50.
>>> Since the RFC is only about server/schema changes, the change proposed
>>> by Jason could be implemented with an "alternativeImage" and an
>>> "enableVisibilityChange" attribute.
>>>
>>> The change in the viewers should also be very limited.
>>> I can update the RFC with this enhancement, if no-one objects.
>>> _______________________________________________
>>> mapguide-internals mailing list
>>> mapguide-internals at lists.osgeo.org
>>> http://lists.osgeo.org/mailman/listinfo/mapguide-internals
>>>
>>>     
>> _______________________________________________
>> mapguide-internals mailing list
>> mapguide-internals at lists.osgeo.org
>> http://lists.osgeo.org/mailman/listinfo/mapguide-internals
>> _______________________________________________
>> mapguide-internals mailing list
>> mapguide-internals at lists.osgeo.org
>> http://lists.osgeo.org/mailman/listinfo/mapguide-internals
>>