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

Wrap x feature for pull request #198

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

Wrap x feature for pull request #198

wants to merge 7 commits into from

Conversation

dulcedilip
Copy link

attribute added to the olHelper service for all source types

oSource = new ol.source.OSM({
                attributions: createAttribution(source),
                wrapX: (source.wrapX === undefine) ? 1 : source.wrapX // added for single map tile
            });

By default it take multiple maps for tiles.
usuage :

source: {
            type: "OSM",
            url:"",
            wrapX: true // wrapx: false  or remove attribute it will take default
        }

@juristr
Copy link
Collaborator

juristr commented Nov 9, 2015

@dulcedilip Awesome, thx for your contribution. Would you mind squashing your changes into a single commit that follows our contribution guidelines?

Let me know if you need help with that. Once done, I'm happy to merge it in!

@dulcedilip
Copy link
Author

Thank you @ juristr. I created two branches one for my usuage in my project with dist files and another for pullrequest with your standarads. You can consider wrapxforpullfequest and ignore the other.

@juristr
Copy link
Collaborator

juristr commented Nov 20, 2015

@dulcedilip Ok, I see. could you submit that branch as another PR? Would that be possible? thx 😄

@dulcedilip dulcedilip changed the title Wrap x feature for pull request Wrap x feature for pull request Nov 20, 2015
@dulcedilip
Copy link
Author

For reference : Wrap x feature for pull request #198
Still this is open you can take this as pull request.
i am unable to give you pullrequest

@StevenHeinrich StevenHeinrich mentioned this pull request Sep 8, 2016
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.

2 participants