Skip to content
This repository has been archived by the owner on Oct 6, 2021. It is now read-only.

Switching accounts in saver broken on Chrome #17

Closed
lutherism opened this issue Mar 26, 2015 · 9 comments
Closed

Switching accounts in saver broken on Chrome #17

lutherism opened this issue Mar 26, 2015 · 9 comments

Comments

@lutherism
Copy link

Transposed from an internally filed bug:

Steps to reproduce:

Open Kloudless Saver.
Associate an account with Dropbox.
Save to Dropbox.
Reopen Kloudless Saver.
Click account switcher dropdown and then click "Accounts".

Expected: Able to pick another account or another cloud service.

Observed: No response.

Note: In the uploader, this isn't happening. You can work around this by clicking on the first dropbox account and then clicking on "Accounts", but it's very unintuitive, and should work directly.

@vinodc
Copy link
Member

vinodc commented Mar 26, 2015

I'm having a little trouble reproducing this. Is it reproducible at http://jsfiddle.net/pseudonumos/PB565/embedded/result/ ? If not, but it's reproducible in your Saver, it might be a particular sequence of method calls that results in the document.location.hash of the iframe being set to "#/accounts" without the view updating to the Accounts page. That would cause the symptoms being seen. If it's reproducible on an internal test server that I could get access to, it would be great if you could email me a link to it (vinod@kloudless.com). Thanks.

@lutherism
Copy link
Author

Sorry for not getting back to this till now.

Yes, I can reproduce this on your JSFiddle link, following the listed steps exactly. Here's a screenshot of where it's broken. From that state, if I click the Accounts item in the dropdown, nothing happens.

http://imgur.com/5wAO2uh

This also fails in FF for me. I'd guess there's some relative path problem?

@vinodc
Copy link
Member

vinodc commented Mar 30, 2015

Thanks for the update Alex. I'm still encountering some trouble reproducing
here. Could you let me know which browser you were using?

When I re-open the Saver, it opens to the Accounts page rather than the
file explorer page, so the document.location.hash matches the view
currently being displayed and there is no Accounts button to click on.
Could you help me troubleshoot this via the Chrome Inspector? Could you
switch the Inspector's window to point to the explorer.html iframe and then
let me know what document.location.hash points to immediately after you
re-open the Explorer and it displays the list of files? Thanks.

On Mon, Mar 30, 2015 at 11:35 AM, Alex Jansen notifications@github.com
wrote:

Sorry for not getting back to this till now.

Yes, I can reproduce this on JSFiddle, following the listed steps exactly.
Here's a screenshot of the where it's broken. From that state, if I click
the Accounts item in the dropdown, nothing happens.

http://imgur.com/5wAO2uh


Reply to this email directly or view it on GitHub
#17 (comment)
.

@lutherism
Copy link
Author

It does not pop into the account page for us on any machine if you have an account logged in. Looking into the document location now.

@lutherism
Copy link
Author

The iframe's document.location.hash is "#/accounts" when I open it while logged in. And it displays the file browsing view.

@lutherism
Copy link
Author

When I click an account name (as part of the work around) the hash becomes "#/files/59685949" and the view stays on the file browser.

@aschiopu
Copy link

@vinodc

steps to reproduce:

  1. visit http://jsfiddle.net/pseudonumos/PB565/embedded/result/
  2. click save a file
  3. pick dropbox
  4. authenticate
  5. click save in order to save the file
  6. click save a file again (should open up in the dropbox folder)
  7. click top left where the account dropdown is .. by default showing you the email. 2 options should come up. The email and "Accounts"
  8. Click "Accounts".

Result: Does nothing. It closes the dropdown but otherwise does nothing.

steps to workaround:
9. Click the accounts dropdown again.
10. This time click on your email. (it will appear like nothing happened- the dropdown closed)
11. Click the accounts dropdown again.
12. Now Click "Account" and it will work.

@vinodc
Copy link
Member

vinodc commented Apr 2, 2015

Thanks Alexei, Alex. I've reproduced the problem and am working on a
solution. Will keep you updated.

On Tue, Mar 31, 2015 at 11:42 AM, Alexei Schiopu notifications@github.com
wrote:

@vinodc https://github.com/vinodc

steps to reproduce:

  1. visit http://jsfiddle.net/pseudonumos/PB565/embedded/result/
  2. click save a file
  3. pick dropbox
  4. authenticate
  5. click save in order to save the file
  6. click save a file again (should open up in the dropbox folder)
  7. click top left where the account dropdown is .. by default showing you
    the email. 2 options should come up. The email and "Accounts"
  8. Click "Accounts".

Result: Does nothing. It closes the dropdown but otherwise does nothing.

steps to workaround:
9. Click the accounts dropdown again.
10. This time click on your email. (it will appear like nothing happened-
the dropdown closed)
11. Click the accounts dropdown again.
12. Now Click "Account" and it will work.


Reply to this email directly or view it on GitHub
#17 (comment)
.

vinodc added a commit that referenced this issue Apr 2, 2015
…n the files page was displayed in some cases previously (see #17).
vinodc added a commit that referenced this issue Apr 6, 2015
@vinodc
Copy link
Member

vinodc commented Apr 9, 2015

This issue should now be resolved in production. Thanks for your assistance in trouble-shooting it.

@vinodc vinodc closed this as completed Apr 9, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants