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

Allow description meta tag to be specified separately from the role. #696

Merged
merged 3 commits into from Sep 18, 2018

Conversation

Projects
None yet
2 participants
@droy
Contributor

droy commented Sep 18, 2018

If it is set, the 'meta_description' Param now used instead of 'role' to set the description meta tag.

Purpose

Having just the role in the description meta tag does not make sense for optimizing search results. This approach provides the user the ability to easily set the tag in the config.toml file.

droy and others added some commits Sep 18, 2018

Allow description meta tag to be specified separately from the role i…
…n Params. Role still used if meta_description not set.

@gcushen gcushen merged commit 9c0d14c into gcushen:master Sep 18, 2018

@gcushen

This comment has been minimized.

Show comment
Hide comment
@gcushen

gcushen Sep 18, 2018

Owner

Thanks @droy !

Owner

gcushen commented Sep 18, 2018

Thanks @droy !

ltiao added a commit to ltiao/hugo-academic that referenced this pull request Oct 10, 2018

Merge remote-tracking branch 'upstream/master'
* upstream/master: (39 commits)
  Update Readme
  i18n: Update Chinese translation (gcushen#717)
  posts: Add featured image option and stream layout for widget
  Add experience widget
  shortcodes: Update figure and video to support files within page dir
  CSS: Prevent Hugo encoding Google Fonts and fix docs sidebar toggle bg
  docs layout: Fix responsiveness of sidebar over range of resolutions
  posts: Make using content template (e.g. hugo new) optional
  project widget: Make filter toolbar responsive
  Support custom site description and thumbnail image for SEO and sharing (gcushen#696)
  docs layout: Fix case where side nav links can overlap
  Revise table of contents for docs layout and top navbar
  Add slides feature
  about widget: Refactor avatar (gcushen#683)
  publication: Add optional DOI link (gcushen#677)
  JS: Fix Algolia template and refactor Hero
  Refactor most absolute URLs to relative URLs
  CSS: Fix Bootstrap 4 grid system for Safari
  docs layout: Support nested docs folders
  i18n: Add Catalan translation (gcushen#676)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment