nextcloud / nextcloud/collectives
Replace h2 in empty content with h1
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 195
- Forks
- 51
- Avg merge
- 1d 2h
- Merged PRs (30d)
- 48
Description
On the initial landing page there is no heading of level one (h1). The landing page heading is h2 at the moment.
For good accessibility with screen readers it's recommended to not skip heading levels and therefore start with an h1:
it is important to not skip one or more heading levels. Doing so may create confusion, as the person navigating this way may be left wondering where the missing heading is.
Screenshot ("Collectives" is the h2):
Originally posted by @mspae in https://github.com/nextcloud/collectives/issues/159#issuecomment-1381781103
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
No file or test is named. Start at the initial landing page and inspect the heading currently labelled “Collectives”; update the heading hierarchy so the page begins with an h1, then verify the rendered markup and screen-reader heading order.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- accessibility, frontend
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 42/100