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

🐞 [BUG] Multi-screen Maximized Transparent Window, Click Throughable -wrong reseted Clickable Bounds #4888

Closed
wis opened this issue May 24, 2016 · 16 comments

Comments

@wis
Copy link

wis commented May 24, 2016

@jtg-gg #4846 I installed the latest nightly build, and tried it, the issue wasn't fixed.

here is a 10 Seconds video with explanation https://www.youtube.com/watch?v=0Y8MwzwIFUc

  • the bug is not in my code, the window covers all the screens, I can drag the yellow rectangle to the far left of the second monitor and the white 50 px thick solid border is to prove that the window extends to all the screens.
  • the new clickable bounds of the window gets reseted to wrong clickable bounds, which seems to be the bounds of the primary screen, but the bounds of the window does not change, see the white border, in the Screen shot. I had some guesses of the bug origin, guess 1 ::ResetWindowRegion || guess 2 || guess 3 || guess 4
  • see the screenshot and explanation,
    the horizontal red line seperates the two monitors, and the blue line marks the far right end of the updated clickable bounds, they are the exact same bounds of the primary screen of the system, I counted the pixels with the Paint select tool..
  • the clickable bounds update seems to happen on mouse up event, because I can drag the yellow rectangle to the far far right with only one drag and then I can't click it.
@wis wis changed the title Transparent Window,Click Through,Multi-Monitor,Fullscreen -wrong updated Clickable Bounds 🪲 [BUG] Multi-screen Maximized Transparent Window, Click Throughable -wrong updated Clickable Bounds May 24, 2016
@wis wis changed the title 🪲 [BUG] Multi-screen Maximized Transparent Window, Click Throughable -wrong updated Clickable Bounds 🐞 [BUG] Multi-screen Maximized Transparent Window, Click Throughable -wrong updated Clickable Bounds May 24, 2016
@wis wis changed the title 🐞 [BUG] Multi-screen Maximized Transparent Window, Click Throughable -wrong updated Clickable Bounds 🐞 [BUG] Multi-screen Maximized Transparent Window, Click Throughable -wrong reseted Clickable Bounds May 24, 2016
@rogerwang
Copy link
Member

CC @jtg-gg

@jtg-gg
Copy link
Member

jtg-gg commented May 27, 2016

@wesamco I'm a bit busy now, will look into it next or next next week

@jtg-gg
Copy link
Member

jtg-gg commented May 31, 2016

@wesamco I can't access the youtube video, it is private, can you make it public ?

@wis
Copy link
Author

wis commented May 31, 2016

@jtg-gg Done.

@jtg-gg
Copy link
Member

jtg-gg commented Jun 2, 2016

@wesamco I think I'll need you sample code.
I was using testing the test app from #4846, resizing the window size to extend 2 monitor when i click, I was able to click the button and right end my 2nd monitor

@jtg-gg
Copy link
Member

jtg-gg commented Jun 6, 2016

@wesamco I've this error when running your sample code
chrome-extension://anaelpolhngelhbmhepmjcfmmkjdngid/index.html:26
$(function () {
^

ReferenceError: $ is not defined
at chrome-extension://anaelpolhngelhbmhepmjcfmmkjdngid/index.html:26:1

and what should I do with the jquery-ui-1.11.4.zip ?

@wis
Copy link
Author

wis commented Jun 6, 2016

@jtg-gg the $ is the Jquery javascript framework object,
download the jquery libraries I linked from the official jquery website,
this error is caused because they are not being imported to index.html with the script tags

<script src="jquery-2.1.3.min.js"></script>
  <script src="jquery-ui.js"></script>

because you didn't download the libraries and put them in the project folder, with package.json.

@jtg-gg
Copy link
Member

jtg-gg commented Jun 6, 2016

I'm using now

  <script src="https://code.jquery.com/jquery-2.2.4.min.js"></script>
  <script src="https://ajax.aspnetcdn.com/ajax/jquery.ui/1.11.4/jquery-ui.min.js"></script>

now the error is gone, but I don't see any yellow box to drag ?

I've downloaded them and extract them in the same folder as package.json
but nw.js can't load them, then I use the online version

@jtg-gg
Copy link
Member

jtg-gg commented Jun 6, 2016

still nothing displayed, just white canvas when i disable transparency
when I enable, nothing

@wis
Copy link
Author

wis commented Jun 6, 2016

I am making a repro project repo, will be ready in minutes.

@jtg-gg
Copy link
Member

jtg-gg commented Jun 6, 2016

thx @wesamco I can see and drag the yellow box
but, I can't reproduce your bug, I can drag the yellow box on both end of my monitor
can you share your monitor configuration ?

my main screen 1st is on left side, my 2nd is on right side
the 1st monitor is 1920x1200
the 2nd monitor is 1920x1080

well I'm running windows on VM, so I can't swap the left and right monitor to test :(

@wis
Copy link
Author

wis commented Jun 6, 2016

image
still working exactly like in the video

@wis
Copy link
Author

wis commented Jun 6, 2016

@jtg-gg is it really vital to debug it and step in code? I described what caused it the bug in all details, can't you guess what might have caused it?
are you running Windows 10? could it be Desktop Window Manager (dwm) related?

@jtg-gg
Copy link
Member

jtg-gg commented Jun 6, 2016

@wesamco I still can't reproduce your bug =( have you try it on another machine ?
I can drag the yellow box on both end of the screen
yes I'm using win10, I've changed my resolution and display configuration like yours

@jtg-gg
Copy link
Member

jtg-gg commented Jun 8, 2016

@wesamco have you tried on another machine?

@jtg-gg
Copy link
Member

jtg-gg commented Jun 10, 2016

closing this issue since I can't reproduce the bug

@jtg-gg jtg-gg closed this as completed Jun 10, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants