digitalfabrik / integreat-app

React JS and React Native App for Integreat
https://integreat.app
MIT License
45 stars 15 forks source link

Show number of search results #2520

Open steffenkleinle opened 9 months ago

steffenkleinle commented 9 months ago

Is your feature request related to a problem? Please describe. As a user using the search I want to know how many results are found.

Describe the solution you'd like This is relevant on both web and native and probably for both the normal category search and the city search. Somewhere a number of search results should be shown. Or do you disagree and think the number of search results is not necessary?

Describe alternatives you've considered None.

Additional context Relates to #2352.

nikolahoff commented 1 month ago

@steffenkleinle can you tell me what these items are that are shown when no search term has been entered yet? (img below) Is it possible in this case to shoe "most searched" items?

nikolahoff commented 1 month ago
Bildschirmfoto 2024-05-14 um 12 18 54
steffenkleinle commented 1 month ago

@steffenkleinle can you tell me what these items are that are shown when no search term has been entered yet? (img below)

@nikolahoff those are just an unsorted list of all categories available. Those should be removed with #2755 though.

Is it possible in this case to shoe "most searched" items?

You mean "most searched" as in most clicked categories in the search among all users? I think this is very much effort, but yes, in theory probably possible.

nikolahoff commented 1 month ago

Ok, I think then we don't need to implement it. What was the reason for implementing the number of search results? @steffenkleinle

steffenkleinle commented 1 month ago

Ok, I think then we don't need to implement it. What was the reason for implementing the number of search results? @steffenkleinle

I think it was somehow related to accessibility/have a better overview. I think just for accessibility we can just announce it without visually showing the number of search results though. So if you think it does not make sense from an UI/UX perspective we can also just reject this :)