Skip to content

Make the search form more clear to understand #31

Closed
@subins2000

Description

@subins2000

https://github.com/knadh/dictpress/blob/master/site/static/main.js#L16C1-L17C1

.replace(/[^a-z\u00E0-\u00FC\s]/ig, '')

This regex replace is unnecessary for a default example template. I was confused why the search was not working and ending up in a HTTP authentication required page.

The form didn't look like it had a JS backing. One more suggestion is to remove the name attribute from the search form so that it is more clear. Why need a name if the form submission is actually handled by JS.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

questionFurther information is requested

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions