When making patch for this, please don't forget checking if opacity is not set to 100 - it causes needless performance decrease (especially when paning layers with such opacity).

Tomas Novotny


Date: Thu, 15 Nov 2007 14:47:07 +0100
From: " S?bastien Roch " <[EMAIL PROTECTED]>
Subject: Re: [OpenLayers-Dev] layer opacity
To: [email protected], "Christopher Schmidt"
	<[EMAIL PROTECTED]>
Message-ID:
	<[EMAIL PROTECTED]>
Content-Type: text/plain; charset=ISO-8859-1

Yep I'll do this (ticket + maybe patch)

On Nov 15, 2007 12:49 PM, Christopher Schmidt <[EMAIL PROTECTED]> wrote:
  
On Thu, Nov 15, 2007 at 12:46:58PM +0100, S?bastien Roch wrote:
    
Hello,
I've been playing with layer opacity, and noticed something wrong if I
try to set opacity to 0 : doesn't work...
After a quick look in the code, it's simply this line in function
modifyDOMElement (Utils.js) where the issue is :
if (opacity){
....
}
0 gives "false" in a boolean context.
      
Cool. Make a ticket? With a patch, if you can; with a patch and tests if
your'e feeling extra brave :)

Regards,
--
Christopher Schmidt
MetaCarta

    

  

_______________________________________________
Dev mailing list
[email protected]
http://openlayers.org/mailman/listinfo/dev

Reply via email to