Prestashop: The tooltip of Delete/Edit product from cart ( order view page migrated) - wrong behaviour

Created on 31 Mar 2020  路  4Comments  路  Source: PrestaShop/PrestaShop

Steps to reproduce

  • Go to order view page (migrated ) for order with id=4 (sell/orders/orders/4/view).
  • Add a product to existing cart. Now you have two products but there is a problem of the tooltip when hovering on "delete icon" and "Edit" icon, it doesn't work for the newly added products.
    image

    • Delete the newly added product, now you see a weird behaviour of the tooltip:

image

  • play with page scroll, you will see the tooltip go to the top left of the screen:

image

Expected behavior

The tooltip must appear for every product on cart when hovering on icon, in other words, for both producst (those who are loaded on first page load and those who are loaded via ajax).
When the delete icon is clicked, all tooltips must disapear till we hover on one of them.

  • Mozilla Firefox 74.0

    • New installation of 1.7.7.0

1.7.7.x BO Bug Fixed Order PR available Regression Trivial migration

Most helpful comment

Fixed by #18445

All 4 comments

Hi @arouiadib,

Thanks for your report.
I manage to reproduce the issue with 1.7.7.x branch, the tooltip for "Delete" and "Edit" buttons are not displayed when we add a new product, it is OK when we refresh the page.
image
image
I鈥檒l add this to the debug roadmap so that it鈥檚 fixed. If you have already fixed it on your end or if you think you can do it, please do send us a pull request!
Thanks!

Fixed it, also there was another bug, when you was adding a product, tooltips was not added to this product until the reload of the page.

Yes you are right, @NeOMakinG , thank you very much for the PR.

Fixed by #18445

Was this page helpful?
0 / 5 - 0 ratings

Related issues

marionf picture marionf  路  3Comments

khouloudbelguith picture khouloudbelguith  路  3Comments

matks picture matks  路  3Comments

Van-peterson picture Van-peterson  路  3Comments

centoasa picture centoasa  路  3Comments