main ← fix/layout-problems-rtl
opened 10:27PM - 29 May 23 UTC
We currently have some CSS rules in `common/base/rtl.scss` that were added to wo…rkaround shortcomings of the R2 gem that we used to use to generate a version of our CSS that's suitable for RTL layouts. Those workarounds are mostly duplicate rules with the only difference being that they're flipped to suit RTL layouts.
However, we've recently replaced R2 with `rtlcss` which doesn't have those shortcomings of R2 (see https://github.com/discourse/discourse/commit/f94951147ea9d1c48ea68f0dabd568e70cb94674) which means those workarounds/duplicate rules need to be removed because they're getting flipped by `rltcss`, essentially reverting them to their original LTR version and causing issues with RTL layouts.
This PR removes those workarounds that are no longer needed, and cleans up the the file that contains our RTL-specific CSS.
Meta topic: https://meta.discourse.org/t/avatar-in-rtl-website-in-wrong-place/264676?u=osama.