Add a rule to detect LTR properties#533
Conversation
|
Do you hand roll LTR files or generate them with something like CSSJanus? On Fri, Aug 1, 2014 at 5:55 AM, Lewis Nyman notifications@github.com
|
|
We used to hand roll separate files, which was a maintenance nightmare. Now we've switched to using [dir="rtl"] selectors directly after the LTR CSS in Drupal 8. I've never seen CSSJanus before, interesting! I'll ask around if it's been tried previously. |
|
I had a lot of fun working on this rule, the test process is great! I don't know if this rule should be added to csslint or if it's applicable to everyone, but it would be nice to get some feedback. |
|
@lewisnyman BTW, there is also a php version of cssjanus. We use it in MediaWiki and it really saves you a ton of trouble. |
There was a problem hiding this comment.
Please remove the trailing comma.
Drupal has a requirement to support RTL languages, this is a lot of work to achieve and often slips through our review process. I want to make csslint a formal part of our review process and having a rule that detects LTR specific properties would be a great help.
I've opened the equivalent issue to track this on Drupal.org here:
https://www.drupal.org/node/2313783