Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change removed navigationControl to zoomControl and panControl. #257

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

cyberhobo
Copy link
Contributor

This fixes issue #253, and should repair the broken methods addControls(), addLargeControls(), and addSmallControls(), as well as the global loadoptions.addControlArgs initialization variable for zoom and pan controls.

In tests/core.html, action 12 "Add Controls" should work again.

@icebergdelphi
Copy link

Hi cyberhobo,I have implemented your new code, and works great, forget my last comment.Thanks.

I believe this is what the former navigationControl option did.
@mlandauer
Copy link

This fix works for me. 👍 to merging it

henare pushed a commit to openaustralia/planningalerts that referenced this pull request Jan 13, 2014
* master:
  Upgrade to ruby-2.0.0-p247
  Add test for html email for authority comment notification. Include test for multiple paragraphs
  Update newrelic agent for extra features
  Use maps client id if it is available for maps api use
  Move list of assets to precompile over to application.rb so it is available even when production.rb is overridden
  Actually make it possible to use maps api v3 in production
  Use google maps api v3 (via mapstraction) and add satelite option to map
  Applied fix for googlev3 controls not showing from mapstraction/mxn#257
  Updated to Mapstraction v 2.0.18 (and added googlev3 support)
  don't need our own backported geokit anymore. Can use a release version

Conflicts:
	config/environments/production.rb
JWells pushed a commit to JWells/planningalerts-app that referenced this pull request Feb 22, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants