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

VACMS-19794: Removes Connect with us tech debt #20303

Merged
merged 4 commits into from
Jan 21, 2025

Conversation

omahane
Copy link
Contributor

@omahane omahane commented Jan 17, 2025

Description

Relates to #19794

Testing done

Cypress
Manual

Screenshots

There should be no discernible difference, now that these hidden fields are removed.

QA steps

The following should look the same:

Content Prod Tugboat
Office Edit Screenshot 2024-11-27 at 10 57 04 AM Screenshot 2024-11-27 at 10 57 04 AM
CLP Edit Screenshot 2024-11-27 at 10 56 56 AM Screenshot 2024-11-27 at 10 56 56 AM
BHLP Edit Screenshot 2024-11-27 at 10 56 47 AM Screenshot 2024-11-27 at 10 56 47 AM
Office View Screenshot 2024-11-27 at 11 24 56 AM Screenshot 2024-11-27 at 11 24 56 AM
CLP View Screenshot 2025-01-21 at 09 33 38 Screenshot 2025-01-21 at 09 33 38
BHLP View Screenshot 2025-01-21 at 09 36 31 Screenshot 2025-01-21 at 09 36 31

Front-end

Back-end

Select Team for PR review

  • CMS Team
  • Public websites
  • Facilities
  • User support
  • Accelerated Publishing

@omahane omahane requested review from a team as code owners January 17, 2025 22:04
@va-cms-bot va-cms-bot temporarily deployed to Tugboat January 17, 2025 22:04 Destroyed
Copy link

Checking composer.lock changes...

@va-cms-bot va-cms-bot temporarily deployed to Tugboat January 17, 2025 22:22 Destroyed
Copy link

Checking composer.lock changes...

@va-cms-bot va-cms-bot temporarily deployed to Tugboat January 18, 2025 08:44 Destroyed
@va-cms-bot va-cms-bot temporarily deployed to Tugboat January 19, 2025 08:42 Destroyed
@va-cms-bot va-cms-bot temporarily deployed to Tugboat January 20, 2025 08:51 Destroyed
@va-cms-bot va-cms-bot temporarily deployed to Tugboat January 21, 2025 08:43 Destroyed
@va-cms-bot va-cms-bot temporarily deployed to Tugboat January 21, 2025 14:20 Destroyed
@laflannery
Copy link
Contributor

Just in case it matters in the future, the Twitter links have since been updated form when these screenshots where taken and are now X, just wanted to note this:
Screenshot 2025-01-21 at 9 48 38 AM

Copy link
Contributor

@laflannery laflannery left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@github-actions github-actions bot added the Facilities Facilities products (VAMC, Vet Center, etc) label Jan 21, 2025
Copy link
Contributor

@dsasser dsasser left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes look great:
Screenshot 2025-01-21 at 10 55 03 AM
Screenshot 2025-01-21 at 10 53 59 AM
Uploading Screenshot 2025-01-21 at 10.53.18 AM.png…

@omahane omahane enabled auto-merge (squash) January 21, 2025 19:34
@va-cms-bot va-cms-bot temporarily deployed to Tugboat January 21, 2025 19:34 Destroyed
Copy link

Checking composer.lock changes...

@va-cms-bot
Copy link
Collaborator

Cypress Accessibility Violations

/test-data-rerum

ID: button-name
Impact: critical
Tags: cat.name-role-value, wcag2a, wcag412, section508, section508.22.a, TTv5, TT6.a, EN-301-549, EN-9.4.1.2, ACT
Description: Ensure buttons have discernible text
Help: Buttons must have discernible text
Nodes:

  • HTML: <button class="proofing-element-help" role="tooltip" data-proofing-help-title="About 'Page introduction' field" data-proofing-help="Add an introduction that helps visitors understand if information on the page is relevant to them."> <span aria-hidden="true">i</span> </button>
    Impact: critical
    Target: .field--name-field-intro-text-limited-html > .field__label > .proofing-element-help[role="tooltip"]
    Summary: Fix any of the following:
    Element does not have inner text that is visible to screen readers
    aria-label attribute does not exist or is empty
    aria-labelledby attribute does not exist, references elements that do not exist or references elements that are empty
    Element has no title attribute
    Element does not have an implicit (wrapped) <label>
    Element does not have an explicit <label>
    Element's default semantics were not overridden with role="none" or role="presentation"

  • HTML: <button class="proofing-element-help" role="tooltip" data-proofing-help-title="About 'Generate a table of contents from major headings' field" data-proofing-help="By checking this box, all h2's below this point on the page will be linked with with anchor links. This helps users navigate content on very long pages. Do not check this box unless there is at least 2 h2's on the page.">
    Impact: critical
    Target: .field--name-field-table-of-contents-boolean > .field__label > .proofing-element-help[role="tooltip"]
    Summary: Fix any of the following:
    Element does not have inner text that is visible to screen readers
    aria-label attribute does not exist or is empty
    aria-labelledby attribute does not exist, references elements that do not exist or references elements that are empty
    Element has no title attribute
    Element does not have an implicit (wrapped) <label>
    Element does not have an explicit <label>
    Element's default semantics were not overridden with role="none" or role="presentation"

  • HTML: <button class="proofing-element-help" role="tooltip" data-proofing-help-title="About 'Main content' field" data-proofing-help="The main body of the page, which appears below the featured content."> <span aria-hidden="true">i</span> </button>
    Impact: critical
    Target: button[data-proofing-help-title="About 'Main content' field"]
    Summary: Fix any of the following:
    Element does not have inner text that is visible to screen readers
    aria-label attribute does not exist or is empty
    aria-labelledby attribute does not exist, references elements that do not exist or references elements that are empty
    Element has no title attribute
    Element does not have an implicit (wrapped) <label>
    Element does not have an explicit <label>
    Element's default semantics were not overridden with role="none" or role="presentation"

@omahane omahane merged commit a492a40 into main Jan 21, 2025
18 checks passed
@omahane omahane deleted the VACMS-19794-connect-with-us-tech-debt branch January 21, 2025 20:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Facilities Facilities products (VAMC, Vet Center, etc)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants