Skip to content

Conversation

patrick96
Copy link
Member

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

Related Issues & Documents

Part of #2689

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

All the tray settings in the wiki need to be marked as deprecated and mention the new tray module.

Replaces the commented out `tray-position`
@patrick96 patrick96 mentioned this pull request Aug 6, 2023
11 tasks
@patrick96 patrick96 marked this pull request as ready for review August 6, 2023 13:18
@patrick96 patrick96 added this to the 3.7.0 milestone Aug 6, 2023
@codecov
Copy link

codecov bot commented Aug 6, 2023

Codecov Report

Merging #3002 (b75ab8a) into master (b754790) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #3002      +/-   ##
==========================================
- Coverage   12.69%   12.69%   -0.01%     
==========================================
  Files         160      160              
  Lines       12530    12534       +4     
==========================================
  Hits         1591     1591              
- Misses      10939    10943       +4     
Flag Coverage Δ
unittests 12.69% <0.00%> (-0.01%) ⬇️

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

Files Changed Coverage Δ
src/x11/legacy_tray_manager.cpp 0.00% <0.00%> (ø)

@patrick96 patrick96 merged commit 1043354 into polybar:master Aug 6, 2023
@patrick96 patrick96 deleted the tray-deprecation branch August 6, 2023 13:26
@patrick96 patrick96 mentioned this pull request Nov 5, 2023
24 tasks
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