Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Beta] Applied Accessibility - Tests fails to check for closing </figure> tag #16420

Closed
Asjas opened this issue Jan 6, 2018 · 3 comments
Closed

[Beta] Applied Accessibility - Tests fails to check for closing </figure> tag #16420

Asjas opened this issue Jan 6, 2018 · 3 comments
Labels
help wanted Open for all. You do not need permission to work on these.

Comments

@Asjas
Copy link
Contributor

Asjas commented Jan 6, 2018 •
edited

Challenge Name

https://beta.freecodecamp.org/en/challenges/applied-accessibility/improve-chart-accessibility-with-the-figure-element

Issue Description

I've found two problems in this challenge.

Firstly, it states no div tags may be used in the code. But if I leave in a closing </div> tag the test still passes. It's not checking for closing div tags.

Second, the test isn't checking if the closing </figure> tag is being used. It seems to be only checking for the opening tag and then passes.

Browser Information

  • Browser Name, Version: Chrome, Version 63.0.3239.108 (Official Build) (64-bit)
  • Operating System: Windows 10 Pro
  • Mobile, Desktop, or Tablet: Desktop

Your Code

<body>
  <header>
    <h1>Training</h1>
    <nav>
      <ul>
        <li><a href="#stealth">Stealth &amp; Agility</a></li>
        <li><a href="#combat">Combat</a></li>
        <li><a href="#weapons">Weapons</a></li>
      </ul>
    </nav>
  </header>
  <main>
    <section>
      
      <!-- Add your code below this line -->
      <figure>
        <!-- Stacked bar chart will go here -->
        <br>
        <figcaption>Breakdown per week of time to spend training in stealth, combat, and weapons.</figcaption>
        </div>
      <!-- Add your code above this line -->
      
    </section>
    <section id="stealth">
      <h2>Stealth &amp; Agility Training</h2>
      <article><h3>Climb foliage quickly using a minimum spanning tree approach</h3></article>
      <article><h3>No training is NP-complete without parkour</h3></article>
    </section>
    <section id="combat">
      <h2>Combat Training</h2>
      <article><h3>Dispatch multiple enemies with multithreaded tactics</h3></article>
      <article><h3>Goodbye world: 5 proven ways to knock out an opponent</h3></article>
    </section>
    <section id="weapons">
      <h2>Weapons Training</h2>
      <article><h3>Swords: the best tool to literally divide and conquer</h3></article>
      <article><h3>Breadth-first or depth-first in multi-weapon training?</h3></article>
    </section>
  </main>
  <footer>&copy; 2016 Camper Cat</footer>
</body>

Screenshot

figure
@Asjas Asjas changed the title [Beta] Applied Accessibility - Challenge isn't checking for closing </figure> tag [Beta] Applied Accessibility - Tests fails to check for closing </figure> tag Jan 6, 2018
@raisedadead raisedadead added beta help wanted Open for all. You do not need permission to work on these. labels Jan 6, 2018
@sagarjhaa
Copy link
Contributor

@raisedadead I would like to work on it.

@raisedadead
Copy link
Member

Yes, sure go ahead, any issue that is tagged help wanted, and does not have a open PR is open for contribution. You do not need confirmation.

Goodluck and happy contributing!

@sagarjhaa
Copy link
Contributor

Working on it.

anhtran1906 added a commit to fccSeniorSeminar/freeCodeCamp that referenced this issue Jan 26, 2018
@anhtran1906
fix(seed): Add test cases for closing tags
036f6cf 
Added test cases with assert statements

BREAKING CHANGE: no

Closes freeCodeCamp#16415
Closes freeCodeCamp#16424
Closes freeCodeCamp#16423
Closes freeCodeCamp#16422
Closes freeCodeCamp#16420
Closes freeCodeCamp#16419
Closes freeCodeCamp#16417
Closes freeCodeCamp#16414
@anhtran1906 anhtran1906 mentioned this issue Jan 26, 2018
Merged
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted Open for all. You do not need permission to work on these.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@raisedadead @Asjas @sagarjhaa