-
-
Notifications
You must be signed in to change notification settings - Fork 122
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
Comments
@Popolechien I guess you got confused beetween the content search and the library search box |
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 |
@Popolechien What you propose is too complex IMO. Here is how the mockup looks like: |
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. |
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. |
@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. |
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.
Imho, these are the advantages:
Screen mockups: 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. 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:
|
@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:
|
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. |
@juuz0 @Popolechien Should we try to handle this now? |
@kelson42 Sure thing. |
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. |
@suterma 's version feels more natural to me; like Firefox. |
Yes, but the search bar appears at the bottom. No confusion possible. |
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. |
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. |
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. |
The search function is now moved to filters area for more consistency Fix #554
The search function is now moved to filters area for more consistency Fix #554
The search function is now moved to filters area for more consistency Fix #554
The search function is now moved to filters area for more consistency Fix #554
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.The text was updated successfully, but these errors were encountered: