Skip to content

Remove jQuery from Techniques#4981

Open
fstrr wants to merge 12 commits intomainfrom
remove-jquery-from-working-examples
Open

Remove jQuery from Techniques#4981
fstrr wants to merge 12 commits intomainfrom
remove-jquery-from-working-examples

Conversation

@fstrr
Copy link
Copy Markdown
Contributor

@fstrr fstrr commented Mar 7, 2026

ARIA18

Updated HTML snippet to use updated working example code

Working example:

  1. remove jquery
  2. add viewport
  3. update css to use modern positioning
  4. add inert attribute

ARIA 19

  1. Removed jQuery
  2. Add autocomplete attributes
  3. Removed 404 resource link
  4. Removed old editors CSS link
  5. Added code tags
  6. Updated the list of errors to use a list instead of 3 paragraphs

ARIA21

  1. Remove outdated content
  2. Update examples to add autocomplete attributes
  3. Updated resource links
  4. Split the example code into separate pre blocks

Identifying Errors In Data Format example

  1. removed the jQuery and rewrote using vanilla JavaScript
  2. removed several unused functions
  3. simplified the date validity field
  4. added a replacement image for the one that was missing
  5. removed an unnecessary alert role
  6. moved explanatory content to the technique
  7. removed out of date assistive technology information

Previews:

Also:
1. Add `autocomplete` attributes
2. Removed 404 resource link
3. Removed old editors CSS link
4. Added `code` tags
5. Updated the list of errors to use a list instead of 3 paragraphs
@fstrr fstrr marked this pull request as draft March 7, 2026 00:35
@netlify
Copy link
Copy Markdown

netlify bot commented Mar 7, 2026

Deploy Preview for wcag2 ready!

Name Link
🔨 Latest commit f9c3117
🔍 Latest deploy log https://app.netlify.com/projects/wcag2/deploys/69d23b7f57d0ae00083ef391
😎 Deploy Preview https://deploy-preview-4981--wcag2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@fstrr fstrr self-assigned this Mar 9, 2026
fstrr added 9 commits March 10, 2026 14:25
1. removed the jQuery and rewrote using vanilla JavaScript
2. Removed several unused functions
3. simplified the date validity field
4. added a replacement image for the one that was missing
5. removed an unnecessary `alert` role
6. moved explanatory content to the technique
7. removed out of date assistive technology information
1. removed jQuery
2. added CSS
3. removed the `focus` functionality for the error message—there’s no point in having a live region with that present
4. added `autocomplete` attributes
5. removed the empty `h2` element
6. moved the error message inside the form
7. slightly tweaked the content of the error message
1. Remove outdated content
2. Update examples to add `autocomplete` attributes
3. Updated resource links
4. Split the example code into separate `pre` blocks
1. remove jquery
2. add `viewport`
3. update css to use modern positioning
4. add `inert` attribute
@patrickhlauke
Copy link
Copy Markdown
Member

Moved this back into "In progress", as this will likely need considerable untangling once #4980 lands (possibly to the point where it might be quicker to just redo the PR as a fresh one)

@patrickhlauke
Copy link
Copy Markdown
Member

out of interest, i did try a naive "rebase this against the other branch from #4980" but it ended up with merge conflicts all over the shop... so it may indeed be easier to just do a fresh PR that only touches the jquery stuff and nothing else, cleanly. may have a go at it tomorrow...

@patrickhlauke
Copy link
Copy Markdown
Member

Resolved the merge conflicts following #4980 but suggest these need to all be carefully checked again now, by hand, to verify they're still doing what they're supposed to and that the styling hasn't gone wrong in the process either

@w3c w3c deleted a comment from fstrr Apr 5, 2026
@w3c w3c deleted a comment from fstrr Apr 5, 2026
@w3c w3c deleted a comment from fstrr Apr 5, 2026
@patrickhlauke
Copy link
Copy Markdown
Member

Reviewed the techniques and verified that the working examples work correctly now. This is good to go, as far as i'm concerned

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants