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.

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

HTML Report Include Hidden #2026

Closed
Tracked by #2019
nam-singh opened this issue Sep 5, 2024 · 4 comments · Fixed by #2034 or #2056
Closed
Tracked by #2019

HTML Report Include Hidden #2026

nam-singh opened this issue Sep 5, 2024 · 4 comments · Fixed by #2034 or #2056
Assignees

Comments

@nam-singh
Copy link
Contributor

nam-singh commented Sep 5, 2024

  • Work on Hidden filter

Export the hidden data from extension

Filter addition for hidden

Adding hidden icon for hidden issues

  • Daily Review

Image

@philljenkins
Copy link
Contributor

philljenkins commented Sep 5, 2024

I noticed that 2 links are broken in the HTML report:

  1. Quick unit test for accessibility should change to Quick unit test for developers and link to the new place at https://www.ibm.com/able/toolkit/develop/overview/#unit-testing. Also change "...for accessibility" to "...for developers" in the link text.
  2. Full accessibility test process should be changed to link to the new place at https://www.ibm.com/able/toolkit/verify/overview

Both pages had been moved in the Toolkit, but links were not updated in the Checker.

Screenshot 2024-09-05 at 12 38 02 PM

@philljenkins
Copy link
Contributor

hmm, actually, this may be working correctly,
I was reviewing the following file https://github.com/IBMa/equal-access/blob/master/common/module/test/report/report1_expected.html which has the bad links.

I'm not sure if this is an "old left over" file in the repo, but it doesn't match the actual results when testing a web page and exporting the results.
When a user exports using v3.1.75, the resulting .html file has "good" links

@nam-singh nam-singh changed the title HTML Report Sprint 4 HTML Report Include Hidden Sep 9, 2024
@ErickRenteria ErickRenteria mentioned this issue Sep 10, 2024
10 tasks
@drjoho drjoho linked a pull request Sep 12, 2024 that will close this issue
5 tasks
@nam-singh
Copy link
Contributor Author

nam-singh commented Sep 12, 2024

The above link issue is been addressed in the recent PR #2034 . The links are working as expected in HTML report

ErickRenteria added a commit that referenced this issue Sep 13, 2024
feature(extension): HTML Report Hidden Filter Addition #2026
@ErickRenteria
Copy link
Contributor

Validated

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment