[14.0][FIX] fix white-space display in markdown widget

[huguesdk]

add css rule to override the default white-space declaration for text fields (pre-wrap) to ensure the html generated from markdown is correctly displayed.

[github-actions]

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

[huguesdk]

@OCA/web-maintainers would someone please review or merge this?

[etobella]

Can you use a proper commit description 🙏 ?

Also, it would be nice to show a before and after with screenshots, but well, it is not needed right now.

[huguesdk]

hi @etobella and thank you for the review! what do you mean by proper commit description? isn’t the current message clear enough?

source:

# hello, this is an h1

## hello, this is an h2

this is an ul:

*   an element
*   another element

before:

after:

[etobella]

I am talking about the commit message.

It is

[FIX] fix white-space display in markdown widget
add css rule to override the default white-space declaration for text
fields (pre-wrap) to ensure the html generated from markdown is
correctly displayed.

And should include the module name at the begining:

[FIX] web_widget_text_markdow: fix white-space display in markdown widget
add css rule to override the default white-space declaration for text
fields (pre-wrap) to ensure the html generated from markdown is
correctly displayed.

https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#commit-message

[huguesdk]

thanks for the explanation. done. i didn’t know that the module name was mandatory. with modules with long names it leaves not much space for a meaningful message since the subject is limited to 50 characters.

[huguesdk]

there is a problem with the tests that seems unrelated to this change.

[hbrunn]

old js tests are a mess, and I've given up trying to fix them

[etobella]

/ocabot merge patch

[OCA-git-bot]

This PR looks fantastic, let's merge it!
Prepared branch 14.0-ocabot-merge-pr-2433-by-etobella-bump-patch, awaiting test results.

[OCA-git-bot]

@etobella your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-2433-by-etobella-bump-patch.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

[hbrunn]

@etobella given the failing tests, ocabot won't merge this. did so manually

[huguesdk]

thanks all!