magento2-page-builder icon indicating copy to clipboard operation
magento2-page-builder copied to clipboard

When added a new empty row to product description it gives a Uncaught ReferenceError: jarallax is not defined

Open andrewdaluz opened this issue 8 months ago • 2 comments

Preconditions (*)

  1. Any version of page-builder installed

Steps to reproduce (*)

  1. Go to a product edit page on admin 2.Edit the description field with Page Builder
  2. Add a new empty row
  3. Save the product
  4. After the page is reloaded, try to edit with Page Builder and remove the empty row
  5. An error is triggered on console with Uncaught ReferenceError: jarallax is not defined
  6. Save the product (after the row is removed)
  7. Then the page is reloaded and the empty row continues appearing there and it's not possible to remove it

Expected result (*)

  1. The row is removed with no errors after the product is saved

Actual result (*)

  1. The empty row continues showing up even after his removal

andrewdaluz avatar Mar 04 '25 17:03 andrewdaluz

Hi @andrewdaluz. Thank you for your report. To speed up processing of this issue, make sure that you provided sufficient information. Add a comment to assign the issue: @magento I am working on this


Join Magento Community Engineering Slack and ask your questions in #github channel.

m2-assistant[bot] avatar Mar 04 '25 17:03 m2-assistant[bot]

It looks like the issue is related to the misspelling of the jarallax variable that should be _jarallax:

_proto.destroy = function destroy() {
      _previewCollection2.prototype.destroy.call(this);

      if (this.element) {
        jarallax(this.element, "destroy");
      }
    }

to

_proto.destroy = function destroy() {
      _previewCollection2.prototype.destroy.call(this);

      if (this.element) {
        _jarallax(this.element, "destroy");
      }
    }

andrewdaluz avatar Mar 04 '25 17:03 andrewdaluz

Hello @andrewdaluz ,

Thank you for your report and collaboration.

We attempted to reproduce the issue in version 2.4 develop but were unable to do so. Attached the screen recording for your references.

Image Image Image

Image

Image

Steps to Reproduce:

  • Login to magento admin
  • Go to a product edit page on admin
  • Edit the description field with Page Builder
  • Add a new empty row
  • Save the product
  • After the page is reloaded, try to edit with Page Builder and remove the empty row
  • Save the product (after the row is removed)

We followed these steps and did not encounter any errors. Could you please recheck the issue in the latest version? Alternatively, let us know if we missed any steps while reproducing the issue.

Therefore, we are marking this ticket as "Needs Update."

Thank You!

engcom-November avatar Apr 03 '25 12:04 engcom-November

Interesting. Might that be a Firefox or Chrome tests? The issue is on https://github.com/magento/magento2-page-builder/blob/develop/app/code/Magento/PageBuilder/view/adminhtml/web/js/content-type/row/preview.js You can see that the variable in the function (_jarallax) while in the scope of the function it's been called as jarallax() and not the _jarallax() method.

andrewdaluz avatar Apr 03 '25 15:04 andrewdaluz

Hello @andrewdaluz ,

We attempt to reproduce this issue in my latest 2.4 develop instances but we are unable to reproduce it and we verified in chrome and Firebox but the issue is not reproducible.

Steps to Reproduce:

  • Login to magento admin
  • Go to a product edit page on admin
  • Edit the description field with Page Builder
  • Add a new empty row
  • Save the product
  • After the page is reloaded, try to edit with Page Builder and remove the empty row
  • Save the product (after the row is removed)

We followed these steps and did not encounter any errors. Could you please recheck the issue in the latest version? Alternatively, let us know if we missed any steps while reproducing the issue.

Therefore, we are marking this ticket as "Needs Update."

Thank You!

engcom-November avatar Sep 25 '25 08:09 engcom-November

Hello,

This issue is being closed since it has not been updated in a long time. Please feel free to reopen or raise a new ticket if the issue still exists.

Thank You!

engcom-November avatar Oct 24 '25 13:10 engcom-November