Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Remove duplicate path from CSS manifest comment #14084

Closed
wants to merge 1 commit into from

4 participants

@leejarvis

No description provided.

@carlosantoniodasilva

In fact the description tries to explain that files in vendor/assets/stylesheets of the application itself, and vendor/assets/stylesheets of plugins, can all be referenced. So there seems to be a duplication but they relate to different things. Is that too confusing? Maybe we could make it more clear instead, if necessary.

@leejarvis

Hah right I see it now. Yeah I think this could be cleared up it wasn't clear on first glance

@senny
Owner

I think it's fine as is. If there are suggestion to make it more readable please submit a new PR. Dropping that part is not an option though.

@leejarvis thank you for your work.

@senny senny closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Feb 17, 2014
  1. @leejarvis
This page is out of date. Refresh to see the latest.
View
4 railties/lib/rails/generators/rails/app/templates/app/assets/stylesheets/application.css
@@ -2,8 +2,8 @@
* This is a manifest file that'll be compiled into application.css, which will include all the files
* listed below.
*
- * Any CSS and SCSS file within this directory, lib/assets/stylesheets, vendor/assets/stylesheets,
- * or vendor/assets/stylesheets of plugins, if any, can be referenced here using a relative path.
+ * Any CSS and SCSS file within this directory, lib/assets/stylesheets, or vendor/assets/stylesheets,
+ * of plugins, if any, can be referenced here using a relative path.
*
* You're free to add application-wide styles to this file and they'll appear at the bottom of the
* compiled file so the styles you add here take precedence over styles defined in any styles
Something went wrong with that request. Please try again.