Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

"remember password" odd behavior #2957

Closed
diegodoval opened this issue Aug 4, 2016 · 10 comments
Closed

"remember password" odd behavior #2957

diegodoval opened this issue Aug 4, 2016 · 10 comments

Comments

@diegodoval
Copy link

diegodoval commented Aug 4, 2016

Did you search for similar issues before submitting this one?
yes

Describe the issue you encountered:
-on some sites the bar flashes in and disappears, on others it never shows up
-on developer.apple.com the 'remember this password?' bar flashes in and goes away. When logging out, it appears again. (verified multiple times, have screen recording if necessary)

That aside: sites that don’t show ‘remember password’ at all
citibank.com
imgur.com
gfycat.com

Sites that behave correctly:
reddit.com
amazon.com

Expected behavior:

@diracdeltas
Copy link
Member

the flashing-and-disappearing issue may be due to d1b09e4 if the post-login URL is a different origin than the login page

@bsclifton bsclifton added feature/password-manager needs-info Another team member needs information from the PR/issue opener. labels Oct 12, 2016
@bsclifton
Copy link
Member

+1 from support

When I use a new password and the pop-up asks me if I
want to save it, the pop-up goes away too fast. I don't have enough time
to drag my mouse over to answer yes.

@alexwykoff
Copy link
Contributor

@luixxiul
Copy link
Contributor

luixxiul commented Jan 8, 2017

@luixxiul luixxiul added this to the Backlog milestone Jan 8, 2017
@diracdeltas
Copy link
Member

seems like a lot of complaints compared to the previous behavior where the password message box lingered until you closed it manually. i can change it back.

@bradleyrichter
Copy link
Contributor

@diracdeltas Is it now tab specific? Or window specific?

Perhaps it can auto-close when the domain changes instead of being page level?

@diracdeltas
Copy link
Member

Perhaps it can auto-close when the domain changes instead of being page level?

that was the cause of this issue actually

@bradleyrichter
Copy link
Contributor

Ah..so the only choice is to keep it around until the tab is closed? (assuming it was not acted upon)

@diracdeltas
Copy link
Member

right now it doesn't close unless you close it. at the time this was implemented, there was no good way to specify a particular frame that the message box belonged to. may be possible now.

bkilrain pushed a commit to bkilrain/browser-laptop that referenced this issue Jan 13, 2017
fix brave#2957
note that this partially un-fixes brave#1928.

Auditors: @alexwykoff

Test Plan:
1. go to developer.apple.com
2. click on 'Account' in the top right
3. enter your apple username and password
4. the login page should redirect to https://developer.apple.com/
5. after the redirect, you should still see a password manager dialog from Brave that says 'Would you like Brave to remember the password for [username] on https://idmsa.apple.com?'
@luixxiul
Copy link
Contributor

Test plan: b466a44

@luixxiul luixxiul added QA/test-plan-specified release-notes/include and removed needs-info Another team member needs information from the PR/issue opener. labels Jan 16, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.