-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Feature Request: Improved Account Settings Multi-Account #1693
Comments
@CorneelDragon certainly this requests makes a lot sense so that we keep both clients as similar as possible. For tracking purposes, on the iOS for the moment, we only have 3 options:
owncloud/ios-legacy#640 (comment) @AndyScherzinger can you help us here? this is mainly a design thing, do you have any idea of how we may include a button to access to this menu instead of using the long press button? |
@rperezb sure thing, the drawer branch (#1559 / #1556) already implements an ownCloud account manager which allows delete and change password where we could also add the clear cache (even though I do not know what that means and if and how this should be triggered, but @davivel might be able to help me out with this. As for the screenshots on the actual branch for the drawer please see:
I am not sure about the iOS options you mentioned:
In case you want to implement this with the solution actual present in the latest release(s) you would need to extend the long press menu, which would be fine in my opinion and could then be moved to the #1559 implementation as a new menu item where I would then (see second screenshot link) remove the icons and add a 3-dot menu. |
for any reason, #1559 is the one 😄 we do have this pr on the backlog, meaning this it will be included in the short-term @CorneelDragon will it work this approach for you? it's slighly different to the iOS one:
If you click on the "Manage accounts" option, you may see: There are, for the moment, 2 options:
Is this approach ok for you? |
@AndyScherzinger about the Clear cache: this option is a new feature that we have just developen within the iOS app (included in the following version) |
Actually, I'm a bit confused, when I suggested 'Forget Account' (in the iOS topic), it was disregarded due to the need for consistency with Android and Desktop Clients. Now, I see 'key'-buttons for editing of credentials, a 'cross' for remove accounts, and 'management account' option in Settings, which is also accessible with a 'triangle'-button. I have my doubts about the proposed Android UI, just a 'settings'-button (like the 'three dots' in the iOS) with behind that a popup with 'Edit credentials / Clear Cache / Remove / Forget Account' would seem to me more intuitive. |
@CorneelDragon you are right as in inconsistent with the iOS UI but consistent with standard Android UI concepts :) |
@AndyScherzinger That could very well be the case (I'm no Android Developer). Nonetheless, from a user point-of-view and if I compare it with the Dropbox-app, I still have my doubts (but of course I respect an expert-decision of your side). My point was that in the iOS-thread the decision was made to choose the name 'Remove Account' instead of 'Forget Account' since this was consistent with Android. Now I see that Android will not have any button named 'Remove Account' since this is replaced with a cross. I reckon that an iOS-decision isn't the responsibility of the Android team, but there seems to be some lack of communication (or miscommunication) between the iOS and the Android team (the new iOS version will be sub-optimal in order to be consistent with the old Android version). |
@CorneelDragon no worries :) I still don't understand it :( What do you mean be refering to Dropbox? TO me it seams Dropbox is not multi account capable. So the better choices to take a look at would be the Play store app, GMail, Google Drive, G+, Inbox, ...
This is a misinterpretation. The triangle just toggles the menus below With this new drawer there won't be any account management in settings anymore but a separated one, just like any Google release app and also like the Android account management itself. The icons might be removed anyways and there will be just a three-dot menu in the account list, since it is likely that there will be more than just two actions. As for consistency it is not necessarily (always) the goal to be consistent UI-wise but platform-consistent. |
@AndyScherzinger Thanks for correcting my mis-interpretation :) Your point on Dropbox not supporting multi-acount is fair and, as said, I respect any decision in this regard and I see the importance of being consistent with other Android apps over being consistent with the iOS. Regarding my second-point (the decision regarding iOS being consistent with old Android-version) maybe @rperezb can clear this up :) |
@CorneelDragon glad we got things sorted and I hope this is all fine with you :) And yes @rperezb is probably the one to clear things up with regards to iOS |
@CorneelDragon you are right, I missed this pr by the time when we were talking about the change on iOS. Where remove is also used @dragotin your input? |
My thoughts:
|
@rperezb Fair enough, (and I don't want to keep bugging about such a small thing). I do hope that if the desktop is updated, 'forget account' will come through. Anyway, thanks for the responses on this and iOS and case be closed :) |
I wouldn't close, we can take this issue as the starting point for reviewing the layout of settings, having in mind the multiple accounts. |
Expected behaviour
Within 'settings' in the iOS-app (multi-account enabled) users see a 'three dot button' next to their name, when they click this a pop-up menu opens with: Edit credentials / Clear Cache / Log Out / Remove Account. See the discussion and mock-ups in: https://github.com/owncloud/ios/issues/640#issuecomment-220978127
Actual behaviour
In the android users still have to press and hold on to their name. In addition, there is no log-out or clear cache option.
Feature Request
Could we get a similar UI in the android app?
The text was updated successfully, but these errors were encountered: