Skip to content

Conversation

@patrick96
Copy link
Member

@patrick96 patrick96 commented Feb 18, 2024

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

If the root pixmap does not fully cover the bar window, some pseudo-transparent areas were filled with unitialized data, causing pixelation or other rendering artifacts.

Now, observed background slices are first filled with black to make sure this does not happen and they print an error if not the full pixmap can be filled.

Related Issues & Documents

Fixes #3041

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

@patrick96 patrick96 added this to the 3.7.2 milestone Feb 18, 2024
@patrick96 patrick96 force-pushed the fix/partial-wallpaper branch from 99b9723 to 1cbc6cc Compare February 18, 2024 18:10
@codecov
Copy link

codecov bot commented Feb 18, 2024

Codecov Report

Attention: Patch coverage is 0% with 9 lines in your changes missing coverage. Please review.

Project coverage is 12.74%. Comparing base (3563802) to head (309ad80).

Files with missing lines Patch % Lines
src/x11/background_manager.cpp 0.00% 9 Missing ⚠️
Additional details and impacted files
@@               Coverage Diff                @@
##           hotfix/3.7.2    #3096      +/-   ##
================================================
- Coverage         12.75%   12.74%   -0.01%     
================================================
  Files               160      160              
  Lines             11598    11604       +6     
================================================
  Hits               1479     1479              
- Misses            10119    10125       +6     
Flag Coverage Δ
unittests 12.74% <0.00%> (-0.01%) ⬇️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

If the root pixmap does not fully cover the bar window, some
pseudo-transparent areas were filled with unitialized data, causing
pixelation or other rendering artifacts.

Now, ovserved background slices are first filled with black to make sure
this does not happen and they print an error if not the full pixmap can
be filled.

Fixes polybar#3041
@patrick96 patrick96 force-pushed the fix/partial-wallpaper branch from 1cbc6cc to 309ad80 Compare August 17, 2024 11:08
@patrick96 patrick96 merged commit c0d3d7a into polybar:hotfix/3.7.2 Aug 17, 2024
@patrick96 patrick96 deleted the fix/partial-wallpaper branch August 17, 2024 11:13
@patrick96 patrick96 mentioned this pull request Aug 17, 2024
1 task
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