[BUG] Changing short-cut for "Clear" to "Del" or "Ctrl Del" causes a popup JS error

[christoferd]

🛻 LaraDumps Bug Report

Information

Changing short-cut for "Clear" to "Del" or "Ctrl Del" causes a popup JS error.

Pre-steps

1. Click settings
2. Click shortcuts
3. Click [ Edit ] button
4. Click in the clear input shortcut box
5. Press the Del key on the keyboard
6. Notice the error popup as shown in the attached screenshot

Have you searched through other issues to see if your problem is already reported or has been fixed?

• Yes - I did not find it.
• No

Did you read the documentation?

• Yes - I did not find it.
• No

The word "shortcut" does not appear within your whole website.

Have you tried to reconfigure LaraDumps?

To configure LaraDumps, run: php artisan ds:init.

• Yes - I didn't work.
• No, I believe this error is not related the configuration.

I don't think it's related to the issue.

Software Version

You can run composer show -i and npm list to list installed package with their versions.

Software	Version (exactly)
LaraDumps App	4.8.2
LaraDumps package	4.5.2
LaraDumps-core	3.2.6
Laravel	12.31.1
Operating System	Windows 11

What is your project Environment?

• Laravel Herd
• Laravel Valet
• Laravel Sail
• Docker
• XAMPP
• Laragon
• Other - please specify: FlyEnv, PHP 8.4 php artisan serve

---

Describe the bug

Changing short-cut for "Clear" to "Del" or "Ctrl Del" causes a popup JS error... and the shortcut doesn't register, however the text in the input box remains the same causing the error to popup every time "Edit" button is clicked.
And, also pops up with an error now every time I open the app (even in Administrator mode).

Note: I did not open the app in Administrator mode, just opened normally.

What happened?

I did FOO expecting BAR as a result. However, I received BAZ...

Steps to Reproduce the problem

1. Click settings
2. Click shortcuts
3. Click [ Edit ] button
4. Click in the clear input shortcut box
5. Press the Del key on the keyboard
6. Notice the error popup as shown in the attached screenshot

Suggestions

I don't know about these things.
Note that I was still able to set the shortcut back to the default Ctrl + Shift + K
Now it is working as normal.

Extra information

[luanfreitasdev]

Hi @christoferd . I still haven't been able to find the root cause of this problem. I'll close this issue because of that.
But you can reopen it or submit a pull request with the solution.

Thank you.