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

Fix SVG rendering issues #1207

Open
wants to merge 56 commits into
base: main
Choose a base branch
from

Conversation

dstenger
Copy link
Contributor

@dstenger dstenger commented Oct 5, 2021

This includes:

  • Processing of SVG in external graphics was moved from parser to renderer.
  • deegree.rendering.graphics.size.strict=true parameter was introduced making it possible to still use aspect ratio of the SVG.

This pull request bases on following pull request which shall be merged first.

@dstenger dstenger added the bug error issue and bug (fix) label Oct 5, 2021
@tfr42 tfr42 added the WMS deegree Web Map Service label Oct 25, 2021
lgoltz and others added 15 commits December 15, 2021 08:04
…cement/migrations-styling-enhacments-dee-83-7692
…of github.com:lat-lon/deegree3 into enhancement/migrations-styling-enhacments-dee-83-7692
…nhacments-dee-83-7692' into bugfix/rendering-fixes-dee-274-200-7692
…00-7692' into bugfix/dee-364-svg-rendering-289-7692
…-138-196-7692' into bugfix/dee-364-svg-rendering-289-7692
…nhacments-dee-83-7692' into bugfix/rendering-fixes-dee-274-200-7692
…00-7692' into bugfix/dee-364-svg-rendering-289-7692
@tfr42 tfr42 added the CI failing CI build job fails label Feb 4, 2022
…nhacments-dee-83-7692' into bugfix/rendering-fixes-dee-274-200-7692
…00-7692' into bugfix/dee-364-svg-rendering-289-7692
…nhacments-dee-83-7692' into bugfix/rendering-fixes-dee-274-200-7692
…00-7692' into bugfix/dee-364-svg-rendering-289-7692
…nhacments-dee-83-7692' into bugfix/rendering-fixes-dee-274-200-7692
…00-7692' into bugfix/dee-364-svg-rendering-289-7692
…nhacments-dee-83-7692' into bugfix/rendering-fixes-dee-274-200-7692
…00-7692' into bugfix/dee-364-svg-rendering-289-7692
@tfr42 tfr42 added the needs rebase PR is not up to date and needs rebase label May 16, 2022
@tfr42
Copy link
Member

tfr42 commented Nov 9, 2022

Relates to #1405

@tfr42 tfr42 added the invalid invalid label Nov 9, 2022
@dstenger
Copy link
Contributor Author

Decision during TMC meeting 2023-08-23:

This pull request contains content which might still be relevant. However, the code base of the pull request is outdated and thus a complete re-work is necessary. The pull request shall be replaced by an issue describing the requirements for the re-worked pull request. After the issue will have been created, this PR can be closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug error issue and bug (fix) CI failing CI build job fails invalid invalid needs rebase PR is not up to date and needs rebase WMS deegree Web Map Service
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants