Address Layout in Module

[Fiber]

Change https://github.com/CE-PhoenixCart/Phoen ... 1-L144C137 to

Code: Select all

      $text = strip_tags(html_entity_decode(preg_replace("{<br[^>]*>\r?\n?}gi", $this->lf, $email_text), ENT_QUOTES | ENT_SUBSTITUTE | ENT_HTML5, $this->build_params['text_charset']));

Text emails strip out the br tags.

No luck still the same..strange

afterchangemailphp.png

[MyGamesShop]

Not a bug.

Can we restore support for the .jpeg file extension.

Ta

[bonbec]

Hello,
I don't know if it's a bug (version 1.0.9.1) or if it's normal. I want to bold some words in the category description. <b></b> tags cause a line break.

baliseB.jpg

[heatherbell]

I don't know if it's a bug (version 1.0.9.1) or if it's normal. I want to bold some words in the category description. <b></b> tags cause a line break.

It does not do that on a default clean install. Guess you have a style that is causing it, maybe something with display:block;

[bonbec]

Thank you for your reply but it is the same in a fresh install (version 1.0.9.1). I also tested on a clean installation before posting my message.

[burt]

If not standard text in a card-body, it must be in a card-text container.
See https://getbootstrap.com/docs/4.6/compo ... rd/#images

In other words:

Code: Select all

<div class="card-text">
  abc <b>123</b> def
</div>

or

Code: Select all

<p class="card-text">
  abc <b>123</b> def
</p>

etc

[heatherbell]

Thank you for your reply but it is the same in a fresh install (version 1.0.9.1). I also tested on a clean installation before posting my message.

Sorry, tested Product Description, not Category Description

[bonbec]

...

Code: Select all

<p class="card-text">
  abc <b>123</b> def
</p>

...

Thank you @burt
I find that it is more intuitive and practical to write the text directly with the tags than to think about adding the code cited above.
So I placed the modified file in my template:
templates/MyTemplate/includes/modules/content/index_products/tpl_cm_ip_category_manufacturer_description.php
It work fine as I like.

@heatherbell No problem, I wasn't explicit enough in my message I think.

[bonbec]

Hello,
Not a bug but a space and a / to remove in includes/modules/header_tags/ht_canonical.php line 57 :

Code: Select all

      $GLOBALS['Template']->add_block('<link rel="canonical" href="' . $this->build_link() . '" />' . PHP_EOL, $this->group);

The generated line is :

Code: Select all

<link rel="canonical" href="http://localhost/1091/product_info.php?products_id=9" />

It must be :

Code: Select all

<link rel="canonical" href="http://localhost/1091/product_info.php?products_id=9">

[ecartz]

It must be :

According to whom?