Skip to content

fix: improve Add Filter functionality in resource view forms#116

Merged
jguo144 merged 1 commit into
masterfrom
jguo144/2026-04-02/update-voice-over
Apr 3, 2026
Merged

fix: improve Add Filter functionality in resource view forms#116
jguo144 merged 1 commit into
masterfrom
jguo144/2026-04-02/update-voice-over

Conversation

@jguo144
Copy link
Copy Markdown
Collaborator

@jguo144 jguo144 commented Apr 2, 2026

Description

This branch improves the resource view filter UI when using the OpenGov custom theme.

  • Adds view_form_filters.html: filters block wired to CKAN’s resource-view-filters-form module, with aria-labels on the field <select> and filter value input, accessible Remove filter control, and type="button" on Add/Remove.
  • Updates theme.js
    • VoiceOver a11y fix: After Select2 wraps the filter value field, copy the input’s aria-label onto Select2’s .select2-focusser, and re-run when filter rows are added or removed (MutationObserver).
    • Keyboard: Intercept Enter inside .resource-view-filters so it does not trigger implicit form submission (which could activate Delete); open Select2 or the native field picker instead, without breaking Enter when the Select2 list is open.

@jguo144 jguo144 requested a review from nolanpruett April 2, 2026 23:25
Copy link
Copy Markdown

@orca-security-us orca-security-us Bot left a comment

Choose a reason for hiding this comment

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

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Infrastructure as Code high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Vulnerabilities high 0   medium 0   low 0   info 0 View in Orca

@jguo144 jguo144 enabled auto-merge (squash) April 2, 2026 23:28
@jguo144 jguo144 merged commit 1b7cb57 into master Apr 3, 2026
5 checks passed
@jguo144 jguo144 deleted the jguo144/2026-04-02/update-voice-over branch April 3, 2026 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants