reactjs / reactjs/react-tabs

Use anchor tags as Tab links

Open
#250 5 comments 13 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
JavaScript
Stars
3.1k
Forks
455
Avg merge
5h 9m
Merged PRs (30d)
2

Description

I think it'd be better for semantics to use <a> tags instead of <li> tags for the Tab buttons.
This article reminded me of this, stating that

If it (the button) goes to a different URL or changes most of the page’s content, use a link ()

I reckon a tab usually does change most of the page's content, so we should use anchor tags.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating the component code that renders the tabs and inspect how the current

  • elements act as buttons. Update the tab markup to use elements, then verify that the existing tab behavior remains intact and that the rendered tabs follow the requested semantics.

  • Written by the indexing model from the issue text.

    Assessment

    Tech stack
    javascript, react
    Domain
    accessibility, frontend
    Issue type
    Feature
    Difficulty
    4/5
    Estimated time
    3-5 days
    Activity status
    Stale
    Clarity
    Mostly clear
    Newbie friendliness
    35/100

    Get new issues in your inbox

    A short digest of beginner-friendly GitHub issues.