Haptic-Apps / Slide

Slide is an open-source, ad-free Reddit browser for Android.
https://reddit.com/r/slideforreddit
GNU General Public License v3.0
1.78k stars 344 forks source link

Toggle to confirm before exiting/sliding back out to post listing #1554

Closed SaladMandrake closed 8 years ago

SaladMandrake commented 8 years ago

A new toggle to confirm before going back out to post listing. Slide is really nice and convenient with all the sliding, but I kept accidentally slide back out to post listing when I meant to slide to previous post.

Nxt3 commented 8 years ago

You are just sliding from too far over. If you're trying to go back to a previous post by sliding--start more from the middle.

I really don't think a dialog is the way to go--as this is such a niche feature.

SaladMandrake commented 8 years ago

Point taken, but further tuning the slide to exit sensitivity might vastly improve user experience, just my opinion =).

Nxt3 commented 8 years ago

I'm pretty sure it was adjusted when @ccrama changed how the swipeback was accomplished. Not sure there is much else to change--I don't have any problem with this and ever since the change--I don't think anyone else has either.

SaladMandrake commented 8 years ago

https://reddit.com/r/slideforreddit/comments/4gcw4n/disable_sliding_out_of_albums/

Here is at least one more person having issues with sliding aspect of the app.

Nxt3 commented 8 years ago

Yeah--in the albums. The albums don't happen to use the new slide system for whatever reason. See #1283

Alexendoo commented 8 years ago

I think this isn't that @Nxt3 but the swiping in from the edge (trying to go back an image) though yeah that's an issue as well. I do agree a bit that it's rather strange to have two distinct actions on roughly the same gesture, can be a bit unintuitive, not sure what would be best there though

Nxt3 commented 8 years ago

@Alexendoo I'm not sure I understand. The album view doesn't use the new way of swiping back. As a result, it is far more sensitive.

If you swipe back from the middle of the screen--you'll go back an image; if you swipe from the far left side of the display--you'll leave the album. It isn't unintuitive.

ccrama commented 8 years ago

I agree with @Nxt3 , I use both actions and they are both very different actions. And you can cancel the action by sliding back, not like it's impossible to cancel closing if you accidentally drag from too far over

Alexendoo commented 8 years ago

For the first part @Nxt3 I was saying yes, that's an issue, but not the one the post/this thread is talking about.

For a large device I can appreciate that yeah, I haven't had it happen since using a nexus 5 because it takes a lot of movement to get over to the left edge, but with a smaller device it happened quite a lot

SaladMandrake commented 8 years ago

Sorry about the mis close, my keyboard bugged out on me, have to reboot the phone.

Anyway, it's rather subjective whether it's different or not. IMO it's slightly different but very similar and users could easily make mistakes and quit when it's unintended, I myself and that guy in the Reddit link above. You could deny it but my mistakes are really happening.

I could still use the app without any of those improvements, but it's all about the user experiences that won people over.

Nxt3 commented 8 years ago

@Alexendoo I know, but he posted a link to someone having issues with the album view swipe. I was responding to that.

You guys are talking about two different issues. The album swipe back has it's own problem independent of the problem this issue addresses.

@ArthurHeng Maybe it is sensitive, maybe it isn't. You're the only person who has reported the issue with the swipe back mechanism in comment view (doesn't mean it isn't an issue for you, though). Adding an option is too niche and there really isn't much else that can be done in terms of sensitivity. We're using a page in a ViewPager to fake a "blank" page to swipe back to. This is something Android natively has. As a result, it isn't easy (and in some cases, even possible), to modify something like the swiping sensitivity.

Alexendoo commented 8 years ago

