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 improvements #4922

Merged
merged 25 commits into from
Dec 2, 2020
Merged

Legend improvements #4922

merged 25 commits into from
Dec 2, 2020

Conversation

nf-s
Copy link
Contributor

@nf-s nf-s commented Oct 28, 2020

Legend improvements

  • remove legend broken image icon
  • tsxify Legend
  • Re-enable geoserver legend scaling
    • Also added geoserver legend font, font size and label margin
    • Add imageScaling trait
  • Move LegendTraits to CatalogMemberMixin
  • Comment out unused old Legend code
  • Extend WMS isGeoserver to be true if URL has geoserver in it
  • Add WMS supportsGetLegendRequest trait
  • Improved handling of WMS default styles

Fixes #4787 and #5021

A new geoserver legend:
image

Checklist

  • There are unit tests to verify my changes are correct or unit tests aren't applicable (if so, write quick reason why unit tests don't exist)
  • I've updated CHANGES.md with what I changed.

@reginapramesti reginapramesti self-assigned this Nov 2, 2020
@reginapramesti
Copy link
Contributor

@nf-s Is there a URL I can try out for this? I could only find https://services.arcgisonline.com/ArcGIS/rest/services/World_Imagery/MapServer/ but it doesn't behave the way it does on your screenshot in the issue 😬

@nf-s
Copy link
Contributor Author

nf-s commented Nov 4, 2020

mm that is interesting!
Not sure why that is happening

@nf-s
Copy link
Contributor Author

nf-s commented Nov 4, 2020

@nf-s nf-s changed the title Remove legend broken image icon Legend improvements Nov 4, 2020
@nf-s nf-s marked this pull request as ready for review November 5, 2020 05:38
@nf-s
Copy link
Contributor Author

nf-s commented Nov 5, 2020

Sorry @reginapramesti this has turned into a bit of a monster PR 😶

@nf-s nf-s marked this pull request as ready for review November 30, 2020 12:03
@nf-s nf-s marked this pull request as draft December 1, 2020 06:28
@nf-s nf-s marked this pull request as ready for review December 2, 2020 01:11
@KeyboardSounds KeyboardSounds self-assigned this Dec 2, 2020
lib/Traits/LegendTraits.ts Outdated Show resolved Hide resolved
lib/Map/Legend.js Show resolved Hide resolved
lib/Map/LegendUrl.js Show resolved Hide resolved
lib/Traits/CatalogMemberTraits.ts Show resolved Hide resolved
lib/Models/WebMapServiceCatalogItem.ts Outdated Show resolved Hide resolved
lib/ReactViews/Workbench/Controls/Legend.tsx Show resolved Hide resolved
@KeyboardSounds
Copy link
Contributor

👍 for adding useful comments to WebMapServiceCatalogItem

@nf-s nf-s merged commit c85ba14 into next Dec 2, 2020
@nf-s nf-s deleted the remove-legend-broken-image-icon branch December 2, 2020 06:13
@steve9164 steve9164 mentioned this pull request Dec 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants