-
Notifications
You must be signed in to change notification settings - Fork 60k
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
Tiny headline adjustment for the Issue event types page #2004
Conversation
Thanks for opening this pull request! A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines. |
@abjerner Thanks for opening a PR 👍 Please fill out the pull request template in the top comment in this pull request, then we'll get this up for review! |
@janiceilene I've updated the PR now. I wasn't sure about check list, but makes more sense now that the tests have completed. |
Thanks so much @abjerner! I'll get this triaged for review ⚡ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for fixing this, @abjerner!
Thanks very much for contributing! Your pull request has been merged 🎉 You should see your changes appear on the site in approximately 24 hours. |
Why:
The Issue event types page has a number of Event object properties headlines. All but one are H4s, as a single is a H3 instead.
As a H3 gets a link in the page navigation to the right, I thought I would change that to a H4 as well 😉
Since this is such a tiny change, I haven't created an issue.
Check off the following: