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

Legend image wasn't shown after it was hidden #17

Closed
wants to merge 1 commit into from

Conversation

vmx
Copy link
Member

@vmx vmx commented May 16, 2012

No description provided.

@peppelinux
Copy link

I patched this issue in a quick way.

The problem is that when the layer beign hidden mqlayermanager add a css attribute: visibility->hidden.
When the user increase the opacity of the layer, mplayermanager instead of remove visibility attribute leave it and append a new opacity attribute.

I abandoned the visibility approach and I patched in jquery.mapquery.mqLayerManager.js, at line 260:

    //update legend image
    layerElement.find('.mq-layermanager-element-legend img').css(
        {opacity:layer.visible()?true:0});
},

@vmx
Copy link
Member Author

vmx commented May 20, 2012

I had another look. I don't fully understand what you mean.

When you deselect the checkbox, the layer and the legend symbol is hidden. To make the legend symbol and the map visible again, you have two ways. Either select the checkbox again, or drag the slider.

Is it different for you (I've only tried it in Firefox).

@peppelinux
Copy link

I use google-chrome


Da: Volker Mische reply@reply.github.com
A: peppelinux peppelinux@yahoo.it
Inviato: Domenica 20 Maggio 2012 19:22
Oggetto: Re: [mapquery] Legend image wasn't shown after it was hidden (#17)

I had another look. I don't fully understand what you mean.

When you deselect the checkbox, the layer and the legend symbol is hidden. To make the legend symbol and the map visible again, you have two ways. Either select the checkbox again, or drag the slider.

Is it different for you (I've only tried it in Firefox).


Reply to this email directly or view it on GitHub:
#17 (comment)

@stvno stvno closed this Aug 3, 2012
@stvno
Copy link
Member

stvno commented Aug 3, 2012

I meant to write a comment on the closing: the issue is twofold, there is the _layerVisible function and the _layerOpacity function. The pull request only solves the first issue, the aeaead4 commit fixes both

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

Successfully merging this pull request may close these issues.

None yet

3 participants