kiwix-android
kiwix-android copied to clipboard
Find in page conflated with Find in ZIM
Basically when I start typing text the "find in page" button disappears and the search only applies to the whole zim.
I would probably advocate that "search" and "find in page" be dissociated (the latter being pushed to a menu item), unless there's an elegant way to switch from or select either on that search bar.
This bug was also fixed in #1771 , the find in page
buttons visibility is now controlled by whether or not there is text in the search field.
I do agree that Find In Page should be accessible from the main screen, potentially as an overflow option. It probably wasn't because of the overpopulation of that list that exists now
Could we split things in two tickets? Here we have a bug which needs to be fixed. The other part is an idea of improvement (which I'm not really in favor of).
The bug is fixed
chrome has
Find in Page
in its' overflow menu, as an example
The original discussion about that is at #154. I'm not sure I see new arguments here to chance the current behaviour.
We could try to adopt a more browser like behaviour and abandon the search icon because it is the search icon that is confusing people.
If we had a chrome-like "address bar" at the top with the title in it and we could only open Search
by clicking in it then I think people would look in the overflow menu for Find in Page
Sounds good/reasonable to me. Obviously the two-in-one approach isn't ideal, so let's shamelessly copy what others have already figured out.
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 will refresh/reassess this ticket.
@MohitMaliFtechiz Please look at this and make a proposal how we coulf tackle this ticket
@kelson42 We can move FIND IN PAGE
to the overflow menu like Chrome(Since this is the place where we can take it) but before changing it we have a few points to consider. I have noticed a few things about the current behavior, and if we move this button to the overflow menu then we have these benefits.
- It will remove the
FIND IN PAGE
button from theSearchScreen
as its functionalities is not there. - It will increase the
search box
area. if the user types a longsearch term
he will easily see that. - If any user only wants to use the
FIND IN PAGE
feature he can directly use that from the overflow menu. For now, the user has to open the search and type a single character in it to initiate theFIND IN PAGE
feature. It will also start thesearch
which is unnecessary for those who only want to use theFIND IN PAGE
feature(The search will unnecessarily be triggered which is useless in this case).
https://github.com/kiwix/kiwix-android/assets/34593983/6a728910-52fc-44f6-8ef5-1763d307e189
Apart from these small points, the current approach is not too bad, it has its own benefits.
- The current UI looks good from the UI, and user point of view. Since it has 2 search functionalities(Search for entry and search in page) at the same place the user easily use both functionalities from one place, and in most cases, it is beneficial for the users to have the same type of functionality at one place(Personally I like the current approach since it allows me to use the same type of functionality from a single screen).
- Both functionalities do not impact each other as these functionalities are implemented on different screens. For example
Search
in performed inside theSearchFragment
, and theFIND IN PAGE
functionality is performed in thereader
screen. - The current UI is familiar to the users, and they are habited to it as they have been using it for a long time.
These are my current observations. @kelson42 let me know what do you think?
@MohitMaliFtechiz Thank you for wrapping up the pros and cons. Whatbwoukd be your personal recommandation?
@kelson42 Personally I would like to have the current approach but with an enhancement in code. I want to make the FIND IN PAGE
button always clickable, because it will directly open this feature without triggering the search in ZIM file(which is unnecessary when the user only wants to use the FIND IN PAGE
functionality).
@MohitMaliFtechiz I will think a bit more about it and will come back to you
chrome has
Find in Page
in its' overflow menu, as an example
I also think that the FIND IN PAGE
feature should be in the overflow menu.
The current UX is confusing.