pan/zoom panels

See also r6393, http://dev.openlayers.org/sandbox/crschmidt/demo/examples/pan-zoom-panels.html

more control

This is a nice enhancement Chris. Thanks for the work. The updated patch works against r7312. (I assume you'll also add the relevant theme/default/img/*.png.)

My only other comment is that I like the names OpenLayers.Control.ZoomIn etc better than the ZoomInButton equivalents - since someone could just as well be triggering these controls with an anchor or an image or any other element or some random event. (Though I make an exception for the *Panel classes.) With or without that change, I think this is a great addition.

Please commit.

some things i notice about this patch:

1) @requires OpenLayers/Control/Control.js

(should be OpenLayers/Control.js, no?)

2) Engrish: "The type of OpenLayers.Control -- When are added to a Control.Panel the panel uses this to determine how to handle our events"

3) Constants

OpenLayers.Control.PanButton.NORTH = "NORTH";  
OpenLayers.Control.PanButton.SOUTH = "SOUTH";  
OpenLayers.Control.PanButton.EAST = "EAST";  
OpenLayers.Control.PanButton.WEST = "WEST";  

... what if you set them instead as this:

OpenLayers.Control.PanButton.NORTH = "North";  
OpenLayers.Control.PanButton.SOUTH = "South";  
OpenLayers.Control.PanButton.EAST = "East";  
OpenLayers.Control.PanButton.WEST = "West";  

then this:

     switch(this.direction){  
             case OpenLayers.Control.PanButton.NORTH:  
                 this.CLASS_NAME += "North";  
                 break;  
             case OpenLayers.Control.PanButton.SOUTH:  
                 this.CLASS_NAME += "South";  
                 break;  
             case OpenLayers.Control.PanButton.EAST:  
                 this.CLASS_NAME += "East";  
                 break;  
             case OpenLayers.Control.PanButton.WEST:  
                 this.CLASS_NAME += "West";  
                 break;  
         }  

... could become this:

         this.CLASS_NAME += this.direction;  

4) tabs

... see forthcoming patch

4 updates as described in comment

couple more things

1) no need to override draw() in ZoomPanel

2) this code doesn't seem to ever get called... is the idea that users can override the class name? i can't see why anyone would want to do that. for now i have pulled these lines out. there well may be something i'm blatantly overlooking, feel free to fill me in.

        if(options && options.displayClass){
			this.displayClass = options.displayClass;
        }

see forthcoming patch...

last few changes to proposed patch

of note, these png's don't look good in ie6. sugerencias?

also of note, should we maybe consider doing the sprite trick like for the history navigation control?

Uses css sprites, only tested on FF3

pan images

zoom images

Nice changes folks. I'm attaching one more patch. This one changes some doc comments, gives the example some style, and changes names like ZoomInButton to ZoomIn. I can talk more about why I think this is a better name if anybody is interested. If you don't like these names, please consider the other (minor) changes to the example and comments.

zoom and pan panls

some small changes to ND for Pan file

I've added yet a few more small mods to tschaub's patch, just some changes to the ND in for the Pan.js file's Pan Constructor.

I would say this is good to go, all but for the fact that I load it in ie6 and the alpha bits on the controls don't show up correctly.

Can anyone take a look at this?

no alpha channel

no alpha channel

I can't look at it in IE, but there are some images w/o alpha transparency.

Now, how about a nice little CSS hack for IE6 so these ugly images only show up there?

<!--[if lte IE 6]>
    do something
<![endif]-->

and, in case it is not clear, "do something" above would be a link to an ie6 only stylesheet

see #1607