Regression: Brave: 0.12.1 - Lastpass doesn't open

[bitcoinuser]

Hi,

1- I left clicked to open menu.
2- Tried to open lastpass.

See the screenshots of 1 and 2

[bbondy]

@bridiver is this fixed?

[bridiver]

not sure, I haven't tested on linux

[srirambv]

Confirmed on Windows 10 too. Selecting user account from 1P mini app doesn't populate the user credentials.

[bsclifton]

@srirambv this is for LastPass- did you post in the wrong issue?

[bbondy]

@srirambv do we need another issue for it?

[aekeus]

Data point - LastPass is operating correctly on osx

[bridiver]

@srirambv - are you checking 1password or lastpass? Also, please check against RC1 because there are some extension buttons fixes from yesterday that I believe are in it. @bbondy can confirm

[srirambv]

My bad got confused. It was for 1Password. Will open a separate issue for that.

[srirambv]

Just an update on the 1Password. It was working in 0.12.1 preview1 build. Weirdly its now working on 0.12.1 Preview3 build as well. When checked in morning on a clean install of preview 3 didn't work at all.

[bitcoinuser]

Preview3 on Ubuntu:

[bridiver]

@bitcoinuser is that from the extension button or context menu?

[bitcoinuser]

Extension button.

Em 12/09/2016 23:29, "Brian Johnson" notifications@github.com escreveu:

@bitcoinuser https://github.com/bitcoinuser is that from the extension
button or context menu?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#3883 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABH86MgxSd3dBBUBKtkb2VjhtLlJN4LJks5qpgqdgaJpZM4J54A_
.

[ayumi]

On Ubuntu 16.04 w/ Brave 0.12.1dev preview 3, confirming LastPass extension looks weird as @bitcoinuser described:

[bitcoinuser]

@bridiver Context menu too. Same problem. Same screenshot.

[bridiver]

looks like the same thing happens on windows because neither of them get preferred size updates. I'll have to dig around to see what we can use

[srirambv]

Verified on Ubuntu. the screen flashes at the bottom and then loads proeprly

[bbondy]

@srirambv pls post a separate issue for that, it will be prioritized differently so we can release 0.12.1

[bridiver]

I'm not seeing what the issue is. The behavior in the screenshot looks correct to me

On Sep 15, 2016, at 5:23 AM, Brian R. Bondy notifications@github.com wrote:

@srirambv pls post a separate issue for that, it will be prioritized differently so we can release 0.12.1

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.