Add config option for more target link

Please reroll against 2.0.x-dev.

Furthermore "more_target" isn't a good and self-explaining name for the key. Please change it to more_target_blank for example.

I'll work on it!

I've done a reroll from #2, and added the changes requested by #4. I also corrected some minor CS.

If there is anything that could be improved, feel free to give me any kind of feedback!

Moving this to NR.

Hi, i will review your patch.

I reviewed your patch and IMHO your patch is solving all the points mentioned above, so I'm moving to RTBC.

Could we please have an interdiff from #2 to #6 to see the changes?

Okay, gonna make the interdiff.

Done.

The use statements had an extra space at the beginning of each line, and had some unused classes. This patch fixes it.

Although manual tests worked, and it looks ok, code-wise, I've changed the default config types to match the other Booleans, just to be more clear.

Lets get some other eyes on your patch before marking it RTBC please.

I'll review it.

The substitutions on the arrays from #12 are correct.

I've re-rolled the patch and made a couple of changes:

1. I've added a functional test.
2. I've added the new field to smart_trim.schema.yml
3. My editor made a few additional minor coding standard fixes.

Needs a quick review before merging.

-mike

Patch was not applying - re-rolled.

-mike

Reviewed locally and then committed. Sorry, I skipped the RTBC step.