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.

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

Clarify CSS structure #711

Open
niccokunzmann opened this issue Feb 5, 2018 · 7 comments
Open

Clarify CSS structure #711

niccokunzmann opened this issue Feb 5, 2018 · 7 comments

Comments

@niccokunzmann
Copy link
Member

niccokunzmann commented Feb 5, 2018

Expected results

For each page, I would like to know:

  • which file do I have to edit to change this page
  • without breaking other pages

Actual results

  • not clear which file is used where, unclear names like "custom.css"
  • CSS is replaced by JavaScript

Implementation/Solution Ideas

I would like to have

  • one css file per page
  • one css file with the proposed design from the UI guidelines

Related

@niccokunzmann niccokunzmann changed the title Change CSS Clarify CSS structure Feb 5, 2018
@nzec
Copy link
Contributor

nzec commented Feb 5, 2018

Don't have any time right now! However, I feel really strongly about this, If anyone doesn't take this up in ~5 days. I'll work on it.

@himanshuc3
Copy link
Contributor

I would like to refactor and clean up.

@nzec
Copy link
Contributor

nzec commented Feb 7, 2018

@himanshuc3 Sure! Go ahead!

@nzec
Copy link
Contributor

nzec commented Mar 15, 2018

Damn. This is gonna be quite a task. Will work on it @himanshuc3 is not.

@samyakbvs
Copy link
Contributor

I would like to work on it :) @niccokunzmann @duttaditya18

@nzec
Copy link
Contributor

nzec commented Sep 16, 2018

@samyakbvs Sure. Go ahead!

@niccokunzmann
Copy link
Member Author

niccokunzmann commented Sep 16, 2018 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants