I won’t go so far to say I know the code inside out, but I’ll chip in my ![]()
I looked to see if any tests might break if current_user_rtl or site_locale_rtl were removed. I found none and in fact the only place they exist in the Core code is in the models/rtl.rb file.
So removing them might break some plugins if they are using them, but it shouldn’t break Discourse.
I don’t think “i18n” would need to be used in the proposed enabled but I’ve seen it used in other files.