Skip to content

Conversation

@rfultz
Copy link
Contributor

@rfultz rfultz commented Aug 5, 2024

Summary

Required reviewers

  • front-end
  • UX?

Impacted areas of the application

A chunk of the front-end interactivity where jQuery is involved, especially the deprecated parts of it. Specifically:

Screenshots

No visible changes

Related PRs

None

How to test

  • Pull the branch
  • npm i
  • npm run build
  • ./manage.py runserver
  • Legal
    • legal page
      • submit button
    • Citation filter
      • down arrow or tab
      • up arrow
      • enter
    • Text filter
      • enter
  • Audit Category Sub Category
    • change listener
  • Audit Tags
    • primary category change listener
    • secondary category change listener
    • close_sub click handler
    • close_primary click handler
  • Calendar
    • .fc-event, .fc-more, .fc-close (simulateClick on enter)
    • .fc-more (click, placing focus)
    • Tooltip
      • trigger focus on close (though I don't know why)
        URL: http://127.0.0.1:8000/calendar/
        TEST: switch to grid view, click an event to trigger its "tooltip," click the ❌
  • Download
    • focus on button if needed (like with datatables' "export this data");
  • Dropdowns
    • .show() (set focus on first checkbox)
    • esc button (put focus back on the button)
    • input[type="checkbox"]
      • enter - trigger change event
      • set focus on next/prev items
    • .dropdown__item--selected
      • click
  • Election Map
    • while drawing districts, unique sort during drawBackgroundDistricts()
  • Election Search
    • trigger 'change' event on ZIP code change
  • Date Filters
    • fromQuery - setValue() should trigger 'change' for $minDate and $maxDate
    • handleModifyEvent - minDate, maxDate change trigger
    • handleMinDateSelect - grid li - mouseenter/mouseleave (hover states)
    • handleMaxDateSelect - grid li - mouseenter/mouseleave (hover states)
    • handleGridItemSelect - removing mouseenter and mouseleave listeners
  • Election filters
    • fromQuery- trigger change
    • handleElectionChange - trigger change
    • handleCycleChange - cycle trigger change, full trigger change
  • Filter base
    • input set value, then trigger change
  • Filter panel
    • .show() should set focus into first input, select, button
  • Filter set
    • handleTagRemoved - trigger click
  • Filter typeahead
    • handleKeypress - "enter" to submit
    • handleSubmit - field trigger change
    • appendCheckbox - checkbox.trigger change
  • Filter select
    • fromQuery - input trigger change
  • Text filter
    • handleChange - settimeout, button trigger focus
  • Toggle filter
    • fromQuery - elm input trigger change
  • Validate date filters
    • fromQuery - minDate & maxDate trigger change
  • Form Nav
    • clearNamesIfNull - form trigger submit
  • helpers.js
    • getCookie REMOVED (does this look broken anywhere?)
  • Maps
    • appendStateMap - trigger change on the select inside the last .state-map.last()
  • search.js
    • Focus the search field on typing /
      NOTE: I DON'T THINK IS BEING USED ANYWHERE. REMOVE $('.js-search').each(function() { FROM data-init.js?
  • Skipnav
    • findTarget - on click, focus on first non-nav content on page
    • findTarget - on Enter, focus on first non-nav content on page
      HOW: load any page, press tab button, click and Enter the skipnav element in the upper left
  • Tables
    • modalRenderFactory - focus the close button
    • hidePanel - focus on panel button
    • checkFromQuery (JCarroll - Tested all tables that need that function and it works on all but Filings/Reports, fixed by PR 6481)
    • trigger 'change' on :checked)
  • TopEntities
    • updateElectionYearOptions - trigger change
      HOW: homepage on local only, change the election year
  • Typeahead
    • searchSite - trigger submit
  • Contact form
    • initTypeahead - #id_u_contact_title trigger focus then trigger blur
  • datatable-audit
    • add event listeners for #sub_category_id.ready and #sub_category_id.ajaxComplete
  • Election reporting dates tables
    • mediaQueryResponse - close.trigger focus
  • Reporting dates tables
    • mediaQueryResponse - close.trigger focus
  • 404
    • open feedback window
  • 500
    • open feedback window

@rfultz rfultz added this to the 25.i milestone Aug 5, 2024
@rfultz rfultz self-assigned this Aug 5, 2024
@codecov
Copy link

codecov bot commented Aug 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.83%. Comparing base (cfb73e9) to head (a525283).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #6411   +/-   ##
========================================
  Coverage    79.83%   79.83%           
========================================
  Files          237      237           
  Lines         5218     5218           
========================================
  Hits          4166     4166           
  Misses        1052     1052           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@rfultz rfultz modified the milestones: 25.i, 26.1 Aug 27, 2024
@rfultz rfultz marked this pull request as ready for review September 6, 2024 12:24
@patphongs patphongs modified the milestones: 26.1, 26.3 Sep 11, 2024
@rfultz rfultz added this to the 26.i milestone Nov 19, 2024
@rfultz rfultz modified the milestones: 26.i, 27.1 Dec 11, 2024
@rfultz rfultz modified the milestones: 27.1, 27.2 Dec 23, 2024
@rfultz rfultz modified the milestones: 27.2, 27.3 Jan 7, 2025
@rfultz rfultz modified the milestones: 27.3, 27.4 Jan 21, 2025
@rfultz rfultz modified the milestones: 27.4, 27.5 Feb 5, 2025
@rfultz rfultz modified the milestones: 27.5, 27.6 Feb 18, 2025
@patphongs patphongs modified the milestones: 27.6, 27.i Mar 19, 2025
@rfultz rfultz modified the milestones: 27.i, 27.7 Mar 27, 2025
@rfultz
Copy link
Contributor Author

rfultz commented Apr 11, 2025

Closing in favor of #6750, #6751, #6752, #6753

@rfultz rfultz closed this Apr 11, 2025
@github-project-automation github-project-automation bot moved this from 👀 Ready to ✅ Done in Website project Apr 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

jQuery Deprecations

3 participants