{"action":"opened","issue":{"url":"https://api.github.com/repos/powerbuoy/SleekCSS/issues/4","repository_url":"https://api.github.com/repos/powerbuoy/SleekCSS","labels_url":"https://api.github.com/repos/powerbuoy/SleekCSS/issues/4/labels{/name}","comments_url":"https://api.github.com/repos/powerbuoy/SleekCSS/issues/4/comments","events_url":"https://api.github.com/repos/powerbuoy/SleekCSS/issues/4/events","html_url":"https://github.com/powerbuoy/SleekCSS/issues/4","id":186102635,"number":4,"title":"RTL support","user":{"login":"saeedseyfi","id":5411866,"avatar_url":"https://avatars.githubusercontent.com/u/5411866?v=3","gravatar_id":"","url":"https://api.github.com/users/saeedseyfi","html_url":"https://github.com/saeedseyfi","followers_url":"https://api.github.com/users/saeedseyfi/followers","following_url":"https://api.github.com/users/saeedseyfi/following{/other_user}","gists_url":"https://api.github.com/users/saeedseyfi/gists{/gist_id}","starred_url":"https://api.github.com/users/saeedseyfi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/saeedseyfi/subscriptions","organizations_url":"https://api.github.com/users/saeedseyfi/orgs","repos_url":"https://api.github.com/users/saeedseyfi/repos","events_url":"https://api.github.com/users/saeedseyfi/events{/privacy}","received_events_url":"https://api.github.com/users/saeedseyfi/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2016-10-29T21:42:03Z","updated_at":"2016-10-29T21:42:03Z","closed_at":null,"body":"Hi Ante,\r\nWhat do you think about adding RTL support to SleekCSS?\r\n\r\nI know three methods to handle it, please take a look and share your thoughts:\r\n1. [Handle it using sass mixins][1]\r\n  - pros: \r\n    1. Smaller output file in comparison to method `#2`\r\n    2. You'll have a semantic sass because of using `rtl` and `ltr` mixins\r\n  - cons:\r\n    1. You can't support both `rtl` and `ltr` both in a single output file\r\n\r\n2. [Handle it using `dir` attr of `html` tag][2]\r\n  - pros:\r\n    1. You don't need to compile your sass file twice (you can support `rtl` and `ltr` in a single file)\r\n  - cons:\r\n    1. Your output file might get heavy because of long `css` rules\r\n\r\n3. [Handle it using `sass` vars] [3]\r\n  - pros:\r\n    1. Smaller output file in comparison to both above methods\r\n  - cons:\r\n    1. You can't support both `rtl` and `ltr` both in a single output file\r\n    2. Your `sass` files are not very semantic like method `#1`\r\n\r\n\r\n  [1]: https://gist.github.com/saeedseyfi/4eb9f4634d5eb053da05375c59092da8\r\n  [2]: https://gist.github.com/saeedseyfi/368c339ecacc6d3d251d3ef13ffe2bc7\r\n  [3]: https://www.sitepoint.com/manage-rtl-css-sass-grunt/\r\n\r\n\r\nI can take over adding the RTL support to Sleek.\r\nPlease advise."}}
{"actor":{"display_login":"saeedseyfi"}}