Fix: improve reactivity by reducing debounce time for value changes

[max65482]

Fixes #4994 by decreasing debounce time from 500 ms to 250 ms.

[SebastianKrupinski]

@hamza221 any objections?

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[hamza221]

We might need to fully remove debounce completely for button click, I don't think it makes sense debouncing at all here.
The method is used by other components (so debouce should stay for them and removed only for button click) I can update the PR if needed

[SebastianKrupinski]

I think the de-bounce is on the input not the button click. It slows the update of the contact object from my reading of the code. Its part of the property mixins, it was added 8 years ago so hard to know what the purpose was

[hamza221]

I took a second look at the code, It weird that most components overwrite it, I think we actually need to clean the code here.
Approving as a temporary fix, but not sure if we should merge. I can take a deep five next week

[SebastianKrupinski]

Okay, Look in to it... then merge it if it makes sense...

[hamza221]

Set to draft to avoid accidental merge until next week, @max65482 I hope you're okay with that 😊