-
Notifications
You must be signed in to change notification settings - Fork 444
Search service shows not all options due to amount of search services configured #11139 #11144
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Search service shows not all options due to amount of search services configured #11139 #11144
Conversation
dsuren1
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rowheat02
The functionality looks fine. Just a few minor amendments. Kindly have a look.
- Move the tooltip placement on
all servicesto the left, to be consistent with the other services in the list

- Please apply a pointer cursor when hovering over these search service menu items. Alternatively, consider using
MenuIteminstead ofdivto automatically apply the pointer cursor. Also, kindly ensure that the entire row displays the pointer on hover

|
@dsuren1 re-requested the review.
I avoided using the Menu item in the custom sub-menu container as the CSS was messed up when using MenuItem in a parent other than a dropdown. So, Div has been used, and the pointer cursor has been fixed in CSS. |
|
@dsuren1, tests have been fixed. |
|
@ElenaGallo Kindly test this in DEV when available. Thanks! |
… configured geosolutions-it#11139 (geosolutions-it#11144) * fix: new UI for multiple search services * fix: updated requested changes * fix: unit test fixes * Update web/client/themes/default/less/searchbar.less --------- Co-authored-by: Suren <dsuren1@gmail.com>
|
Test passed, @rowheat02 please backport to 2025.01.xx. Thanks |
Description
New UI if multiple search services are available.

When hovering the mouse over "Search by location Name", new Sub menu container opens to pick the service
If the number of services takes more space than the limited height, then it is in scroll view.
The idea was dropped to search the services as it may give wrong meaning while selecting use all services option.

All other options in the search bar will always be visible.
Please check if the PR fulfills these requirements
What kind of change does this PR introduce? (check one with "x", remove the others)
Issue
fixes #11139
What is the current behavior?
#11139
What is the new behavior?
Even if there are huge number of search services, no UI components will be hidden.
Breaking change
Does this PR introduce a breaking change? (check one with "x", remove the other)
Other useful information