Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Usability Improvement: Sessions vs. App Pins #6365

Closed
ho1ger opened this issue Sep 5, 2017 · 5 comments
Closed

Usability Improvement: Sessions vs. App Pins #6365

ho1ger opened this issue Sep 5, 2017 · 5 comments
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap design Design, UI, UX, etc. enhancement
Milestone

Comments

@ho1ger
Copy link

ho1ger commented Sep 5, 2017

In NC 12 the views for sessions and App Pins were merged. Now you can revoke App Pins using a (hidden) context menu from the list of current/previous sessions. This has caused some irritations already, c.f. nextcloud/twofactor_totp#186 (comment) and #6075 (comment). It seems people do not find the App Pins in the sessions list.

Let me explain my thoughts about this (copied from my post from the other thread):

  1. Imho a session and an App Pin are entirely different concepts. For me an App Pin is something quite persistent that typically lives as long as you want to use a device. A session, however, is a communication relationship between a server and a device that is created and destroyed many many times.
  2. Furthermore, not all sessions can be associated to an App Pin; only those sessions started by an app that doesn't support TOTP.

Hence, I never got the idea to search App Pins in the sessions menu.

If we want to keep the menus merged, I suggest to mitigate the issue by permanently showing something in the sessions list that indicates that the list has more capabilities than being a list of sessions. However, as I argued, I think the new combination is not really logical and it might confuse people.

@jancborchardt, @ChristophWurst

@ChristophWurst
Copy link
Member

Thanks for filing this ticket here, @ho1ger!

@ChristophWurst ChristophWurst added 0. Needs triage Pending check for reproducibility or if it fits our roadmap design Design, UI, UX, etc. labels Sep 5, 2017
@jancborchardt
Copy link
Member

Maybe this will be resolved when we add icons to the list? Sessions will have device-specific icons, and app pins will have a separate one.

But yeah in general the list should be kept together, sorted by most recently used first.

@AND51
Copy link

AND51 commented Jan 28, 2018

I agree to @ho1ger ; I installed new NC 13 Beta yesterday and expected to find the old, seperate App PIN list, to clean up my PINs. Hovering the mouse over the sessions by accident, I revealed the three-dots menu. This was not intuitive at all.

Acutally, I would vote for separating PINs and sessions again. I tell you why. Please, have a look at my sessions list:

image

As you can see, I have an iPhone syncing CalDAV and CardDAV. You see the same user-agent, but multiple times. In NC 12 (and I believe also in NC 11) this list grew incredibly long. Dozens of entrys from my iPhone – the same iPhone. The same thing happens/happened with Vivaldi. Vivaldi creates different sessions. Here it is "this session" and 11, 12, 12 and 14 hours ago.

If you want to withdraw an App PIN, say Vivaldi, which entry do you click? And what if you want to delete an old App PIN when you have 50+ entries? Therefore, there should be a better UI as @jancborchardt mentioned. I think then the idea of merging both lists is fine.

/// Update:

If I'm right then all Apps which use a PIN won't produce multiple entries. I cleared the whole list and gave everything but the browser a PIN. But at least one question stays, "how will the list look when browsers which don't use PINs pollute the list?"

@ChristophWurst
Copy link
Member

list grew incredibly long

Fixed by #6544

@nextcloud-bot nextcloud-bot added the stale Ticket or PR with no recent activity label Jun 20, 2018
@jancborchardt
Copy link
Member

The menu is also not hidden anymore but permanently shown. Closing this. :)

@nextcloud-bot nextcloud-bot removed the stale Ticket or PR with no recent activity label Jun 21, 2018
@jancborchardt jancborchardt added this to the Nextcloud 14 milestone Jun 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap design Design, UI, UX, etc. enhancement
Projects
None yet
Development

No branches or pull requests

6 participants