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

Fix typo on <ul> closing tag #21

Merged
merged 1 commit into from
Oct 1, 2020
Merged

Fix typo on <ul> closing tag #21

merged 1 commit into from
Oct 1, 2020

Conversation

tannerdolby
Copy link
Contributor

Hey, Bryan. I came across a typo while reading Using Nunjucks 'If Expressions' in 11ty to create a simple active navigation state.

The <ul> element is closed with a </div> rather than the expected </ul> closing tag. Lines 60-65.

I added the correct unordered list closing tag to fix the typo. Thanks! @brob

@netlify
Copy link

netlify bot commented Oct 1, 2020

Deploy request for bryanlrobinson pending review.

Review with commit 7efb8a0

https://app.netlify.com/sites/bryanlrobinson/deploys

@brob brob merged commit 76e33eb into brob:master Oct 1, 2020
@brob
Copy link
Owner

brob commented Oct 1, 2020

Great catch! Thanks for finding and fixing

@tannerdolby tannerdolby deleted the pr/fixTypo branch October 1, 2020 19:37
@tannerdolby
Copy link
Contributor Author

No problem! 👍

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.

None yet

2 participants