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

[BOOM-3055] Formatting of text incorrect if long feature description used #9978

Closed
prestonBot opened this issue Aug 22, 2018 · 5 comments

Comments

@prestonBot
Copy link
Collaborator

@prestonBot prestonBot commented Aug 22, 2018

This issue has been migrated from the Forge. Read the original ticket here.

  • Reporter: gray
  • Created at: Tue, 16 May 2017 19:58:17 +0200

Formatting of text incorrect if long feature description used. Check attached image of 'filter by'

  • How to reproduce the issue ?

N/A

@prestonBot

This comment has been minimized.

Copy link
Collaborator Author

@prestonBot prestonBot commented Aug 22, 2018

This comment has been migrated from the Forge. Read the original comment here.

  • Author: vincentbz
  • Created at: Wed, 17 May 2017 11:43:46 +0200

Salut Tristan Lehot est ce qu'on pourrait regarder ça ensemble stp ?

@rdy4ever

This comment has been minimized.

Copy link
Contributor

@rdy4ever rdy4ever commented Aug 31, 2018

I did a PR, not sure if it is the best solution but feel free to review/comment. My PR truncates long feature titles so it will keep everything on one line.
image

@PierreRambaud

This comment has been minimized.

Copy link
Contributor

@PierreRambaud PierreRambaud commented Feb 21, 2019

Ping @TristanLDD any idea? What about @rdy4ever suggestion?

@colinegin colinegin added this to Backlog in PrestaShop 1.7.6 via automation Feb 26, 2019
@colinegin colinegin moved this from Backlog to To do in PrestaShop 1.7.6 Feb 27, 2019
@colinegin colinegin added To Do and removed TBS waiting for UX labels Mar 8, 2019
@PierreRambaud PierreRambaud moved this from To do to In progress in PrestaShop 1.7.6 Mar 18, 2019
@PierreRambaud

This comment has been minimized.

Copy link
Contributor

@PierreRambaud PierreRambaud commented Mar 19, 2019

Also fixed in facetedsearch module. WIthout having to change Core css.

@PierreRambaud PierreRambaud moved this from In progress to To be reviewed in PrestaShop 1.7.6 Mar 19, 2019
@eternoendless

This comment has been minimized.

Copy link
Member

@eternoendless eternoendless commented Mar 25, 2019

Done here I think: PrestaShop/ps_facetedsearch#52

@Saycile Saycile removed the To Do label Mar 29, 2019
@marionf marionf added the Fixed label Apr 10, 2019
@marionf marionf moved this from To be reviewed to To be merged in PrestaShop 1.7.6 Apr 15, 2019
@marionf marionf closed this Apr 15, 2019
PrestaShop 1.7.6 automation moved this from To be merged to Done Apr 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.