USACE / USACE/groundwork

Various issues with sidebar

Open
#99 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
JavaScript
Stars
6
Forks
4
PR merge metrics
No merged PRs in 30d

Description

1. Adding the sidebar from 2.7.1 causes the Home to mash into the usace box. Possible issues with CSS
PR

  • #100 addresses item 1
  1. If you spam click the Home button on the sidebar it will keep appending the base path, this was because we had the href set to # - if we changed it to / this resolved the issue

  2. For 508 compliance we probably should not have a, remove home button on the home page

Contributor guide

Open the contributing guide

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 reproducing the sidebar Home-button behavior described in items 2 and 3. Inspect the sidebar implementation and its Home link, then verify that repeated clicks do not append the base path and that the Home button is removed from the home page for 508 compliance. PR #100 already addresses item 1.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript, react
Domain
accessibility, frontend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.