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: render product descriptions with images properly #1263

Merged
merged 3 commits into from
Sep 2, 2022

Conversation

shauke
Copy link
Collaborator

@shauke shauke commented Aug 31, 2022

  • use ishServerHtml directive to handle embedded image resources

PR Type

[x] Bugfix

What Is the Current Behavior?

  • Product descriptions (short or long) are not always rendered as HTML and are not rendered with specific treatment for server side images and ICM specific link notation.
  • The ishServerHtml directive does not handle ICM embedded images without ismediaobject notation. TinyMCE embedding format after update just uses simple relative URLs in the Intershop static file system
  • Directly navigating between different product detail pages does not reset a previously selected product detail tab (default would be the description tab).

What Is the New Behavior?

  • Product descriptions (short or long) are rendered with specific treatment for server side images and ICM specific link notation using the ishServerHtml directive.
  • The ishServerHtml directive handles ICM embedded images without ismediaobject notation.
  • When navigating between different product detail pages the selected product detail tab is reset to the product description.

Does this PR Introduce a Breaking Change?

[x] No

Other Information

AB#79244

@shauke shauke added this to the 3.1 milestone Aug 31, 2022
@shauke shauke self-assigned this Aug 31, 2022
shauke added 2 commits August 31, 2022 13:32
* use `ishServerHtml` directive to handle embedded image resources
…source format (#1263)

* images are no longer embedded with `ismediaobject` markers after TinyMCE update
* added handling for simple relative image URLs into the '/INTERSHOP/' static file system
shauke added a commit that referenced this pull request Aug 31, 2022
@shauke shauke force-pushed the bugfix/product_descriptions_with_images branch from 38838a2 to abf4d73 Compare August 31, 2022 11:35
@shauke shauke marked this pull request as ready for review August 31, 2022 11:42
@shauke shauke requested review from Eisie96 and SGrueber August 31, 2022 11:42
shauke added a commit that referenced this pull request Aug 31, 2022
@shauke shauke force-pushed the bugfix/product_descriptions_with_images branch from abf4d73 to b75974f Compare August 31, 2022 12:45
shauke added a commit that referenced this pull request Aug 31, 2022
@shauke shauke force-pushed the bugfix/product_descriptions_with_images branch from b75974f to 2976009 Compare August 31, 2022 13:37
Eisie96
Eisie96 previously approved these changes Sep 1, 2022
…product detail pages (#1263)

Co-authored-by: Marcel Eisentraut <meisentraut@intershop.de>
@shauke shauke force-pushed the bugfix/product_descriptions_with_images branch from 56c111d to 8a2f12f Compare September 2, 2022 07:33
@shauke shauke merged commit 8138800 into develop Sep 2, 2022
@shauke shauke deleted the bugfix/product_descriptions_with_images branch September 2, 2022 07:34
shauke added a commit that referenced this pull request Sep 2, 2022
* use `ishServerHtml` directive to handle embedded image resources
shauke added a commit that referenced this pull request Sep 2, 2022
…source format (#1263)

* images are no longer embedded with `ismediaobject` markers after TinyMCE update
* added handling for simple relative image URLs into the '/INTERSHOP/' static file system
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