Account created on 7 November 2018, about 6 years ago
#

Recent comments

πŸ‡ΊπŸ‡¦Ukraine rolki
πŸ‡ΊπŸ‡¦Ukraine rolki

Thanks, I like these changes.

πŸ‡ΊπŸ‡¦Ukraine rolki

It seems that the merge did not succeed.

πŸ‡ΊπŸ‡¦Ukraine rolki

LGTM, thanks.

πŸ‡ΊπŸ‡¦Ukraine rolki

Thanks, David,
Please check one comment I made regarding Markup.

πŸ‡ΊπŸ‡¦Ukraine rolki

Thanks, just one improvement suggestion from me.

πŸ‡ΊπŸ‡¦Ukraine rolki

PR looks good, just one small comment.

πŸ‡ΊπŸ‡¦Ukraine rolki

PR looks good, just one small comment.

πŸ‡ΊπŸ‡¦Ukraine rolki

Hi,
Thanks for working on this, a few small comments, please check it out.

πŸ‡ΊπŸ‡¦Ukraine rolki

The changes look good, let's just change the deprecated message in the templates, thanks!

πŸ‡ΊπŸ‡¦Ukraine rolki

πŸ’ͺ🏽πŸ’ͺ🏽πŸ’ͺ🏽

πŸ‡ΊπŸ‡¦Ukraine rolki

The changes look logical.

Argument for the community: In cases where the module changes the color scheme for everything, these changes must be forced, as it makes no sense to do this in all other extensions. Accordingly, after changing the scheme or turning off the module, everything will return to its previous state.

Thanks!

πŸ‡ΊπŸ‡¦Ukraine rolki

Please change the MR status to ready.

πŸ‡ΊπŸ‡¦Ukraine rolki

Changes LGTM. Thanks!

πŸ‡ΊπŸ‡¦Ukraine rolki

This is a good idea, and I agree that such a problem does exist. It would be great if it was integrated into the editor_advanced_link module in future releases.

πŸ‡ΊπŸ‡¦Ukraine rolki

The changes look good to me.

Production build 0.71.5 2024