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

Map disappears #1730

Closed
pblain opened this issue Apr 21, 2015 · 9 comments
Closed

Map disappears #1730

pblain opened this issue Apr 21, 2015 · 9 comments
Labels

Comments

@pblain
Copy link
Contributor

pblain commented Apr 21, 2015

Chrome: Version 41.0.2272.101

Steps to reproduce:

  1. Step 1: Add a data collection
  2. Step 2: Create a subset (anything - doesn't matter)
  3. Step 3: Download the subset (or something else). The important thing is that the download bar appears at the bottom of chrome (showing the file name)
  4. Go back to Step 2
  5. It may also be necessary to click the X at the far right of the download bar (to close the bar)

What should happen:

  1. Not much.

What does happen:

  1. The map disappears

image

@anguss00
Copy link
Contributor

Also present in version 42.0.2311.90, but need to close the download bar

@dnahodil dnahodil added the bug label Apr 23, 2015
@danfruehauf
Copy link
Contributor

Maybe we should update OpenLayers?

@anguss00
Copy link
Contributor

Could be a good idea. I've investigated this issue and it is a bug between OpenLayers and google chrome. See openlayers/openlayers#1181 and openlayers/openlayers#1207. The issue is caused (I think) by the window resizing that occurs when the download bar is removed in Chrome. The workaround suggested is adding the following to CSS:

svg {
-webkit-transform: none !important;
}
.olTileImage {
-webkit-backface-visibility: visible !important;
-webkit-transform: none !important;
}

@danfruehauf
Copy link
Contributor

As much as this will probably fix it, I think that can also be a good reason to investigate an upgrade of OpenLayers. We're a major version behind after all.

@anguss00
Copy link
Contributor

Yep, I'd agree with that for sure. I'd prefer to update rather than introduce a hacky fix.

@danfruehauf
Copy link
Contributor

Lets just note that this may actually be "more than a bug" in our definition. Can get quite involved.

@anguss00
Copy link
Contributor

Yeah, agreed. If we want to fix this properly it's more than a bug

@anguss00
Copy link
Contributor

Ok, @dnahodil and I have looked at this further (including the upgrade). The upgrade process is more involved than a simple replace (and there is virtually no documentation available). Notably, in the comments in openlayers/openlayers#1181 users who are on 2.13.1 are still reporting the issue (although some users have had success by forcing refreshes on tile resize). The upshot is, we would have to upgrade to 2.13.1 (as the refresh fix does not work with 2.10) and then apply the refresh fix, hoping that it then works. What do you think @pblain? I think that this is going to get a lot bigger than a bug fix at the moment.

@danfruehauf
Copy link
Contributor

Closed by #1754

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

No branches or pull requests

4 participants