Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Application.css needs a way to specify order #1353

Closed
melvinram opened this issue May 27, 2011 · 3 comments
Closed

Application.css needs a way to specify order #1353

melvinram opened this issue May 27, 2011 · 3 comments

Comments

@melvinram
Copy link

When dealing with application.css, I'm getting an overlap. This is what my file looks like:

/*
 * This is a manifest file that'll automatically include all the stylesheets available in this directory
 * and any sub-directories. You're free to add application-wide styles to this file and they'll appear at
 * the top of the compiled file, but it's generally better to create a new file per style scope.
 *= require_self
 *= require reset
 *= require general
 *= require_tree . 
*/

and it adds the reset and general file twice. Ideally, Rails would look at which files have already been required and would ignore those files when doing *= require_tree .
@TheEmpty
Copy link
Contributor

This should be reported in sprockets, the rails tie that handles this cc @josh

@melvinram
Copy link
Author

Gotcha. I added it as an issue on there as well: sstephenson/sprockets#65

jake3030 pushed a commit to jake3030/rails that referenced this issue Jun 28, 2011
@mislav @NZKoz
auto_link helper: add intelligent ending closing bracket handling. ad…
4f984c9 
…d new tests and reorder new ones for readability

Signed-off-by: Michael Koziarski <michael@koziarski.com>
[rails#1353 state:committed]
@dmathieu
Copy link
Contributor

dmathieu commented Jul 6, 2011

This is a sprockets issue, not belonging to rails and it correctly exists there. It could be closed - cc @spastorino

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@dmathieu @melvinram @spastorino @TheEmpty