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

Right Click doesn't work after logging into LastPass extension #5293

Closed
srirambv opened this issue Oct 31, 2016 · 2 comments · Fixed by #5298
Closed

Right Click doesn't work after logging into LastPass extension #5293

srirambv opened this issue Oct 31, 2016 · 2 comments · Fixed by #5298
Assignees
@darkdh
Labels
OS/Windows QA/checked-Linux QA/checked-macOS QA/checked-Win32 QA/checked-Win64
Milestone
0.12.8dev

Comments

@srirambv
Copy link
Collaborator

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

Describe the issue you encountered:
Right Click doesn't work after logging into LastPass extension

Expected behavior:
Should be able to right click and open context menu

  • Platform (Win7, 8, 10? macOS? Linux distro?):
    Windows 10 x64

  • Brave Version:
    0.12.8 Preview 1

  • Steps to reproduce:

    1. Enable LastPass from password managers in preferences
    2. Login to your LastPass account
    3. Right click on the page doesn't work, Console shows error

  • Screenshot if needed:
    image

  • Any related issues:
    cc: @bridiver
@srirambv srirambv added this to the 0.12.8dev milestone Oct 31, 2016
@darkdh darkdh self-assigned this Oct 31, 2016
darkdh added a commit that referenced this issue Oct 31, 2016
@darkdh
Undefined properties and nested folders handling
8970035 
fix #5291
fix #5293

Auditors: @bridiver, @bbondy

Test Plan:
1. Go to about:preferences#security
2. Select Lastpass as password manager
3. Login to Lastpass
4. Right click to show context menu
5. Nested menu of Lastpass should show
@darkdh darkdh mentioned this issue Oct 31, 2016
Merged
4 tasks
darkdh added a commit that referenced this issue Oct 31, 2016
@darkdh
Undefined properties and nested folders handling
64d594e 
fix #5291
fix #5293

requires brave/muon#85

Auditors: @bridiver, @bbondy

Test Plan:
1. Go to about:preferences#security
2. Select Lastpass as password manager
3. Login to Lastpass
4. Right click to show context menu
5. Nested menu of Lastpass should show
darkdh added a commit that referenced this issue Nov 1, 2016
@darkdh
Undefined properties and nested folders handling
6fb27a1 
fix #5291
fix #5293

requires brave/muon#85

Auditors: @bridiver, @bbondy

Test Plan:
1. Go to about:preferences#security
2. Select Lastpass as password manager
3. Login to Lastpass
4. Right click to show context menu
5. Nested menu of Lastpass should show
@srirambv
Copy link
Collaborator Author

srirambv commented Nov 1, 2016

Reopening the issue as change in pref and restarting browser causes the same issue.
5293

@srirambv srirambv reopened this Nov 1, 2016
@srirambv
Copy link
Collaborator Author

srirambv commented Nov 1, 2016

Closing the issue as unable to reproduce in preview 2 build.

@srirambv srirambv closed this as completed Nov 1, 2016
This was referenced Nov 2, 2016
Closed
Closed
Closed
This was referenced Nov 4, 2016
Closed
Closed
Closed
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@darkdh darkdh

Labels
OS/Windows QA/checked-Linux QA/checked-macOS QA/checked-Win32 QA/checked-Win64
Projects
None yet
Milestone
0.12.8dev
Development

Successfully merging a pull request may close this issue.

Undefined properties and nested folders handling brave/browser-laptop
5 participants
@bbondy @alexwykoff @luixxiul @darkdh @srirambv