pmartin / plop-reader

A Wallabag application for Pocketbook / TEA Touch Lux ereaders
http://plop-reader.pascal-martin.fr
GNU General Public License v3.0
44 stars 6 forks source link

Problems with Pocketbook 631 French (Tea Touch HD Brown edition) #99

Open Ygster opened 6 years ago

Ygster commented 6 years ago

I just switched to this model and even if the application works overall, I noticed some concerns.

1/ The resolution being higher, the display is not done on all the screen (see capture) plop_reader

2/ I have the impression that despite the message during synchronization that the EPUBs are loaded, only the HTML version is available for reading (if I press a long article, the only suggestion is "Read HTML content"). But by looking on the reader in the file system/share/plop-reader/entries-epub the EPUB versions are well present.

If you have information on how to investigate this, I can look into it further or to force the use of the EPUB file.

Ygster commented 6 years ago

For the second point, I have created another account in wallabag and all is working as previously.

pmartin commented 6 years ago

Hi @Ygster, Thanks for this issue!

For the 2nd problem: I've seen this problem once (and only once) while using the app… And I've never seen it again, so I have not investigated… Created a new account like you did… OK, I didn't think about that :-) When I've had this problem, I used (on my ereader) the "Supprimer toutes les données locales" feature -- you can see the corresponding menu entry in the 2nd screenshot of this post: http://plop-reader.pascal-martin.fr/2017/04/05/v0.4.0-is-released.html After deleting everything on the device (with this feature), I re-did a synchronization… And everything worked fine.

As for the 1st problem… Whoa, I'm actually surprised it looks so nice! Thanks for the sceenshot! I did a bit of magic in the code trying to make it work on several resolutions, but never tested it on anything else than my own TL3 (not HD), so… Well, considering… It's not too bad ^^ The solution for the HD device you have would be to change the number of entries displayed on the screen, depending on the number of pixels the device has (vertically) -- yeah, everything is calculated in pixels :-/ For example, the height of one line is hard-coded here. And I think (I haven't taken a single look at the code since more than a year ago -- I don't even have a working development anymore) the number of items on the screen is hard-coded there. Maybe calculating this dynamically (based on the screen height and the item height) would actually work, if you want to give it a try ;-) Unfortunately, I do not have an HD device, and I do not have access anymore to the full range of devices I once had access to…

doc75 commented 4 years ago

I would be ready to work on this, but so far I cannot succeed in building the app (cf. my comment on #100). If anyone has an idea, I'll update the documentation to explain more in detail how the build can be done.

@pmartin the app is working great on HD Touch Lux anyway. Thanks for this great piece of software.