{"action":"created","issue":{"url":"https://api.github.com/repos/mathjax/MathJax-third-party-extensions/issues/20","labels_url":"https://api.github.com/repos/mathjax/MathJax-third-party-extensions/issues/20/labels{/name}","comments_url":"https://api.github.com/repos/mathjax/MathJax-third-party-extensions/issues/20/comments","events_url":"https://api.github.com/repos/mathjax/MathJax-third-party-extensions/issues/20/events","html_url":"https://github.com/mathjax/MathJax-third-party-extensions/pull/20","id":123658315,"number":20,"title":"(WIP) Adding Arabic-specific RTL to MathJax","user":{"login":"OmarIthawi","id":645156,"avatar_url":"https://avatars.githubusercontent.com/u/645156?v=3","gravatar_id":"","url":"https://api.github.com/users/OmarIthawi","html_url":"https://github.com/OmarIthawi","followers_url":"https://api.github.com/users/OmarIthawi/followers","following_url":"https://api.github.com/users/OmarIthawi/following{/other_user}","gists_url":"https://api.github.com/users/OmarIthawi/gists{/gist_id}","starred_url":"https://api.github.com/users/OmarIthawi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/OmarIthawi/subscriptions","organizations_url":"https://api.github.com/users/OmarIthawi/orgs","repos_url":"https://api.github.com/users/OmarIthawi/repos","events_url":"https://api.github.com/users/OmarIthawi/events{/privacy}","received_events_url":"https://api.github.com/users/OmarIthawi/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-12-23T13:20:48Z","updated_at":"2016-01-10T10:45:32Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/mathjax/MathJax-third-party-extensions/pulls/20","html_url":"https://github.com/mathjax/MathJax-third-party-extensions/pull/20","diff_url":"https://github.com/mathjax/MathJax-third-party-extensions/pull/20.diff","patch_url":"https://github.com/mathjax/MathJax-third-party-extensions/pull/20.patch"},"body":"This pull request will be updated to get the Arabic MathJax solution from the [original hack](https://github.com/Edraak/MathJax/blob/bfb43be34c2572e8c356d8934a2fdd77de3926a7/README.md).\r\n\r\nIn this pull request the following will be done:\r\n\r\n - [x] Convert the hack into a reusable `Contrib` extension\r\n\r\n - [x] Split into three (or more) modules\r\n   - Making the TeX jax RTL aware and allow defining lang-dependent commands\r\n   - Making the HTML-CSS jax renders RTL well\r\n   - Add Math-specific language-depedents units to the TeX\r\n   - Add Physics-specific language-depedents units to the TeX\r\n\r\n - [ ] Make the language detection mechanism configurable for the `\\ar` command. \r\n        Note: Now it is hard-coded in the the lang attribute of the <html> tag.\r\n\r\nI will not try to make a one-size fits all RTL solution, instead I will try to do the following:\r\n\r\n - Essential RTL support for Arabic\r\n - Arabic support through custom font (basic font support for now)\r\n - Localized physics units, variables, constants and others\r\n - Make the solution as modular as possible to make future expansions for other language possible"},"comment":{"url":"https://api.github.com/repos/mathjax/MathJax-third-party-extensions/issues/comments/170333489","html_url":"https://github.com/mathjax/MathJax-third-party-extensions/pull/20#issuecomment-170333489","issue_url":"https://api.github.com/repos/mathjax/MathJax-third-party-extensions/issues/20","id":170333489,"user":{"login":"OmarIthawi","id":645156,"avatar_url":"https://avatars.githubusercontent.com/u/645156?v=3","gravatar_id":"","url":"https://api.github.com/users/OmarIthawi","html_url":"https://github.com/OmarIthawi","followers_url":"https://api.github.com/users/OmarIthawi/followers","following_url":"https://api.github.com/users/OmarIthawi/following{/other_user}","gists_url":"https://api.github.com/users/OmarIthawi/gists{/gist_id}","starred_url":"https://api.github.com/users/OmarIthawi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/OmarIthawi/subscriptions","organizations_url":"https://api.github.com/users/OmarIthawi/orgs","repos_url":"https://api.github.com/users/OmarIthawi/repos","events_url":"https://api.github.com/users/OmarIthawi/events{/privacy}","received_events_url":"https://api.github.com/users/OmarIthawi/received_events","type":"User","site_admin":false},"created_at":"2016-01-10T10:45:32Z","updated_at":"2016-01-10T10:45:32Z","body":"Hi @pkra I've make some progress and modularized my extension. However I'm using functions as modules. Do you think it's wise to use Startup hooks and events to declare modules dependency in extensions or is that incorrect?\r\n\r\nMy goal is to put each module in a separate file and perhaps `$ cat arabic/unpacked/*.js > arabic/arabic.js` then `uglify` it.\r\n\r\nI'm not a big fan of huge JS files. \r\n\r\nWhat do you think?"}}