Skip to content

#9254 Add parametric image parsing to template viewer#9260

Merged
offtherailz merged 13 commits into
geosolutions-it:masterfrom
MV88:9254_template_image
Jul 5, 2023
Merged

#9254 Add parametric image parsing to template viewer#9260
offtherailz merged 13 commits into
geosolutions-it:masterfrom
MV88:9254_template_image

Conversation

@MV88

@MV88 MV88 commented Jun 28, 2023

Copy link
Copy Markdown
Contributor

Description

Added possibility to insert image url in markdown

image

Please check if the PR fulfills these requirements

What kind of change does this PR introduce? (check one with "x", remove the others)

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Other... Please describe:

Issue

What is the current behavior?

#9254

What is the new behavior?

  • You can now use markdown syntax in the template editor
  • there is a specific button that wraps the text selected into image syntax
  • removed react-quill from FeatureInfoEditor in favor of react-draft-wysiwyg
  • fixed also style for description editor in map views since that style was creating issues and were not needed,
  • maybe we can discuss if we want the min-height to 200 in feature info editor or not

Breaking change

Does this PR introduce a breaking change? (check one with "x", remove the other)

  • Yes, and I documented them in migration notes
  • No

Other useful information

@tdipisa if we add a layer in gs-stable this would be easier for the tester
image

@MV88 MV88 added New Feature used for new functionalities C265-ATOLCD-2023-DEV_MS_CNR labels Jun 28, 2023
@MV88 MV88 added this to the 2023.02.00 milestone Jun 28, 2023
@MV88 MV88 requested a review from offtherailz June 28, 2023 14:31
@MV88 MV88 self-assigned this Jun 28, 2023
@tdipisa tdipisa linked an issue Jun 28, 2023 that may be closed by this pull request
5 tasks
@offtherailz

offtherailz commented Jun 30, 2023

Copy link
Copy Markdown
Member

From our meeting with @MV88 to provide this functionality, I suggested to use the current image tool, improving it at least, instead of introducing the support to markdown inside wyiswyg that may be a problem for future maintainance of the functionality.
I love markdown and if we are going to introduce it I should do it as a complete alternative syntax.
I'd like to see with you too @allyoucanmap to have your opinion, next week.

@offtherailz offtherailz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

  • Please remove the mixed markdown support and so the new image tool. Use the image tool present in react wysiwyg (replacing react quill)
  • Check for backward compatibility with existing templates created with react quill ( I tried and it seems to work on demo map)

@MV88 MV88 requested a review from offtherailz July 3, 2023 14:15
@MV88

MV88 commented Jul 3, 2023

Copy link
Copy Markdown
Contributor Author
  • Please remove the mixed markdown support and so the new image tool. Use the image tool present in react wysiwyg (replacing react quill)
  • Check for backward compatibility with existing templates created with react quill ( I tried and it seems to work on demo map)

done both @offtherailz please review it again

Comment thread web/client/components/TOC/fragments/settings/FeatureInfoEditor.jsx Outdated
Comment thread web/client/translations/data.de-DE.json Outdated
Comment thread web/client/translations/data.en-US.json Outdated
Comment thread web/client/translations/data.es-ES.json Outdated
Comment thread web/client/translations/data.fr-FR.json Outdated
Comment thread web/client/translations/data.it-IT.json Outdated
Comment thread web/client/components/mapviews/settings/CompactRichTextEditor.jsx Outdated
MV88 and others added 7 commits July 5, 2023 11:28
Co-authored-by: Lorenzo Natali <lorenzo.natali@geosolutionsgroup.com>
Co-authored-by: Lorenzo Natali <lorenzo.natali@geosolutionsgroup.com>
Co-authored-by: Lorenzo Natali <lorenzo.natali@geosolutionsgroup.com>
Co-authored-by: Lorenzo Natali <lorenzo.natali@geosolutionsgroup.com>
Co-authored-by: Lorenzo Natali <lorenzo.natali@geosolutionsgroup.com>
….jsx

Co-authored-by: Lorenzo Natali <lorenzo.natali@geosolutionsgroup.com>
Co-authored-by: Lorenzo Natali <lorenzo.natali@geosolutionsgroup.com>
@MV88 MV88 requested a review from offtherailz July 5, 2023 09:31
Comment thread web/client/components/mapviews/settings/CompactRichTextEditor.jsx Outdated
@offtherailz offtherailz enabled auto-merge (squash) July 5, 2023 10:07
@offtherailz offtherailz merged commit 3c0e7ac into geosolutions-it:master Jul 5, 2023
@MV88

MV88 commented Jul 5, 2023

Copy link
Copy Markdown
Contributor Author

@ElenaGallo please test it in DEV

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C265-ATOLCD-2023-DEV_MS_CNR New Feature used for new functionalities

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add parametric image parsing to template viewer

2 participants