-
Notifications
You must be signed in to change notification settings - Fork 12
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
Move to HTML5 #32
Comments
Yeah sure. It's probably just a matter of changing the doctype and running it through the validator a few times. Can you elaborate on what you mean by using class="" to make theming easier? |
If you use class="" lots, it makes CSS theming easier by having simple 'classes' to theme. In the HTML. e.g.: <div class="diagram">content here</div> In the CSS e.g.: .diagram {
/* Let's use pink text to be odd. :) */
color: pink;
} Is that what you wanted elaborated on? :) |
Ok cool I understand CSS classes, and alloc uses lots of them, no dramas there. Re theming (and believe me I am open to new approaches eg SASS, LESS whatever), but the way the current themes work is that there is a file css/src/style.tpl which is a template that contains placeholders for the different themes, eg:
And when So when CSS modifications are required the only files that should be manually edited by a human are in css/src/. |
OK, cool. I've never used SASS or LESS, but they may be worth a look I guess. :) |
I feel the best path forward for this is first move our css to SASS. Then port one module at a time to HTML5, e.g. the login page, then the task module, etc, etc. |
Heh, I think you might be biting off more than you imagine. |
That is a very good point. My reasoning to shift to a 3rd party template system (not sure if I'll do this or not, but putting the reasoning here anyway) is:
Another problem that I noticed when working on PR #49 is how the html is mixed between templates and PHP files. Figuring out where I had to make the change took me about 1hr, that can be quite off putting for new contributors. Regardless of whether we use a 3rd party template engine or not, we should try to keep the main PHP code and html templates as separate as possible with the switch to html5 Oh, the other thing, I looked into SASS and LESS. Personally I prefer the way SASS works, are you in favour of using SASS or LESS? Both Debian and Fedora have |
All good - totally reasonable reasons. Re CSS compiler, I'm happy to defer to your expertise. |
Currently there are issues with the html that gets generated. It would be good to move to HTML5 and make sure we are following the w3 specs.
When doing this, we should also make extensive use of class="" to make CSS theming easier.
The text was updated successfully, but these errors were encountered: