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

Suggestions and a fix #21

Open
gbw369 opened this issue Apr 16, 2024 · 0 comments
Open

Suggestions and a fix #21

gbw369 opened this issue Apr 16, 2024 · 0 comments

Comments

@gbw369
Copy link

gbw369 commented Apr 16, 2024

As noted below you need to tag the delete button to have it function:

<button data-delete class="key large"> <svg style = "pointer-events: none;" xmlns="http://www.w3.org/2000/svg" height="24" viewBox="0 0 24 24" width="24"> <path fill="var(--color-tone-1)" d="M22 3H7c-.69 0-1.23.35-1.59.88L0 12l5.41 8.11c.36.53.9.89 1.59.89h15c1.1 0 2-.9 2-2V5c0-1.1-.9-2-2-2zm0 16H7.07L2.4 12l4.66-7H22v14zm-11.59-2L14 13.41 17.59 17 19 15.59 15.41 12 19 8.41 17.59 7 14 10.59 10.41 7 9 8.41 12.59 12 9 15.59z"></path> </svg> </button>

Also, I'd suggest moving the word list and dictionary to another file so that the code running this is easier to edit.
Change the html file to say this:

<script src="wordlist.js"></script> <script src="script.js" defer></script>

and then you'll have to remove the duplicate word list and dictionary from script.js

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

No branches or pull requests

1 participant