Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Window locations need to be adapted to current display setup #7735

Open
1 task done
farnulfo opened this issue Feb 14, 2023 · 4 comments
Open
1 task done

Window locations need to be adapted to current display setup #7735

farnulfo opened this issue Feb 14, 2023 · 4 comments

Comments

@farnulfo
Copy link

Is your feature request related to a problem? Please describe.

Laptop connected to a dock with 2 screens
Launch OWASP ZAP
Move ZAP Windows to the second screens.
Shutdown ZAP and the latop.
Start the laptop without any others screens
Start ZAP : the main window is not visible on laptop screen, you must use the Alt+Space shortcut and Move to move the ZAP windows on the visible screen.

Describe the solution you'd like

At startup, the previous recorded window location needs to be checked and modified with the current display setup:
if the x position of ZAP window is no more in current width screen size it need to be modified to 0.
Then the window will be visible to the user

Describe alternatives you've considered

use the Alt+Space shortcut and Move to move the ZAP windows on the visible screen.

Screenshots

No response

Additional context

No response

Would you like to help fix this issue?

  • Yes
@thc202 thc202 changed the title Previous recorded OWASP ZAP main window location need to be adapted to current display setup when starting Window locations need to be adapted to current display setup Feb 14, 2023
@andremralves
Copy link

I tried to reproduce the error following the steps using a second screen on a Linux (Ubuntu) OS, but it's working as intended for me. When I reopen ZAP, the main window appears in my laptop's screen. Could you tell witch OS are you using?

@farnulfo
Copy link
Author

farnulfo commented Apr 20, 2023

Windows 10

@farnulfo
Copy link
Author

I can make a new test but just tell me what file / data I need to check

@thc202
Copy link
Member

thc202 commented Apr 20, 2023

This was not yet fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

3 participants