Close images swiping up or down #13
Closed
opened 2022-07-24 08:55:07 +00:00 by AmineB
·
7 comments
No Branch/Tag Specified
master
fix-bottom-bar
release
br
fix-toast
cleaning
v125051211
v125040991
v125030901
v125030711
v125030681
v125020581
v125020471
v125020411
v125010291
v125010241
v125010201
v125010131
v125010111
v125010031
v124123651
v124123641
v124123421
v124113311
v124113301
v124113252
v124113251
v124041081
v124030731
v124020451
v124010301
v124010191
v124010032
v124010031
v123113311
v123102961
v123102852
v123102851
v123102841
v123061811
v123061651
v123051471
v123051331
v123051321
v123051301
v123051211
v123041021
v123030851
v123030751
v123030681
v123030621
v123020572
v123020571
v123020523
v123020522
v123020521
v123020511
v123020491
v123010301
v123010281
v123010261
v123010241
v123010041
v122123641
v122123631
v122123621
v122123611
v122123602
v122123601
v122123571
v122123532
v122123531
v122123522
v122123521
v122123483
v122123482
v122123481
v122123461
v122123431
v122123421
v122123391
v122123381
v122123371
v122123351
v122113181
v122113172
v122113171
v122113161
v122113131
v122113101
v122102881
v122092701
v122092691
v122092671
v122092572
v122092561
Labels
Clear labels
Difficulty - Beginner friendly
Difficulty = Easy
Difficulty = Hard
Difficulty = Medium
Priority = CRITICAL
Priority = High
Priority = Low
Priority = Normal
Priority = Some day
Status - Fixed somewhere else
Status - No details provided
Status = Can't fix
Status = Duplicate
Status = Help wanted
Status = Invalid
Status = Need more details
Status = Taken
Status = Wontfix
Type - Selfoss works like this
Type = Bug
Type = Chore
Type = Enhancement
Type = Feature
Type = Question
Type = SELFOSS API ISSUE
Type = Tools
Type = UX/Design
Up For Grabs
Assignees
AmineB (Amine Bouabdallaoui)
Clear assignees
No Assignees
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: Louvorg/ReaderForSelfoss-multiplatform#13
Reference in New Issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Prerequisites
CONTRIBUTINGguide ?Description
In the article reader, tapping on a picture opens it in full screen.
To return to the article two actions are possible:
The first action is inconvenient due to the position, which is far away from the lower right which is where most people hold the phone one handed.
The second action is much more convenient, but still requires the movement of the thumb from the center of the screen to the lower right.
In landscape mode, action 1 results easier if holding the phone from the left or two handed while action 2 is more difficult.
Adding one extra action to close the image would ease the use of the application.
Swiping full screen images up or down to close them is widespread among other applications and could thus be a good candidate.
This methodology works equally well when holding the phone from the left or from the right, both in landscape and portrait mode; it can be used two handed as well.
I'll be working on this.
I have this ready and working, but it's using a library which is not updated since a while.
This is the library in question https://github.com/liuguangqiang/SwipeBack
I'm not sure using such an old library is a good idea since there may be problems with newer versions of android.
I'll be looking for an alternative library but I didn't find any at the moment.
Alternatively, I may fork this library and keep it updated; in this way we would also be able to change the behaviour in order to keep it consistent with the iOS implementation.
I would prefer if we didn't add a library just for this feature.
Can't it be done without one ? Even if there isn't any animation ?
Oh, replying by email doesn't work.
Yes, I can do it without library but without animation.
It's probably much better than including that questionable library.
@davidoskky is there any update on this ?
Hey @AmineL, I'm sorry as I've been pretty busy lately.
If I'm not mistaken this is pretty much ready, I'll take a look in the next few days and send a PR.
@davidoskky thanks for the update.
Keep me posted !