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.

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

Do 346 writing spdx statements repeatedly #618

Merged
merged 2 commits into from
Dec 27, 2023

Conversation

Etsija
Copy link
Member

@Etsija Etsija commented Dec 27, 2023

This PR is a "fast fix" to the issue for not remembering the SPDX Expressions which the user has input earlier to the license conclusions form. It utilizes the browser's native autosuggestion feature. Note that as of today, only Firefox seems to remember more than one earlier input, while Microsoft Edge and Chrome seem to have an autosuggestion history of exactly one element.

Autocomplete doesn't seem to natively work with textareas in browsers,
so this commit changes the pattern field of a path exclusion form to
a text input field. This allows the browser to remember the pattern
history, improving the user experience.

Signed-off-by: Etsija <[email protected]>
This commit adds a history to the SPDX Expression input field, by
adding a "name" attribute to the input field. When the user starts
typing, the input field will show a dropdown with the previously
entered SPDX Expressions.

Signed-off-by: Etsija <[email protected]>
@Etsija Etsija requested a review from a team as a code owner December 27, 2023 09:16
Copy link

linear bot commented Dec 27, 2023

@Etsija Etsija temporarily deployed to test-environment December 27, 2023 09:16 — with GitHub Actions Inactive
@Etsija Etsija temporarily deployed to unittest-environment December 27, 2023 09:16 — with GitHub Actions Inactive
@Etsija Etsija merged commit 7d666fd into main Dec 27, 2023
5 checks passed
@Etsija Etsija deleted the do-346-writing-spdx-statements-repeatedly branch December 27, 2023 10:00
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