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

Ignore GSC verification in pa11y, change btn type to submit for availability form #212

Closed
wants to merge 5 commits into from
Closed

Ignore GSC verification in pa11y, change btn type to submit for availability form #212

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
40a26b2
Ignore GSC verification in pa11y
rachidatecs Jan 29, 2024
60a31a7
Fix form JS
rachidatecs Jan 29, 2024
367d966
Remove GSC verification file, refactor scss for availability form, up…
rachidatecs Jan 29, 2024
f65c81d
Merge remote-tracking branch 'origin/main' into accessibility-fixes
rachidatecs Jan 29, 2024
651c35f
Change pra anchor to span
rachidatecs Jan 29, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions .pa11yci
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,9 @@
"urls": [
"http://localhost:8080"
],
"ignore": [
"googlea7b9a2b812e0d81e.html"
],
"defaults": {
"standard": "WCAG2AA",
"runners": ["htmlcs"],
Expand Down
15 changes: 4 additions & 11 deletions _includes/domain-search.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,9 @@

<script>
function checkDomainAvailability() {
// Cancel the default action
event.preventDefault();

const baseUrl = "https://manage.get.gov/api/v1/available/?domain="

var requested_domain = document.getElementById('domain-input').value
Expand Down Expand Up @@ -56,16 +59,6 @@
function render_result(content, append = true){
document.getElementById('domain-available-message').innerHTML = (append == true) ? content : `<div></div>`;
}

document.addEventListener("keypress", function(event) {
// If the user presses the "Enter" key on the keyboard
if (event.key === "Enter") {
// Cancel the default action
event.preventDefault();
// Click the "check availability" button
document.getElementById("check-domain-availability-button").click();
}
});

</script>

Expand Down Expand Up @@ -103,7 +96,7 @@ <h2>
<button
id="check-domain-availability-button"
class="usa-button availability-button"
type="button"
type="submit"
Copy link
Contributor

Choose a reason for hiding this comment

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

When it was changed from button to submit, I think some of the padding and sizing got shifted as well.

What it's supposed to look like:
Screenshot 2024-01-29 at 8 17 12 AM

What it currently looks like (see left side of the button):
Screenshot 2024-01-29 at 8 15 30 AM

Copy link
Contributor

Choose a reason for hiding this comment

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

Needs to be moved margin-left: 1 rem .. looking into why the button edge is overflowing

Copy link
Contributor

Choose a reason for hiding this comment

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

Screenshot 2024-01-29 at 8 22 37 AM

These two border are culprit of the left edges being wonky

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Cool, on it

onclick="checkDomainAvailability()"
onsubmit="return false"
aria-label="Check availability of Domain Name"
Expand Down