[ENHANCEMENT] Preventing reload of the data when clicked in search icon when no search query is typed

[Jynx2004]

Is your feature request related to a problem? Please describe.
When I click on the search icon to search for any feature without typing any search query , then the feature data reloads unnecessarily. For a moment of time the dropdown also gets displyed on clicking the search icon, which doesnt looks good.

Describe the solution you'd like
Prevent the unnecessary reload of the features data.

[mukulpythondev]

@past @jcscottiii Hi Mentor and fellow contributors , I am Mukul Rana experience with Full stack development and i am excited to work this issue and started working on it.
Raise the PR soon

[suprith-hub]

Hey, @Jynx2004 @jcscottiii is clicking on search icon supposed to bring user to 1st page ?
If this feature is to be retained, then search icon should be disabled when user is on 1st page only. Right?

[jcscottiii]

Hi all,

Thank you for your interest in this feature.

There are a few things going on because we use gotoFilterQueryString() in multiple places:

1. Current behavior The search button is currently always enabled. Desired behavior The search button should be disabled if there the current submitted value is the same as the "changed" value. We do a similar thing in the date picker by detecting if the date has changed.
2. Current behavior If a user clicks the enter button, it goes to sets window.location.href regardless if the current href is the same. Desired behavior We should compare the new url with the existing url and we should also make this file use navigateToUrl. And do the comparison inside of there so all navigation events get this benefit.

[Jynx2004]

Hi all,

Thank you for your interest in this feature.

There are a few things going on because we use gotoFilterQueryString() in multiple places:

1. Current behavior The search button is currently always enabled. Desired behavior The search button should be disabled if there the current submitted value is the same as the "changed" value. We do a similar thing in the date picker by detecting if the date has changed.
2. Current behavior If a user clicks the enter button, it goes to sets window.location.href regardless if the current href is the same. Desired behavior We should compare the new url with the existing url and we should also make this file use navigateToUrl. And do the comparison inside of there so all navigation events get this benefit.

This seems to be a more generalized solution . Will start working on it .