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

Passing from unvalid XHTML to valid HTML5 in file tpl/page.html #18

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MrKooky
Copy link

@MrKooky MrKooky commented May 17, 2012

Nowadays HTML5 becomes an interesting approach of web, that's why I changed the XHTML code into W3C valid HTML5.
I also modified some CSS attributes that where not very usefull, like "div#id { style }". I changed them into "#id { style }".
Due to W3C, I also changed some "div" tags into "p".

A bientôt sur Blogosph@re ;)

@legionus
Copy link

Why did you change the permissions for all content from 644 to 755?

@MrKooky
Copy link
Author

MrKooky commented May 18, 2012

Oh, sorry it wasn't on purpose. But it doesn't matter that much, does it ?

@legionus
Copy link

Just sloppy. On unix systems, access rights are important.

@MrKooky
Copy link
Author

MrKooky commented May 19, 2012

Yeah, I'm aware of that. It's just that I could'nt work with lampp on the project without putting everything to 755.: )
I can take back my pull request, change the chmods and put it back on if it matters. ;)

@ghost ghost assigned sebsauvage Feb 26, 2013
@ghost
Copy link

ghost commented Dec 26, 2013

A long time ago, I made all this changes (well… in better), but also making a more advanced CSS and updating javascript code so comments are also more semantic and valid XHTML5.

I just commited this changes: #72 .

So that makes your patch obsolete :/

wware pushed a commit to wware/ZeroBin that referenced this pull request Sep 8, 2015
Updated README.md to explain password feature
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants