Skip to content

Conversation

dabo-odoo
Copy link
Contributor

@dabo-odoo dabo-odoo commented Nov 19, 2024

Description of the issue/feature this PR addresses:
There is an issue where product names that are too long or too big cause certain elements to be pushed offscreen in the mobile version of the cart in the eCommerce.

Current behavior before PR:
Before the change, in the mobile version of the eCommerce cart, if a product had a name that was too long or the font size was too big it would push the elements to its right offscreen.

Desired behavior after PR is merged:
After the change, the overflow of a product name that is too long or too big gets replaced with ellipsis, keeping the elements to its right from beign pushed offscreen.

task-4260720
opw-4253880


I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

@robodoo
Copy link
Contributor

robodoo commented Nov 19, 2024

Pull request status dashboard

@C3POdoo C3POdoo requested a review from a team November 19, 2024 10:24
@C3POdoo C3POdoo added the OE the report is linked to a support ticket (opw-...) label Nov 19, 2024
@chevalierv
Copy link
Contributor

Hello @dabo-odoo

LGTM 👍 In the future, we could also remove the picture to leave more place for the text in that specific use case. I'll investigate that on my side!

@robodoo r+

Have a nice day!

@robodoo
Copy link
Contributor

robodoo commented Nov 20, 2024

@dabo-odoo @chevalierv unable to stage: merge conflict

…e page

Before the change, in the mobile version of the eCommerce cart, if a product
had a name that was too long or the font size was too big it would push the
elements to its right offscreen.
After the change, the overflow of a product name that is too long or too big
gets replaced with ellipsis, keeping the elements to its right from beign
pushed offscreen.

task-4260720
opw-4253880
@dabo-odoo dabo-odoo force-pushed the master-mobile-name-overflow-dabo branch from 7c611b1 to 2ca225c Compare November 20, 2024 12:38
@dabo-odoo
Copy link
Contributor Author

Fixed the merge conflict @chevalierv

@chevalierv
Copy link
Contributor

@robodoo r+

robodoo pushed a commit that referenced this pull request Nov 20, 2024
…e page

Before the change, in the mobile version of the eCommerce cart, if a product
had a name that was too long or the font size was too big it would push the
elements to its right offscreen.
After the change, the overflow of a product name that is too long or too big
gets replaced with ellipsis, keeping the elements to its right from beign
pushed offscreen.

task-4260720
opw-4253880

closes #187698

Signed-off-by: Valentin Chevalier <vcr@odoo.com>
@robodoo robodoo added the 18.1 label Nov 20, 2024
@robodoo robodoo closed this Nov 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

18.1 OE the report is linked to a support ticket (opw-...)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants