Skip to content

toggle pointer-events with suspension#2827

Merged
MichMich merged 1 commit intoMagicMirrorOrg:developfrom
shin10:develop
Mar 26, 2022
Merged

toggle pointer-events with suspension#2827
MichMich merged 1 commit intoMagicMirrorOrg:developfrom
shin10:develop

Conversation

@shin10
Copy link
Contributor

@shin10 shin10 commented Mar 25, 2022

Hi,

I added two classes for the hidden/visible modules to toggle the pointer-events with the new .hidden class.
Further I removed the one with the universal selector, which made hidden modules and just anything in them stealing pointer events, if they were placed in any fullscreen area.

Copy link
Collaborator

@rejas rejas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, looks ok although I cannot test it myself. You will need to add a CHANGELOG entry though before it gets merged.

@codecov-commenter
Copy link

Codecov Report

Merging #2827 (93581ca) into develop (c5e9c9a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #2827   +/-   ##
========================================
  Coverage    65.26%   65.26%           
========================================
  Files            8        8           
  Lines          285      285           
========================================
  Hits           186      186           
  Misses          99       99           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c5e9c9a...93581ca. Read the comment docs.

@MichMich MichMich merged commit a143ebc into MagicMirrorOrg:develop Mar 26, 2022
@shin10
Copy link
Contributor Author

shin10 commented Mar 26, 2022

Sure, thanks.
By the way. I'm currently creating some modules myself. Have to do some refactoring today but I'd like to contribute it to the 3rd party list. Whom might I contact for reviewing them?

@MichMich
Copy link
Collaborator

You can just add them to the wiki. It's open for everybody to edit.

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

Successfully merging this pull request may close these issues.

4 participants