Would it be possible to set the default filter to "Strip Tags"? Or at least give administrators the ability to set the default ourselves? My organization is testing this module for use on a policy website to allow visitors to compare revisions, but the feedback thus far has shown a heavy dislike for the "Raw" filter being the default. And our content creators are finding it just as annoying as our audience. Everyone overwhelmingly would rather compare content first, not markup.

Thank you for creating a much needed module!

CommentFileSizeAuthor
#21 3165141-21.patch12.36 KBjeroent
#7 diff-default-filter-config-export.PNG15.27 KByepa
#7 diff-default-filter.PNG46.74 KByepa
#2 diff_default_filter-3165141-2.patch3.41 KBmihai11

Issue fork diff-3165141

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

gallegosj created an issue. See original summary.

mihai11’s picture

mihai11 commented
StatusFileSize
new diff_default_filter-3165141-2.patch3.41 KB

I have created a patch for you problem. A new configuration has been added on General settings form under FIELD BASED LAYOUT SETTINGS.

bbu23’s picture

bbu23
Location 🇪🇺🇪🇺🇪🇺
commented
Status: Active » Needs review
daddison’s picture

daddison commented

The patch applied cleanly for me and works as expected.

gallegosj’s picture

gallegosj commented

Also confirming, patch at #2 worked for me as expected.

charlie1volley’s picture

charlie1volley commented

Applied successfully and works as designed. Thanks for a great module and the patch at #2 to add this very useful option.

yepa’s picture

yepa
he/him
Primary language French
Location Geneva
commented
Version: 8.x-1.0 » 8.x-1.x-dev
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new diff-default-filter.PNG46.74 KB
new diff-default-filter-config-export.PNG15.27 KB

The patch applied cleanly on Drupal 8.9.8
the code review did not show any errors.

On the config page (admin/config/content/diff/general), we can now choose the default filter:
Diff default filter

The configuration is correctly exported:
Diff default filter config export

It works as expected and it's much appreciated by editors (UX).

gallegosj’s picture

gallegosj commented

When can we expect this to be ported over?

dbielke1986’s picture

dbielke1986 commented

Would be great to have this feature in a official release

bkosborne’s picture

bkosborne
Primary language English
Location New Jersey, USA
commented
Related issues: +#3351660: [DrupalHtmlEngine] CKEditor 5 saves content as single line HTML making revision diffing impossible

Seems important to get this in, especially now that CKEditor 5 collapses all HTML into a single line, making the "raw" comparison method pretty useless. See #3351660: [DrupalHtmlEngine] CKEditor 5 saves content as single line HTML making revision diffing impossible

acbramley’s picture

acbramley commented
Status: Reviewed & tested by the community » Needs work
Issue tags: +Needs tests

Thanks for the work on this one. I've recently taken up maintainership of this project and am looking through the RTBC issues.

I agree here that having the Raw as the default is undesirable.

To get this in, I'll need the patch applied against the latest 8.x-1.x code in an MR with tests.

Thanks!

dbielke1986 opened merge request !50

dbielke1986’s picture

dbielke1986 commented
Status: Needs work » Needs review

Just created a fork and an MR with the changes of #2

Please review and test.

acbramley’s picture

acbramley commented
Status: Needs review » Needs work

Thanks for that, we still need automated tests as well as some phpstan fixes.

pahles’s picture

pahles commented

I tried updating the module today but this patch no longer gets applied. Composer just hangs. It works when I revert to 8.x-1.1 but cannot be applied in 8x-1.2 or 8x-1.3.

jeroent’s picture

jeroent
Primary language Dutch
Location 🇧🇪
commented
Version: 8.x-1.x-dev » 2.x-dev

JeroenT changed the visibility of the branch 3165141-set-the-default to hidden.

JeroenT opened merge request !95

jeroent’s picture

jeroent
Primary language Dutch
Location 🇧🇪
commented
Status: Needs work » Needs review

I created a new MR for version 2.x and added test coverage.

jeroent’s picture

jeroent
Primary language Dutch
Location 🇧🇪
commented

Failing test-only run: https://git.drupalcode.org/issue/diff-3165141/-/jobs/2082920

jeroent’s picture

jeroent
Primary language Dutch
Location 🇧🇪
commented
StatusFileSize
new 3165141-21.patch12.36 KB
gbyte’s picture

gbyte
Location Berlin
commented
Status: Needs review » Reviewed & tested by the community

Works as advertised, code looks clean.

heddn’s picture

heddn
Primary language English
Location Nicaragua
commented
Status: Reviewed & tested by the community » Needs work
Issue tags: -Needs tests +Needs reroll

Needs a rebase

jeroent’s picture

jeroent
Primary language Dutch
Location 🇧🇪
commented
Status: Needs work » Needs review

In rebased the -dev branch to the MR.

heddn’s picture

heddn
Primary language English
Location Nicaragua
commented
Status: Needs review » Needs work
Issue tags: -Needs reroll

Posted feedback on MR.

jeroent’s picture

jeroent
Primary language Dutch
Location 🇧🇪
commented

@heddn,
I can't find any feedback on the MR?

heddn’s picture

heddn
Primary language English
Location Nicaragua
commented

Oops. Posted now.

jeroent’s picture

jeroent
Primary language Dutch
Location 🇧🇪
commented
Status: Needs work » Needs review

dbielke1986 closed merge request !50