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

X.H.EwmhDesktops: Avoid some unnecessary refreshes #371

Closed
wants to merge 2 commits into from

Conversation

liskin
Copy link
Member

@liskin liskin commented Aug 25, 2020

Description

Current version of Steam sends _NET_ACTIVE_WINDOW ClientMessage for
every mouse click which results in a lot of border blinking. Adopt
XMonad.Util.PureX to avoid refreshing unless something actually changes.

Checklist

  • I've read CONTRIBUTING.md

  • I tested my changes with xmonad-testing

  • I updated the CHANGES.md file

  • I updated the XMonad.Doc.Extending file (if appropriate)

@liskin
Copy link
Member Author

liskin commented Aug 25, 2020

The second commit makes it possible to add non-flicker focusNth keybindings. As GitHub doesn't really support dependent pull requests, I'm just putting it here.

@byorgey Are we doing the "two pairs of eyes" reviews in between committers or am I supposed to just commit trivial stuff like this to master?

Current version of Steam sends _NET_ACTIVE_WINDOW ClientMessage for
every mouse click which results in a lot of border blinking. Adopt
XMonad.Util.PureX to avoid refreshing unless something actually changes.
liskin added a commit to liskin/xmonad-contrib that referenced this pull request Nov 5, 2020
Current version of Steam sends _NET_ACTIVE_WINDOW ClientMessage for
every mouse click which results in a lot of border blinking.

Ignore requests that would result in no change.

Related: xmonad#371
@liskin
Copy link
Member Author

liskin commented Nov 5, 2020

Replaced by a commit in #399, closing.

@liskin liskin closed this Nov 5, 2020
@liskin liskin deleted the ewmh-flicker branch November 5, 2020 01:33
liskin added a commit to liskin/xmonad-contrib that referenced this pull request Nov 6, 2020
Current version of Steam sends _NET_ACTIVE_WINDOW ClientMessage for
every mouse click which results in a lot of border blinking.

Ignore requests that would result in no change.

Related: xmonad#371
liskin added a commit to liskin/xmonad-contrib that referenced this pull request Nov 15, 2020
Current version of Steam sends _NET_ACTIVE_WINDOW ClientMessage for
every mouse click which results in a lot of border blinking.

Ignore requests that would result in no change.

Related: xmonad#371
liskin added a commit to liskin/xmonad-contrib that referenced this pull request Nov 16, 2020
Current version of Steam sends _NET_ACTIVE_WINDOW ClientMessage for
every mouse click which results in a lot of border blinking.

Ignore requests that would result in no change.

Related: xmonad#371
liskin added a commit to liskin/xmonad-contrib that referenced this pull request Jan 25, 2021
Current version of Steam sends _NET_ACTIVE_WINDOW ClientMessage for
every mouse click which results in a lot of border blinking.

Ignore requests that would result in no change.

Related: xmonad#371
liskin added a commit to liskin/xmonad-contrib that referenced this pull request Jan 30, 2021
Current version of Steam sends _NET_ACTIVE_WINDOW ClientMessage for
every mouse click which results in a lot of border blinking.

Ignore requests that would result in no change.

Related: xmonad#371
liskin added a commit to liskin/xmonad-contrib that referenced this pull request Feb 7, 2021
Current version of Steam sends _NET_ACTIVE_WINDOW ClientMessage for
every mouse click which results in a lot of border blinking.

Ignore requests that would result in no change.

Related: xmonad#371
liskin added a commit to liskin/xmonad-contrib that referenced this pull request Mar 23, 2021
Current version of Steam sends _NET_ACTIVE_WINDOW ClientMessage for
every mouse click which results in a lot of border blinking.

Ignore requests that would result in no change.

Related: xmonad#371
TheMC47 pushed a commit to TheMC47/xmonad-contrib that referenced this pull request Apr 7, 2021
Current version of Steam sends _NET_ACTIVE_WINDOW ClientMessage for
every mouse click which results in a lot of border blinking.

Ignore requests that would result in no change.

Related: xmonad#371
TheMC47 pushed a commit to TheMC47/xmonad-contrib that referenced this pull request Apr 13, 2021
Current version of Steam sends _NET_ACTIVE_WINDOW ClientMessage for
every mouse click which results in a lot of border blinking.

Ignore requests that would result in no change.

Related: xmonad#371
TheMC47 pushed a commit to TheMC47/xmonad-contrib that referenced this pull request May 21, 2021
Current version of Steam sends _NET_ACTIVE_WINDOW ClientMessage for
every mouse click which results in a lot of border blinking.

Ignore requests that would result in no change.

Related: xmonad#371
liskin added a commit that referenced this pull request May 21, 2021
Current version of Steam sends _NET_ACTIVE_WINDOW ClientMessage for
every mouse click which results in a lot of border blinking.

Ignore requests that would result in no change to get rid of the
annoying border flicker that is inevitable with the current
implementation of XMonad.Operations.windows.

(Note that Steam also sends ConfigureRequest events, and these cause
an additional refresh due to the call to `float` when handling the event
in xmonad core. Not sure if worth fixing.)

Related: #371
Related: #399
TheMC47 pushed a commit to TheMC47/xmonad-contrib that referenced this pull request May 31, 2021
Current version of Steam sends _NET_ACTIVE_WINDOW ClientMessage for
every mouse click which results in a lot of border blinking.

Ignore requests that would result in no change to get rid of the
annoying border flicker that is inevitable with the current
implementation of XMonad.Operations.windows.

(Note that Steam also sends ConfigureRequest events, and these cause
an additional refresh due to the call to `float` when handling the event
in xmonad core. Not sure if worth fixing.)

Related: xmonad#371
Related: xmonad#399
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.

1 participant