@Nxt3 The difference between the old & new systems are for vertical scrolling (see #957), it's the angle at which swipe from anywhere is triggered that changed, not the distance to the edge to trigger the (non swipe from anywhere) exit gesture

The issue here purely the fact that swiping in from the right can have two different actions, and it's easy to (on smaller devices at the least) catch the boundary for what's considered the edge of the screen and thus lose your place

SaladMandrake commented 8 years ago

Chances are for everything I reported/requested, there are only 1-3 person reporting and requesting, doesn't mean it's niche. If you haven't figured out a way to minimize the user mistake of those two actions that are so similar to each other, the toggle is a really simple solution to it.

Once the app gains popularity, even a niche feature will become immensely helpful, especially to new comers.

ccrama commented 8 years ago

Well out of 70k users, only a handful have reported it, while other features like the comment fast scroll have been requested many times (which is why that eventually got implemented). This would be a major change to UX which many users (like myself) use daily, which is why we're debating it

SaladMandrake commented 8 years ago

Those two actions are used throughout the entire app, combined with non-technical users, you will have a lot of users keep backing out when they meant to slide to previous. Now how is that niche.

ccrama commented 8 years ago

It's niche to disable it because the vast majority of my users use that as a top level action to close views, as it's enabled on every view in the app. Removing that would mean a whole new UX for people to get used to, which is a big change for an app with this many users and which has had this UX for over a year.

Personally, I use both actions frequently and would not enjoy the experience as much if I could not back out using the swipe from very edge.

I am not trying to be harsh or disregard your suggestion, it just has a lot more effect on the app and its users than what it seems

SaladMandrake commented 8 years ago

A handful means I'm not the only person with that problem. I consider myself to be quite a dedicated user as this and opengur are the only two reader apps I use extensively, daily. This phone might as well be part of my organ and even I make mistakes of sliding the wrong way.

Unless you guys are planning to change the interface and gestures so much until this toggle no longer serves any function, I think this is a good function to have.

SaladMandrake commented 8 years ago

Well, my suggestion is, either add a toggle to confirm before backing out, or increase the gap between two types of swipe/slide, or make it harder to swipe back out by shrinking the area to swipe to exit, if that makes sense.

ccrama commented 8 years ago

Alright, I'll look into it. The issue is disabling it will disable it on every view

SaladMandrake commented 8 years ago

My suggestion is to add a toggle to show a _confirmation _dialogue when user swipes to exit. Just like the Confirm Exit Dialog, but for backing out to post listing.

SaladMandrake commented 8 years ago

Anyway, hope I didnt cause any offense.

Nxt3 commented 8 years ago

@ArthurHeng The problem with a toggle, with what ccrama is saying--is that if you do put a toggle in, you'd have to put that confirmation everywhere there is a swipeback gesture--otherwise, it would be largely inconsistent; the issue is that this is an app called "Slide"; you'd expect the primary way of navigation to be "sliding" around. Hindering that motion would be aggravating--maybe if it was only enabled with one view it would be fine; but from a UX point of view--it would be expected that this dialog shows everywhere. At that point, the app loses its "uniqueness".

ccrama commented 8 years ago

@Nxt3 he does have a point about it though, but I think a compromise might be to have a checkbox to disable swiping back from comments and vertical albums. This would be better UX than a popup, and would solve the issue about losing uniqueness. This seems to be where most of the issues occur with this, and @ArthurHeng could still swipe back to exit from the first post in the list.

What are you guys' thoughts on this?

ccrama commented 8 years ago

Whoops

Nxt3 commented 8 years ago

@ccrama Is there a reason you can't use the ViewPager page for vertical albums? I figured that would help with that.

So you propose a setting that disables swiping back from everywhere? It's weird to disable swiping back from just two places--otherwise.

ccrama commented 8 years ago

No, just disabling from comments and albums. And it's not that I can't, but I'd have to turn the activity into a fragment and put that into a ViewPager, which is a lot of work

Nxt3 commented 8 years ago

I guess that makes sense. Still seems niche to me; if I didn't want to slide around--I'd use Sync or some other client.

Alexendoo commented 8 years ago

The vertical album one I do find myself doing occasionally but I think that's a slightly different issue, not sure I'd want to disable it entirely as I do like swipe anywhere to exit.

The issue would still be there with a fair few things with that, e.g. horizontal albums, shadowbox come to mind.

I don't think I'd want a setting added for disabling swiping to exit from 2nd + pages in those views (the original suggestion) rather pick a default and stick to it. That said I would enable it if it were an option. I can't think of a decent alternative. Maybe enable comment pane to load more comment threads? would alleviate it there at the very least as there's no swipe from the edge to exit

ccrama commented 8 years ago

This has been tweaked a lot since a few months ago, closing this for now.