Skip to content
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

In library, the two search boxes are confusing the user #554

Closed
Popolechien opened this issue Dec 9, 2020 · 20 comments · Fixed by #946
Closed

In library, the two search boxes are confusing the user #554

Popolechien opened this issue Dec 9, 2020 · 20 comments · Fixed by #946
Assignees
Labels
Milestone

Comments

@Popolechien
Copy link
Member

While looking at All Files I am dealing with hundreds of files if I am set to English. Typing a search for a given term should allow to filter results for that particular string.

@kelson42
Copy link
Collaborator

@Popolechien I guess you got confused beetween the content search and the library search box

@Popolechien
Copy link
Member Author

Popolechien commented Dec 10, 2020

Correct. I guess the Enhancement should be "Search for a zim title while View is on the library, search in zim files when view is in a zim." This also could be a sort of drop-down menu to toggle search range, but that sounds more clunky

@kelson42 kelson42 changed the title Search should allow searching zim titles in the list of available files for download In library, the two search boxes are confusing the user Dec 11, 2020
@kelson42 kelson42 added question UI User Interface labels Dec 11, 2020
@kelson42 kelson42 self-assigned this Dec 11, 2020
@kelson42
Copy link
Collaborator

@Popolechien What you propose is too complex IMO. Here is how the mockup looks like:
image

@Popolechien
Copy link
Member Author

And here is what we have:
image

@stale
Copy link

stale bot commented Feb 13, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be now be reviewed manually. Thank you for your contributions.

@stale stale bot added the stale label Feb 13, 2021
@suterma
Copy link

suterma commented Jul 20, 2021

@Popolechien I guess you got confused beetween the content search and the library search box

I can only second this. Any new user will be completely confused, what to search with each box, the placeholder text is not conculsive. Also, the very existence of two search bars on a screen should be avoided in the first place.

@kelson42
Copy link
Collaborator

@Popolechien @suterma So, the current implementation respects the UI design mockup! We need to agree if there is an implementation problem or a design problem... or both.

@suterma
Copy link

suterma commented Jul 20, 2021

This comment proposes a concrete solution to the issue at hand. The aim is to somewhat mimick the behavior of the Firefox browser with regards to content and settings.

  • The tabs bar is ALWAYS the topmost UI element.
  • Below is some "menu bar" that displays the appropriate navigation within the visible tab's content.
  • Below is the content, respective to the tab type.

Imho, these are the advantages:

  • This design avoids the "double search bar"
  • Since it is similar to an existing application, it does not reinvent the wheel much (Least Surprise)
  • The displayed search bar is always where the searched items are (IN the tab, not above the tab bar)

Screen mockups:

kiwix-toptab-library

A proposal of a freshly started application, with just the initial library tab. Note that the forward/backward buttons plus the goggles are removed in this tab. Here, the search bar (with exemplary search term "geog") obviously searches within the files.

kiwix-toptab-content

When a "content" tab is opened, the search bar searches it's content. Note that here, the forward/backward buttons are displayed, and the goggles. However, the file open icon is not available.

Clearly, this proposal is only a first draft, out of my own experience as a new kiwix user, and should be refined. Especially the UI element backgrounds should get improved to indicate the grouping of tab/tab-content better.

Note:

  • This proposal might also solve Search is broken on empty tab page #499, in case this user was trying to search for a file in the upper search bar, when they should have been using the lower one in the current implementation.

@kelson42
Copy link
Collaborator

@suterma Thank you for your proposal. This is an interesting one. But, honestly I don't fell comfortable to make such a massive change just right like this. This is too much, too quick, and from someone with too few experience with the project.

I propose an alternative which is easier and less risky:

  • Move the library search bar to the left sidebar, below other filters
  • Keep it like it is, but replace "Search files" with "Search catalog"
  • Disable the top search bar, if in library. Activate otherwise

@kelson42
Copy link
Collaborator

For the documentation, here is how the Kiwix Desktop 1.0 library looked like:
image

@stale
Copy link

stale bot commented Sep 19, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be now be reviewed manually. Thank you for your contributions.

@stale stale bot added the stale label Sep 19, 2021
@stale stale bot removed the stale label Jul 2, 2023
@kelson42
Copy link
Collaborator

kelson42 commented Jul 2, 2023

@juuz0 @Popolechien Should we try to handle this now?

@kelson42 kelson42 added this to the 2.4.0 milestone Jul 2, 2023
@Popolechien
Copy link
Member Author

@kelson42 Sure thing.

@kelson42
Copy link
Collaborator

kelson42 commented Jul 3, 2023

To summarize, we have two proposals to solve the problem:

@rgaudin @mgautierfr @veloman-yunkan @Benoit75 @Popolechien @juuz0 @suterma If you have an opinion about which one you think is better and can explain why. Please express yourself here.

@rgaudin
Copy link
Member

rgaudin commented Jul 3, 2023

@suterma 's version feels more natural to me; like Firefox.
Is there a search in page feature? So a potential for an additional search field? Firefox places this last one on the bottom.

@kelson42
Copy link
Collaborator

kelson42 commented Jul 3, 2023

Is there a search in page feature?

Yes, but the search bar appears at the bottom. No confusion possible.

@juuz0
Copy link
Collaborator

juuz0 commented Jul 7, 2023

Proposal 1 (#554 (comment)) sounds good in combining the search bars, but the changing the placement of top search bar looks weird to me. The fullscreen and menu buttons also move with it. They are top-right in apps usually (atleast the fullscreen/maximize button).

Proposal 2 (#554 (comment)) is more natural in the sense all filters are at the same area. Though, rather than disabling the top search bar, I think completely hiding it will be better? Same with hiding anything not useful (back button/forward button/ reading list button). I prefer this proposal.

@kelson42
Copy link
Collaborator

kelson42 commented Jul 7, 2023

Though, rather than disabling the top search bar, I think completely hiding it will be better? Same with hiding anything not useful (back button/forward button/ reading list button). I prefer this proposal.

Hidding will proably not happen as we can not hide everything. But greying yes. But for the searchbar, it will be soon a multizim searchbar and as such there won't be reason to grey it once this is implemented.

@juuz0
Copy link
Collaborator

juuz0 commented Jul 7, 2023

But for the searchbar, it will be soon a multizim searchbar

This will outdate proposal 1 I think? How will multizim search and books filtering work with the same searchbar, if not.

@kelson42
Copy link
Collaborator

kelson42 commented Jul 9, 2023

But for the searchbar, it will be soon a multizim searchbar

This will outdate proposal 1 I think? How will multizim search and books filtering work with the same searchbar, if not.

I don't think this make the request deprecated. Actually I'm almost convinced that having only one bar would be better. But what is sure, is that this has a lot of implication to have this kind of "Awesome bar" and we are definitly not mature for that.

I will open a dedicated ticket to that, but in the meantime please implement this ticket by moving the library search bar at the top of the side bar.

juuz0 added a commit that referenced this issue Jul 14, 2023
The search function is now moved to filters area for more consistency
Fix #554
juuz0 added a commit that referenced this issue Jul 19, 2023
The search function is now moved to filters area for more consistency
Fix #554
juuz0 added a commit that referenced this issue Jul 24, 2023
The search function is now moved to filters area for more consistency
Fix #554
juuz0 added a commit that referenced this issue Jul 30, 2023
The search function is now moved to filters area for more consistency
Fix #554
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants