-
-
Notifications
You must be signed in to change notification settings - Fork 28
Create a widget to search in Kiwix #36
Comments
@kelson42, I already built most of the widget parts. I just want to implement a voice search which will directly search kiwix. Look at the pic |
This looks really fantastic, please merge as soon as you are over with the search voice. I'm really impatient to test it. |
@kelson42 Yup i will soon enough |
@EladKeyshawn Additional commnet: please check that the logo is changed in case we have a custom app. |
@kelson42 I need a transparent logos of the customapps for widget implementation. It looks really bad otherwise. |
@EladKeyshawn Wow! this widget looks amazing! ❤️ |
I have tested id but not with a lot of success... I was able to install it on my homescreen and it looks good but:
|
@kelson42 😢 never encountered it. could you please tell me how to reproduce?. When i tested i could see the search history and open the article no problem. make sure you have the latest code. |
Here the related locat: == Kiwix is close, I click on the widget search part == V/ApplicationPolicy( 2840): isApplicationStateBlocked userId 0 pkgname org.kiwix.kiwixmobile == Kiwix opens and focus on the search, so I see the search history. I click on one of the item == V/ApplicationPolicy( 2840): isApplicationStateBlocked userId 0 pkgname org.kiwix.kiwixmobile == Page reload and I'm still on the search panel (with the history) == |
@kelson42 Did you try clear data and cache of the app after you've installed this update? |
@EladKeyshawn I confirm it works on my other table with 6.0 |
@kelson42 Are you still having problems or can we close this? |
I confirm I still have the same behaviour. No way to get out of the search panel. |
@kelson42 Can I have some details about the device so I can try to emulate? |
My device is a Samsung Galaxy Tab. I post again a recent version of my logcat
|
@mhutti I confirm, if the option "Don't keep activities" is disabled then it works. But it really has to work with this option activated (see reasons on #292). |
Needs same style of fix as #292 |
Still more issues :( Working through now. |
@kelson42 This features should now work. There is a small compromise that on devices not keeping activities, using the widget will ignore previous tab states. I believe this to be acceptable as the widget will likely be used outside of a typical "browsing session" and as such keeping these articles active for a tiny fraction of our users is not time effective at the moment. In the future I suggest rewriting our tab sate code to use a database as well as reevaluating how loadUrl events in our custom webview are called. That should alleviate this issue. |
@mhutti1 I tested your solution looks good for now, so maybe you can close this issue. |
@mhutti1 Same here. Kudos!!! |
Widgets are an important part of Android usability:
https://developer.android.com/design/patterns/widgets.html
It would be really nice to propose a Kiwix search widget (within Kiwix app) to allow users to find things with Kiwix directly from their home page.
The text was updated successfully, but these errors were encountered: