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

Removed default values for title, meta description, better labels. #6834

Merged
merged 1 commit into from
Mar 26, 2017
Merged

Removed default values for title, meta description, better labels. #6834

merged 1 commit into from
Mar 26, 2017

Conversation

paales
Copy link
Contributor

@paales paales commented Oct 1, 2016

  • default_description + default_keywords are removed because they do more harm than good.
  • default_title is not removed, since that field is a requirement to be set, better have a wrong value than no value at all
  • Better label for Title: Page Title (Title is ambiguous, could als imply the first h1)
  • Better label for description+keywords

- default_description + default_keywords are removed because they do more harm than good.
- default_title is not removed, since that field is a requirement to be set, better have a wrong value than no value at all
- Better label for Title: Page Title (Title is ambiguous, could als imply the first h1)
- Better label for description+keywords
@vrann vrann self-assigned this Mar 25, 2017
@vrann vrann added this to the March 2017 milestone Mar 25, 2017
@magento-team magento-team merged commit 528c842 into magento:develop Mar 26, 2017
@vrann
Copy link
Contributor

vrann commented Mar 26, 2017

@paales thank you, merged!

@paales paales deleted the bugfix/meta-description branch March 27, 2017 16:36
@paales paales added the Partner: Reach Digital Pull Request is created by partner Reach Digital label May 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Frontend bugfix Partner: Reach Digital Pull Request is created by partner Reach Digital
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants