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

Search add location icon not working with Internet Explorer #1165

Closed
dpalomino opened this issue Feb 28, 2017 · 10 comments · Fixed by #1307
Closed

Search add location icon not working with Internet Explorer #1165

dpalomino opened this issue Feb 28, 2017 · 10 comments · Fixed by #1307

Comments

@dpalomino
Copy link

dpalomino commented Feb 28, 2017

Steps to reproduce the error

  1. Open Internet Explorer

  2. Go to a project and start adding a new location

  3. Use the search icon included in the drawing tools when adding the location
    screen shot 2017-02-28 at 11 09 40

  4. Search icon is unresponsive

Tested with Windows 10 and IE 11 (latest version), and Windows 7 and IE 11 (latest version).

@seav
Copy link
Contributor

seav commented Mar 1, 2017

@jack17529
Copy link

@dpalomino assign me this.

@wonderchook
Copy link
Contributor

@jack17529 during the application process for GSoC we don't assign issues to anyone. It is okay to have more than one pull request for the same issue, we will still consider it as part of your application.

@wonderchook
Copy link
Contributor

So the examples provided by Mapzen work with IE, so something to keep in mind.

https://mapzen.github.io/leaflet-geocoder/examples/index.html
https://github.com/mapzen/leaflet-geocoder/tree/master/examples

@jack17529
Copy link

@wonderchook , please help when I run ./runserver I am getting this error -
Traceback (most recent call last):
File "/home/jack/Desktop/cadasta-platform-master/cadasta/manage.py", line 10, in
execute_from_command_line(sys.argv)
File "/usr/local/lib/python2.7/dist-packages/django/core/management/init.py", line 367, in execute_from_command_line
utility.execute()
File "/usr/local/lib/python2.7/dist-packages/django/core/management/init.py", line 316, in execute
settings.INSTALLED_APPS
File "/usr/local/lib/python2.7/dist-packages/django/conf/init.py", line 53, in getattr
self._setup(name)
File "/usr/local/lib/python2.7/dist-packages/django/conf/init.py", line 41, in _setup
self._wrapped = Settings(settings_module)
File "/usr/local/lib/python2.7/dist-packages/django/conf/init.py", line 97, in init
mod = importlib.import_module(self.SETTINGS_MODULE)
File "/usr/lib/python2.7/importlib/init.py", line 37, in import_module
import(name)
File "/home/jack/Desktop/cadasta-platform-master/cadasta/config/settings/dev.py", line 1, in
from .default import * # NOQA
File "/home/jack/Desktop/cadasta-platform-master/cadasta/config/settings/default.py", line 17, in
from .languages import FORM_LANGS # noqa
File "/home/jack/Desktop/cadasta-platform-master/cadasta/config/settings/languages.py", line 54
SyntaxError: Non-ASCII character '\xc3' in file /home/jack/Desktop/cadasta-platform-master/cadasta/config/settings/languages.py on line 54, but no encoding declared; see http://python.org/dev/peps/pep-0263/ for details

@wonderchook
Copy link
Contributor

@jack17529 not the right place to ask this question. gitter is the right place to ask it.

@clash99
Copy link
Contributor

clash99 commented Mar 15, 2017

While testing on browser stack - windows 10, internet explorer 11, leaflet-touch styles are overriding other styles for leaflet-pelias-expanded and other styles.

screenshot 2017-03-15 16 32 03

Leaflet/Leaflet#3944

@jack17529
Copy link

@clash99 can you please help me solve my issue?

@clash99
Copy link
Contributor

clash99 commented Mar 15, 2017

@jack17529 - Have you tried looking here? https://gitter.im/Cadasta/cadasta

@clash99
Copy link
Contributor

clash99 commented Mar 16, 2017

Some screenshots of fixed search from browserstack:

screenshot 2017-03-15 20 58 17

screenshot 2017-03-15 21 06 59

screenshot 2017-03-15 21 07 38

screenshot 2017-03-15 21 08 24

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants