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

Mouse cursor visible while in full screen #7966

Closed
Jacalz opened this issue Mar 29, 2017 · 20 comments
Closed

Mouse cursor visible while in full screen #7966

Jacalz opened this issue Mar 29, 2017 · 20 comments

Comments

@Jacalz
Copy link
Contributor

Jacalz commented Mar 29, 2017

Test plan

  1. Open a youtube video (Example: https://www.youtube.com/watch?v=ylTPgMi9SEo )
  2. Open fullscreen
  3. Mouve the mouse so it isn´t on the bar at the bottom
  4. Keep the mouse stationary

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

  • Describe the issue you encountered:
    Mouse is still visable after entering fullscreen in youtube...

  • Platform (Win7, 8, 10? macOS? Linux distro?):
    Windows 10Pro x64

  • Brave Version (revision SHA):
    0.13.5 but also found it working on 1.14.0 RC2

  • Steps to reproduce:

    1. Open a youtube video (Example: https://www.youtube.com/watch?v=ylTPgMi9SEo )
    2. Open fullscreen
    3. Mouve the mouse so it isn´t on the bar at the bottom
    4. Keep the mouse stationary
  • Actual result:
    Mouse is still visable

  • Expected result:
    Mouse should be hidden when not moving it

  • 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

@jonathansampson jonathansampson added this to the 0.14.2 milestone Apr 3, 2017
@jonathansampson jonathansampson changed the title Mouse visable on youtube in fullscreen on youtube Mouse cursor visible while in full screen Apr 3, 2017
@jonathansampson
Copy link
Collaborator

jonathansampson commented Apr 3, 2017

Thank you for filing!

Browser Behavior
Chrome Hides mouse
Firefox Hides mouse
Edge Hides mouse
Brave (electron/muon) Displays mouse
Slack (electron) Displays mouse

@bsclifton
Copy link
Member

Moving to 0.14.3

@bsclifton bsclifton modified the milestones: 0.14.3, 0.14.2 Apr 4, 2017
@bsclifton bsclifton added needs-owner ♞ This issue is tagged for an upcoming release but has no owner. and removed needs-owner ♞ This issue is tagged for an upcoming release but has no owner. labels Apr 4, 2017
@alexwykoff
Copy link
Contributor

Hey @srirambv if you can reproduce please let us know, thanks :)

@srirambv
Copy link
Collaborator

Confirmed cursor not hidden in full screen.

@alexwykoff
Copy link
Contributor

@srirambv could you check vs slack fullscreen?

@alexwykoff alexwykoff modified the milestones: 1.1.0, 0.15.1 Apr 25, 2017
@srirambv
Copy link
Collaborator

Same on slack fullscreen as well. Cursor is not hidden.

@srirambv
Copy link
Collaborator

srirambv commented May 8, 2017

I think this is fixed in 0.15.3 preview 3. I do not see mouse pointer when viewing full screen video. But will keep the issue open just in case.

@Jacalz
Copy link
Contributor Author

Jacalz commented May 8, 2017

Yes it looks as to be fixed with 0.15.3, I wonder what fixed it 👍 😕

@Jacalz
Copy link
Contributor Author

Jacalz commented May 8, 2017

If we found what commit fixed it we can have it in the release notes in my oppinion 👍

@jonathansampson
Copy link
Collaborator

@darkdh @bridiver Are either of you aware of a change that would have resolved this issue?

@bsclifton
Copy link
Member

bsclifton commented May 8, 2017

@Jacalz can you check if it still happens in 0.15.2? We did update Chromium in that release; perhaps it was fixed there (I don't believe we did anything to fix this, but I could be wrong)

@Jacalz
Copy link
Contributor Author

Jacalz commented May 8, 2017

Does not happen in 0.15.2 strange 😕 @bsclifton

@Jacalz
Copy link
Contributor Author

Jacalz commented May 8, 2017

Can somebody also confirm that it got fixed with 0.15.2 so we can close this? 😄
If so I can not belive my eyes that a chromium update fixed it, would never even think as that being possible so great catch @bsclifton 👍

@luixxiul
Copy link
Contributor

luixxiul commented May 8, 2017

I confirmed that, closing.

@luixxiul luixxiul closed this as completed May 8, 2017
@luixxiul luixxiul removed the needs-owner ♞ This issue is tagged for an upcoming release but has no owner. label May 8, 2017
@luixxiul luixxiul removed this from the 1.1.0 milestone May 8, 2017
@luixxiul luixxiul added this to the 0.15.3 milestone May 8, 2017
@bsclifton bsclifton removed this from the 0.15.3 milestone May 8, 2017
@bsclifton
Copy link
Member

@Jacalz not too unbelievable- a Chromium update also introduced it 😄 Thanks for checking all

@Jacalz
Copy link
Contributor Author

Jacalz commented May 8, 2017

Oh, had no idea that it was introduced by a chromium update 😅 No problem 🤗

@Jacalz
Copy link
Contributor Author

Jacalz commented May 8, 2017

Also wich release notes should this be put on? Since @bsclifton removed the milestone 😅

@bsclifton bsclifton added this to the 0.15.3 milestone May 8, 2017
@bsclifton
Copy link
Member

@Jacalz we can keep a milestone if we want release notes. Sorry- I had removed it since there wasn't any fix

@Jacalz
Copy link
Contributor Author

Jacalz commented May 8, 2017

Don't be sorry it was more meant as a question if the release notes include tag should be removed or not 😉 @bsclifton

@luixxiul
Copy link
Contributor

luixxiul commented May 9, 2017

I had removed it since there wasn't any fix

I would say that the app fixed itself ;-)

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