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

Section to div #172

Merged
merged 1 commit into from
Jan 27, 2017
Merged

Section to div #172

merged 1 commit into from
Jan 27, 2017

Conversation

zeroasterisk
Copy link
Contributor

Switched from section --> div in ALL components

Do not use the

element as a generic container; this is what
is for, especially when the sectioning is only for styling purposes. A rule of thumb is that a section should logically appear in the outline of a document.

https://developer.mozilla.org/en-US/docs/Web/HTML/Element/section

@radekmie radekmie added the Type: Feature New features and feature requests label Jan 24, 2017
@radekmie
Copy link
Contributor

OK, I will merge it after #170.

@radekmie
Copy link
Contributor

@zeroasterisk, there are conflicts, due to #170 - I'll merge it after you'll solve them.

@radekmie
Copy link
Contributor

@zeroasterisk will you have some time to do this?

> Do not use the <section> element as a generic container; this is what <div> is for, especially when the sectioning is only for styling purposes. A rule of thumb is that a section should logically appear in the outline of a document.

https://developer.mozilla.org/en-US/docs/Web/HTML/Element/section
@zeroasterisk
Copy link
Contributor Author

Ok - I re-ran this sed after rebasing from master -- should be ok now.

@radekmie radekmie merged commit be7370d into vazco:master Jan 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Feature New features and feature requests
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants