Skip to content

Conversation

ghost
Copy link

@ghost ghost commented Jan 27, 2021

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update
  • Other: Replace this with a description of the type of this PR

Description

Added reverse-scroll option for xworkspaces module. The default is to set it to false, that way users do not have to change their configuration files if they want old behavior.

Related Issues & Documents

None

Documentation (check all applicable)

  • This PR requires changes to the Wiki documentation (describe the changes)
  • This PR requires changes to the documentation inside the git repo (please add them to the PR).
  • Does not require documentation changes

The new setting should be documented in the xworkspaces section, probably under basic settings.

@codecov
Copy link

codecov bot commented Jan 27, 2021

Codecov Report

Merging #2365 (2b05d3d) into master (4ded401) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #2365      +/-   ##
=========================================
- Coverage    8.10%   8.09%   -0.01%     
=========================================
  Files         143     143              
  Lines       10382   10383       +1     
=========================================
  Hits          841     841              
- Misses       9541    9542       +1     
Flag Coverage Δ
unittests 8.09% <0.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
include/modules/xworkspaces.hpp 0.00% <ø> (ø)
src/modules/xworkspaces.cpp 0.00% <0.00%> (ø)

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 4ded401...2b05d3d. Read the comment docs.

Copy link
Member

@patrick96 patrick96 left a comment

Choose a reason for hiding this comment

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

Thanks 😃

We recently decided that the next action should always go to the right and prev always to the left. Because of that the reverse-scroll option should be applied when producing the scroll actions in get_output (the same way it is done in the i3 and bspwm modules)

@ghost ghost requested a review from patrick96 January 27, 2021 16:19
Copy link
Member

@patrick96 patrick96 left a comment

Choose a reason for hiding this comment

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

Looks good now :)

I forgot to mention this before, please add your changes to the changelog: https://github.com/polybar/polybar/blob/master/CONTRIBUTING.md#changelog

@ghost ghost requested a review from patrick96 January 28, 2021 02:10
Copy link
Member

@patrick96 patrick96 left a comment

Choose a reason for hiding this comment

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

Perfect!

@patrick96 patrick96 merged commit c676d7e into polybar:master Jan 28, 2021
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