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

Extension Popup Sized, Positioned Incorrectly #7051

Closed
jonathansampson opened this issue Feb 4, 2017 · 22 comments
Closed

Extension Popup Sized, Positioned Incorrectly #7051

jonathansampson opened this issue Feb 4, 2017 · 22 comments
Assignees
Labels
bug/dpi/monitor bug feature/extensions/LastPass feature/extensions fixed-with-brave-core This issue will automatically resolved with the replacement of Muon with Brave Core. priority/P4 Minor loss of function. Workaround usually present. wontfix

Comments

@jonathansampson
Copy link
Collaborator

jonathansampson commented Feb 4, 2017

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

  • Describe the issue you encountered:
    The extension popup window may have the wrong size and location depending on the display scaling.

  • Platform (Win7, 8, 10? macOS? Linux distro?):
    Windows 10 (SurfaceBook, 200%)

  • Brave Version (revision SHA):
    50443ad

  • Steps to reproduce:

    1. Enable LastPass
    2. Click LastPass button in top bar
    3. Note location of popup window
  • Actual result:
    Window is size and positioned incorrectly.

  • Expected result:
    Window should be sized and positioned correctly.

  • Will the steps above reproduce in a fresh profile? If not what other info can be added?
    Yes

  • Is this an issue in the currently released version?
    Yes

  • Can this issue be consistently reproduced?
    Yes

  • Screenshot if needed:
    image

@josiah-keller
Copy link
Contributor

I see a different but similar result - potentially the same issue? (150% scaling)

lastpass-before

@jonathansampson
Copy link
Collaborator Author

jonathansampson commented Feb 4, 2017

@josiah-keller I think it's the same issue; I'm at home now, with my Surface Book docked. On my external display (which is at 100%), the popup is sized and positioned correctly. If I drag Brave to my Surface Book screen (which is at 200%), the popup is sized and positioned incorrectly.

transitions

@josiah-keller
Copy link
Contributor

@jonathansampson Yep, same case with my 100% monitor.

@jonathansampson
Copy link
Collaborator Author

Adding --enable-use-zoom-for-dsf=false to my shortcut resolves the issue. Thanks, @bsclifton, for the tip.

@bbondy
Copy link
Member

bbondy commented Feb 5, 2017

Moving to 0.13.3 since it's already a live problem, we're only blocking at this point on things that are critical or that make what's released worse, @bsclifton maybe you can see what's going on with DPI with extensions?

@bbondy bbondy modified the milestones: 0.13.3, 0.13.2 Feb 5, 2017
@bsclifton bsclifton self-assigned this Feb 5, 2017
@srirambv
Copy link
Collaborator

srirambv commented Feb 8, 2017

One more report from community user on high resolution display
https://community.brave.com/t/lastpass-extension-unusable-on-high-res-screen/1101

@bbondy bbondy modified the milestones: 0.13.5, 0.13.6 Feb 15, 2017
@Dreae
Copy link

Dreae commented Feb 27, 2017

Chiming in, I'm having the same issue on a MBP running linux. Adding --enable-use-zoom-for-dsf=false did not solve the problem in my case. Brave version 71d8ffc Problem is identical to @jonathansampson's screenshot for his 200% display.

@jonathansampson
Copy link
Collaborator Author

@Dreae, which version of Brave are you testing?

@Dreae
Copy link

Dreae commented Feb 28, 2017

@jonathansampson 0.13.4 - 71d8ffc on linux

@jonathansampson
Copy link
Collaborator Author

@Dreae Would you be willing to clone/setup the browser-laptop project, and test a particular fix? I'm happy to walk you though the process if you'd like.

@josiah-keller Same question for you :)

@josiah-keller
Copy link
Contributor

@jonathansampson Sure, but I just got a new machine so I'll get back to you in a few years after VS installs.

@jonathansampson
Copy link
Collaborator Author

@josiah-keller Happy to proceed whenever you're ready (and have confirmed that your new machine reproduces the issue observed here in this thread).

@stshank
Copy link

stshank commented Apr 3, 2017

I'm experiencing this issue too on a Microsoft Surface Book, Windows 10. Clicking the LastPass icon pops up an unusable dialog box. I'm viewing the screen at 250% as configured in the Settings app.
screenshot 2017-04-02 21 54 00

@jonathansampson
Copy link
Collaborator Author

@stshank Can you test the advice given here: #7051 (comment)?

@stshank
Copy link

stshank commented Apr 3, 2017

@jonathansampson Will try. What's the path on Windows to find brave.exe?

@bsclifton
Copy link
Member

bsclifton commented Apr 8, 2017

@stshank sorry for the super long turn-around; the Brave executable is going to be at %userprofile%\AppData\Local\brave. You should be able to cheat and right click > properties on the shortcut also

@jonathansampson
Copy link
Collaborator Author

jonathansampson commented May 14, 2017

This issue is revealed via the new Toolbar and UI elements scale setting.

Scaling toolbar breaks LastPass popup

@jonathansampson jonathansampson added this to the 1.0.0 milestone May 14, 2017
@dotbloup
Copy link

dotbloup commented May 24, 2017

Similar problem on Mac OS. except i see a transparent rectangle on the lower right corner after clicking on the extension icon from lastpass. rev: af7ef42

screen shot 2017-05-24 at 12 07 05

@trosel
Copy link

trosel commented Jul 6, 2017

I need my monitor to be 150% in the windows display settings. Lastpass is unusable on Brave when at 150%.

Thus, I sadly cannot use Brave :(

@trosel
Copy link

trosel commented Jul 6, 2017

@jonathansampson When selecting large or supersize for the UI elements, the lastpass white box gets MASSIVE but the actual lastpass content stays exactly the same size.

@jonathansampson jonathansampson changed the title LastPass' popup opens off-screen Extension Popup Sized, Positioned Incorrectly Aug 29, 2017
@alexwykoff alexwykoff modified the milestones: 0.22.x, 1.0.0 Aug 29, 2017
@bsclifton
Copy link
Member

Also affects Metamask and bitwarden:

image

image

@bsclifton bsclifton added the priority/P4 Minor loss of function. Workaround usually present. label Oct 16, 2017
@bbondy bbondy modified the milestones: 0.22.x (Nightly Channel), Backlog Oct 25, 2017
@bbondy bbondy modified the milestones: Triage Backlog, Prioritized Backlog Nov 2, 2017
@bsclifton bsclifton added the addressed-with-brave-core Needs confirmation, but this issue may be resolved with Brave Core. label Aug 24, 2018
@bsclifton bsclifton removed this from the Backlog (Prioritized) milestone Aug 24, 2018
@bsclifton bsclifton added wontfix fixed-with-brave-core This issue will automatically resolved with the replacement of Muon with Brave Core. and removed addressed-with-brave-core Needs confirmation, but this issue may be resolved with Brave Core. labels Aug 24, 2018
@bsclifton
Copy link
Member

This should be fixed with brave-core (would be good to verify- cc: @jonathansampson

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug/dpi/monitor bug feature/extensions/LastPass feature/extensions fixed-with-brave-core This issue will automatically resolved with the replacement of Muon with Brave Core. priority/P4 Minor loss of function. Workaround usually present. wontfix
Projects
None yet
Development

No branches or pull requests