[BUG] - Page reload when "Choose Language" text box is cleared

[MaxWipfli]

First Check

• This is not a feature request.
• I added a very descriptive title to this issue (title field is above this).
• I used the GitHub search to find a similar issue and didn't find it.
• I searched the Mealie documentation, with the integrated search.
• I already read the docs and didn't find an answer.
• This issue can be replicated on the demo site (https://demo.mealie.io/).

What is the issue you are experiencing?

In the "Choose Language" dialog, a reload is triggered when the language text box is fully cleared.

I noticed this on my fresh containerized deployment (running v2.4.1), and it can be reproduced on the demo site.

Steps to Reproduce

1. Press "Language" in the side bar.
2. Delete all text from the text box.

Please provide relevant logs

not relevant

Mealie Version

Details
Version: nightly
Build: 251544b
Application Mode: Production
Demo Status: Demo
API Port: 9000
API Docs: Enabled
Database Type: sqlite
Default Household: Family
Recipe Scraper Version: 15.3.3

Checks
Secure Site: Yes
Server Side Base URL: Yes
LDAP Ready: No
OIDC Ready: No
OpenAI Ready: No
Email Configured: No

Deployment

Docker (Linux)

Additional Deployment Details

not relevant

[michael-genson]

I honestly had no idea you could do this. This is probably an easy fix, I think we're refreshing any time that selection changes reactively, but we should probably only refresh once the menu is closed, or at least validate the value first. Or maybe just prevent the user from deleting the text

[MaxWipfli]

Or maybe just prevent the user from deleting the text

When I wanted to search for a language, the obvious way is to just start typing as the text is already selected. However, when I did that on my phone (Firefox on Android), this still triggered the reload. So I think preventing the user from deleting the text is not the correct solution.

Also, the reload only happens when selecting a language (intended) OR when the field is empty (unintended). However, when you delete the text character by character, the reload is only triggered when the field is empty.

I guess the simple but sensible solution would just be to only reload if the value is valid, as you suggested.

[michael-genson]

Another option would be to just add a submit/confirm option, removing the reactivity (which doesn't really work anyway, hence why we refresh the page)