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

Added credits alternative for cesium baseLayers. See #1780 #1785

Merged
merged 5 commits into from
May 5, 2017

Conversation

offtherailz
Copy link
Member

Removes wrong html from attribution in cesium.

This pull request belongs to #1780
From this:
image

to this:
image

@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 78.931% when pulling 0601cfb on offtherailz:cesium_credits into 3beb80c on geosolutions-it:master.

@offtherailz offtherailz requested a review from mbarto May 5, 2017 07:29
@@ -75,13 +75,15 @@ Layers.registerType('tileprovider', (options) => {
const isCORS = useCORS.reduce((found, current) => found || url.indexOf(current) === 0, false);
proxy = !isCORS && proxyUrl;
}
const cr = opt.credits;
let credit = cr ? new Cesium.Credit(cr.text, cr.imageUrl, cr.link) : opt.attribution;
return new Cesium.UrlTemplateImageryProvider({
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

use const if credit does not change anymore

@@ -0,0 +1,31 @@
/**
* Copyright 2017, GeoSolutions Sas.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove extra *

@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 78.941% when pulling b78d1c5 on offtherailz:cesium_credits into 3beb80c on geosolutions-it:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 78.941% when pulling 792c051 on offtherailz:cesium_credits into 3beb80c on geosolutions-it:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 78.941% when pulling 5a26dc4 on offtherailz:cesium_credits into 3beb80c on geosolutions-it:master.

@offtherailz offtherailz merged commit e3c46fc into geosolutions-it:master May 5, 2017
@offtherailz offtherailz deleted the cesium_credits branch June 21, 2018 07:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants