-
Notifications
You must be signed in to change notification settings - Fork 7.6k
Mac only: Quick View popover may not appear after resizing window or going fullscreen #3570
Comments
It turns out to be intermittent issue on my mac only. I can't reproduce it on my win7, but on mac I can even reproduce it with older build without your changes. |
We just need to listen for resize event and hide preview on resize. |
Can you reproduce it? It's not hiding issue. It's not showing up after resizing sometimes, but not all the times. |
What I'm seeing is that popover is not dismissed when resizing. You can resize it so popover is still showing, but is off the screen so you can't see it. Mouse over won't show it because it's already showing. Mouse over something else and move mouse back, and popover will reposition on the screen. |
See if this branch fixes what you're seeing: #3577 |
Nope, I still see the issue of not showing quick view. But your changes did hide the quick view. |
I backed out my fix because the mouseout handler now hides preview before you can start resizing. I can't repro on Mac10.8. |
@RaymondLim , @peterflynn I just noticed that this was still assigned to me. Removing myself since I cannot reproduce this one. |
I can repro consistently by going into fullscreen mode. After entering fullscreen mode, all mouse over/move events are missing. In addition to quick view not working, you don't see a resize cursor, and tooltips don't appear. |
I am getting quick view in full screen mode, but it's in the wrong place! It's showing up 300px or so above where the mouse is. |
Very intermittant, though. Seems to have something to do with where last click/mouse position was. Use mouse-wheel to scroll up/down to see different things show up :) |
@redmunds I'm seeing the symptom that you're seeing only if I use two-fingers-click (and/or scroll) on an element that can be shown as Quick View. |
@RaymondLim I'm using a mouse. |
Tagging CEF label as this is not really an issue in Quick View, but in cef. |
Updated title and added workaround to description. |
Is there a known CEF bug already filed? |
Reviewed @RaymondLim - Raymond pleas see if there is an existing CEF bug or file one if not. |
This is fixed CEF3.1453.1255 |
FBNC to @RaymondLim |
Fix confirmed. |
Result: No popover. If you still see popover, then resize the window to the original width and repeat step 3 and 4.
Workaround: Move the mouse to the top of the screen.
The text was updated successfully, but these errors were encountered: