-
Notifications
You must be signed in to change notification settings - Fork 121
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
Add support for Google Analytics GA4 #67
Open
cloudlena
wants to merge
70
commits into
EmielH:master
Choose a base branch
from
cloudlena:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Change hardcoded language tag to be read from config file
.Site.Language is actually an object used for multilingual websites. The correct one to use is .Site.LanguageCode. See https://gohugo.io/variables/site/#site-variables-list Also included a default language code, and fixed an HTML syntax error.
This makes it easier to change or override parts of the theme on your site without losing updates to the theme.
Partial head.html was included in baseof.html, but also in header.html, causing the body to contain html and head tags.
When using a baseURL such as "/", the extra slash in this partial leads to an invalid href of "//".
Remove extraneous slash from header partial
Extract the summary into its own layout file and use the .Render function to include the summary, which is the same for all list templates.
Hugo provides an Author site variable that can be set in the site configuration. Instead of setting the author variable in the params, section from this commit on, you have to use the name variable in the Author section. This adheres more to Hugo standards. See README.MD for an example.
Page's .Next and .Prev are deprecated and will be removed in a future release. Replaced these with .NextPage and .PrevPage. Also removed the links to the previous and next page from the about template. This assumes there is only ever one about page.
Posts appeared next to each other instead of below each other when the title and summary were short enough. See EmielH#9
- Cleanup .travis.yml - Test on multiple versions of Hugo - Make Travis CI also test branches - Add HTMLProofer to validate HTML
Added dynamic header menu, Hugo-style Template from https://gohugo.io/templates/menu-templates Removed unused i18n See EmielH#29
Since EmielH#29, these strings are no longer used.
Move menu section to chapter on configuration
* Add code block long lines wrapping This change is for the code block style - allowing to wrap long lines.
The footer already used .Site.Author.name instead of .Site.Params.Author and now post info uses this variable as well. You can now safely delete .Site.Params.Author from your configuration. See EmielH#36
Tests will now only test the minimal required version for Tale and the newest version of Hugo.
Unlike the other links to favicon, this one was not passed through relURL. Co-authored-by: Lukacs T. Berki <blog@lberki.net>
Add table of contents See EmielH#64
Add dark mode See EmielH#65
@EmielH, can I help any further to get this merged? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #66