? Pending

User tests: Successful: Unsuccessful:

avatar alikon
alikon
8 Jan 2022

Pull Request for Issue #13522.

Summary of Changes

don't remove <joomla-hidden-mail>

Testing Instructions

  • Add an "Articles Category" module on the sample site
  • Under "Display Options" tab: everything on Hide, except "Introtext= show" and "Show Read More=show"
  • Create an article with the below html:
<p>Intro text <a href="mailto:sample@example.com">sample@example.com</a> - more more more</p>
<hr id="system-readmore" />
<p><br />Paragraph 1<br />1<br />2<br />3<br />4<br />5</p>
<p>Paragraph 2<br />1<br />2<br />3<br />4<br />5</p>
<p>Paragraph 3<br />1<br />2<br />3<br />4<br />5</p>
<p>End</p>

Actual result BEFORE applying this Pull Request

This email address is being protected from spambots. You need JavaScript enabled to...

Expected result AFTER applying this Pull Request

Introtext shows the email address

avatar alikon alikon - open - 8 Jan 2022
avatar alikon alikon - change - 8 Jan 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 8 Jan 2022
Category Modules Front End
avatar Fedik
Fedik - comment - 8 Jan 2022

I suspect you have missed a branch, should go to 4.0-dev ?
Joomla 3 does not have <joomla-hidden-mail>

avatar alikon alikon - change - 8 Jan 2022
Title
[3] don't remove <joomla-hidden-mail> from module
[3] don't remove from module
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2022-01-08 09:37:15
Closed_By alikon
Labels Added: ?
avatar alikon
alikon - comment - 8 Jan 2022

oops

avatar alikon alikon - close - 8 Jan 2022

Add a Comment

Login with GitHub to post a comment