This merge request was automatically created by the Project Update Bot. It contains the changes from run 11-121090.
Closes #3373510
Julian Pustkuchen (ee77ee27) at 29 Feb 18:56
Issue #3342433 by fabiansierra5191: Accessibility - Decorative icon...
Closes #3342433
Closes #3254564
Julian Pustkuchen (502b3ae8) at 02 Feb 16:31
Issue #3333202 by LRWebks, Anybody: Allow to add classes for extern...
Closes #3333202
Closes #3411377
Changed the names to extAdditionalLinkClasses
and extAdditionalMailtoClasses
respectively, in order to stay consistent with the previous name changes and to avoid a mix-up between extLinkClasses
and the short module name version used in other variable names (extLink
).
extLinkAdditionalClasses
extMailtoAdditionalClasses
?
To be consistent
Should we also prefix this with additional_ ?
appended to is a bit strange wording. Perhaps better "set on", "added on" or sth. like that?
appended to is a bit strange wording. Perhaps better "set on", "added on" or sth. like that?
appended to is a bit strange wording. Perhaps better "set on", "added on" or sth. like that?
appended to is a bit strange wording. Perhaps better "set on", "added on" or sth. like that?
Whoops, looks like these are still the old and now non-existent settings...
Yes of course,
guess I just thoughtlessly implemented what you proposed in the issue description "Add a setting
'Add custom CSS classes' (boolean)
and if enabled show a textfield 'Custom CSS classes'"
Also plenty of other ExternalLinks settings use this technique as well instead of value checks.