{"action":"created","issue":{"url":"https://api.github.com/repos/fatdude/jquery-ui-themes-rails/issues/10","labels_url":"https://api.github.com/repos/fatdude/jquery-ui-themes-rails/issues/10/labels{/name}","comments_url":"https://api.github.com/repos/fatdude/jquery-ui-themes-rails/issues/10/comments","events_url":"https://api.github.com/repos/fatdude/jquery-ui-themes-rails/issues/10/events","html_url":"https://github.com/fatdude/jquery-ui-themes-rails/pull/10","id":82365431,"number":10,"title":"The imported assets goes into the vendor folder.","user":{"login":"mperrando","id":525572,"avatar_url":"https://avatars.githubusercontent.com/u/525572?v=3","gravatar_id":"","url":"https://api.github.com/users/mperrando","html_url":"https://github.com/mperrando","followers_url":"https://api.github.com/users/mperrando/followers","following_url":"https://api.github.com/users/mperrando/following{/other_user}","gists_url":"https://api.github.com/users/mperrando/gists{/gist_id}","starred_url":"https://api.github.com/users/mperrando/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mperrando/subscriptions","organizations_url":"https://api.github.com/users/mperrando/orgs","repos_url":"https://api.github.com/users/mperrando/repos","events_url":"https://api.github.com/users/mperrando/events{/privacy}","received_events_url":"https://api.github.com/users/mperrando/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-05-29T10:13:37Z","updated_at":"2015-05-29T12:42:04Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/fatdude/jquery-ui-themes-rails/pulls/10","html_url":"https://github.com/fatdude/jquery-ui-themes-rails/pull/10","diff_url":"https://github.com/fatdude/jquery-ui-themes-rails/pull/10.diff","patch_url":"https://github.com/fatdude/jquery-ui-themes-rails/pull/10.patch"},"body":"The imported files are put into the vendor asset folder for stylesheets, as recommended for third party material.\r\nThis method has the advantage of keeping the theme images isolated in their own folder, even in the production environment, also avoiding possible name conflicts.\r\n\r\nEverything is put inside a folder named ```jquery-ui/<name>```: both the theme main file and the theme images.\r\n\r\nThe images are put inside an ```images``` subfolder, while the theme main file name is kept ```theme.css.scss```, as it is in the jquery-ui distribution.\r\n\r\nAfter importing, you need to\r\n\r\n1. add the images folder to the assets precompile in the assets initializer\r\n2. add to the ```application.css``` (or whatever you named it) file the include directive to the theme main file."},"comment":{"url":"https://api.github.com/repos/fatdude/jquery-ui-themes-rails/issues/comments/106792851","html_url":"https://github.com/fatdude/jquery-ui-themes-rails/pull/10#issuecomment-106792851","issue_url":"https://api.github.com/repos/fatdude/jquery-ui-themes-rails/issues/10","id":106792851,"user":{"login":"mperrando","id":525572,"avatar_url":"https://avatars.githubusercontent.com/u/525572?v=3","gravatar_id":"","url":"https://api.github.com/users/mperrando","html_url":"https://github.com/mperrando","followers_url":"https://api.github.com/users/mperrando/followers","following_url":"https://api.github.com/users/mperrando/following{/other_user}","gists_url":"https://api.github.com/users/mperrando/gists{/gist_id}","starred_url":"https://api.github.com/users/mperrando/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mperrando/subscriptions","organizations_url":"https://api.github.com/users/mperrando/orgs","repos_url":"https://api.github.com/users/mperrando/repos","events_url":"https://api.github.com/users/mperrando/events{/privacy}","received_events_url":"https://api.github.com/users/mperrando/received_events","type":"User","site_admin":false},"created_at":"2015-05-29T12:42:04Z","updated_at":"2015-05-29T12:42:04Z","body":"We could create an images folder under vendor/assets, but it would violate the provided structure that does not include it (do not know why).\r\n\r\nBesides, including the images in the application assets images folder does not sound so good.\r\n\r\nI've spent some time figuring out which is the best position and which is the right directive for the assets precompiler, because I don't like solutions like [this one](https://stackoverflow.com/a/14195512).\r\n\r\nI'm glad you've found the outcome acceptable!\r\n\r\nI am posting another PR merged with ad7e90aa15227dd7cebfd515f873e46ff840a8d0 (just merged in my own repo)."}}