Skip to content

LMS: Developer-centric Sass Area - #2046

Merged
talbs merged 1 commit into
masterfrom
talbs/lms-devscss
Jan 3, 2014
Merged

LMS: Developer-centric Sass Area#2046
talbs merged 1 commit into
masterfrom
talbs/lms-devscss

Conversation

@talbs

@talbs talbs commented Dec 27, 2013

Copy link
Copy Markdown
Contributor

This work adds in a known area for developers to add in new Sass, especially when needed and the current styling/UI patterns cannot support the features and bugs they are actively working on. Benefits of this include:

  • a known area in the cascade where styles will be inherited
  • a designated area for developer use and design visual polishing and front end re-factoring (rather than hunting through the entire codebase).
  • a location for developers to learn, request post-production support, receive feedback on any front end work, and ask questions of designers.

@talbs

talbs commented Dec 27, 2013

Copy link
Copy Markdown
Contributor Author

@marcotuts and @sarina (or @adampalay), mind giving this unused, but hopefully helpful down the road area, your blessing?

@adampalay

Copy link
Copy Markdown
Contributor

cool. pending failing tests, 👍

@sarina

sarina commented Jan 3, 2014

Copy link
Copy Markdown
Contributor

👍 once tests pass

* provides new .scss file for developers to add temporary/in-progress Sass for further refactor/polish
* cleans up note/import order in all application scss files
talbs added a commit that referenced this pull request Jan 3, 2014
LMS: Developer-centric Sass Area
@talbs
talbs merged commit dd591a4 into master Jan 3, 2014
@talbs

talbs commented Jan 3, 2014

Copy link
Copy Markdown
Contributor Author

Bringing this to "Mergetown".

@talbs
talbs deleted the talbs/lms-devscss branch January 3, 2014 19:58
shimulch pushed a commit to open-craft/openedx-platform that referenced this pull request Jan 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants