Skip to content

Use Heading-4 for PR template for screen-readers. #1999

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Aug 27, 2024
Merged

Use Heading-4 for PR template for screen-readers. #1999

merged 1 commit into from
Aug 27, 2024

Conversation

tim-schilling
Copy link
Member

Description

To better integrate with GitHub's PR page, our PR template should result in h4 elements rather than h1.

See https://marijkeluttekes.dev/blog/articles/2024/08/19/quick-tip-use-h4-in-github-issue-descriptions/

Checklist:

  • I have added the relevant tests for this change.
  • I have added an item to the Pending section of docs/changes.rst.

To better integrate with GitHub's PR page, our PR template
should result in h4 elements rather than h1.

See https://marijkeluttekes.dev/blog/articles/2024/08/19/quick-tip-use-h4-in-github-issue-descriptions/
@matthiask matthiask merged commit 423ad36 into django-commons:main Aug 27, 2024
25 checks passed
@matthiask
Copy link
Member

Excellent :)

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants