Add CSS inclusion

Works as expected for me on D9.5.10 PHP8.1.7

New config key will need schema update and post_update hook.

Question how come "Only look for links inside these CSS selectors." does meet the case?

We had a content type whose links must open in a new tab, even though they would be considered internal, as they pointed to the same site pages.

The case this feature solved for us was forcing some links to behave as external, even when they were not.

I hope it makes sense to you now! :)

Will have to think about that, module scope is to cover external links which sounds like is not the case it's trying to be used for.

Think about it like "another config to allow the user to solve its needs". Maybe my use case is a bit kind of weird but, in my opinion, It should not harm giving the user a complete include/exclude logic toolset.

Whatever you decide, I'm sure it will be ok :)

So there are other issues trying to get into the 2.0.x beta release. But if this one is worked on and doesn't break anything I'm not opposed to it. Will need test coverage and MR updated for 2.0.x

vidorado → closed merge request !20

vidorado → reopened merge request !20

vidorado → changed the visibility of the branch 8.x-1.x to hidden.

Sorry for the flooding! (my first MR 😅)

I didn't managed to resolve a PHPUnit test issue in the pipeline. In my local environment all tests are passing.

@smustgrave, I figured the problem out :)

All tests passing on Drupalcode!

Thanks I see nothing wrong with the MR.

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