CKEditor has a configuration to convert URLs / emails to links.

When the processing of the WYSIWYG is done by CivicTheme these filters have not been applied (and so they are URLS and not anchor tags) and so we do not add CivicTheme classes to these links.

CivicTheme should apply processing after the HTML has been filtered by other filters.

CommentFileSizeAuthor
#9 FE test.png971.47 KBsonam.chaturvedi
#9 BE.png124.01 KBsonam.chaturvedi

Comments

alex.skrypnyk created an issue. See original summary.

alex.skrypnyk’s picture

Confirmed. When a URL provided as a text - CivicTheme's link classes are not assigned because the URL is not yet converted to the link.

We need to look into processing of links done by CivicTheme to happen as late as possible in the rendering pipeline.

alex.skrypnyk’s picture

Assigned: Unassigned » joshua1234511
fionamorrison23’s picture

Issue tags: +civictheme-release-1.8
fionamorrison23’s picture

alex.skrypnyk’s picture

Status: Active » Needs work

  • 4cb52260 committed on 1.x
    Issue #3415687 by joshua1234511, alex.skrypnyk: Fixed URLs converted to...
alex.skrypnyk’s picture

Assigned: joshua1234511 » sonam.chaturvedi
Status: Needs work » Needs review
sonam.chaturvedi’s picture

Assigned: sonam.chaturvedi » alex.skrypnyk
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new124.01 KB
new971.47 KB

Verified and tested on 1.x-dev

Testing Results:
1 When a URL provided as a text, CivicTheme's classes added - PASS
2 When a URL provided as a link, CivicTheme's classes added - PASS

Screenshots:
BE
test

FE
test2

alex.skrypnyk’s picture

Status: Reviewed & tested by the community » Fixed

Implemented, tested and merged. Will be released in 1.7.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.