Skip to content

Conversation

parlough
Copy link
Member

@parlough parlough commented Nov 5, 2023

The name field on an input is used for submitting the search, so it's okay for the IDs to be different, but the names to remain the same.

Fixes #5295

Copy link

github-actions bot commented Nov 5, 2023

Visit the preview URL for this PR (updated for commit 4aec2a8):

https://dart-dev--pr5319-fix-5295-8x6otdwc.web.app

(expires Sun, 12 Nov 2023 01:22:49 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: d851bc446d3c4d7394c5406c6f07255afc7075f3

Copy link
Contributor

@atsansone atsansone left a comment

Choose a reason for hiding this comment

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

No error found using Wave or Lighthouse. Approved.

@atsansone atsansone assigned parlough and unassigned atsansone Nov 7, 2023
@parlough parlough merged commit 86cb5b7 into main Nov 7, 2023
@parlough parlough removed their assignment Nov 7, 2023
@parlough parlough removed the st.RFM label Nov 7, 2023
@parlough parlough deleted the fix/5295 branch November 7, 2023 18:17
MaryaBelanger pushed a commit that referenced this pull request Nov 14, 2023
The `name` field on an input is used for submitting the search, so it's
okay for the IDs to be different, but the names to remain the same.

Fixes #5295
atsansone pushed a commit to atsansone/site-www that referenced this pull request Jan 26, 2024
The `name` field on an input is used for submitting the search, so it's
okay for the IDs to be different, but the names to remain the same.

Fixes dart-lang#5295
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.

Remove duplicate search form in the side navigation
2 participants