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

testing merging #1

Merged
merged 19 commits into from
Jul 27, 2015
Merged

testing merging #1

merged 19 commits into from
Jul 27, 2015

Conversation

carl-alberto
Copy link
Owner

No description provided.

emiluzelac and others added 19 commits May 16, 2015 19:24
Minor GPL wording update to reflect `GPLv2 or later` what readme already does at the beginning :)
Update theme against WordPress Coding Standards.
Minor link update for languages.
To reflect most recent changes such as `template-parts`.
Updated Tagline markup as per W3C guidelines.
Update theme comment section so it will remain when compiled & compressed.
…ture for titles throughout theme:

In header.php: Site title set to H1 on front page, div on other pages
In functions.php: Widget title set to H2 for semantic hierarchy
In content-search.php and content.php: Heading set to H2 as page title is "Search" or "Archive title" etc.
… front and the posts page is also set to a page as per suggestion by @davidakennedy
Translator comments should be placed directly in the line above the string to be translated, in order for parser to pick it up.
As WordPress encourages to call their webpages with https-protocol, the credits link in the theme footer should use https, too.
Change Credit Link to https://
Change headings structure for better accessibility for assistive technology users.
carl-alberto added a commit that referenced this pull request Jul 27, 2015
@carl-alberto carl-alberto merged commit 322450d into carl-alberto:master Jul 27, 2015
carl-alberto pushed a commit that referenced this pull request May 27, 2016
Switches from livereload to browsersync
carl-alberto pushed a commit that referenced this pull request May 27, 2016
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.

10 participants