Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Zero pad tokens when min spec starts with '0'#1341

Merged
patrick96 merged 1 commit into
polybar:masterfrom
memeplex:zero-pad
Jul 23, 2018
Merged

Zero pad tokens when min spec starts with '0'#1341
patrick96 merged 1 commit into
polybar:masterfrom
memeplex:zero-pad

Conversation

@memeplex

Copy link
Copy Markdown
Contributor

Fixes #1332

@codecov-io

codecov-io commented Jul 22, 2018

Copy link
Copy Markdown

Codecov Report

Merging #1341 into master will decrease coverage by <.01%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #1341      +/-   ##
=========================================
- Coverage    5.94%   5.94%   -0.01%     
=========================================
  Files         161     161              
  Lines        8927    8928       +1     
=========================================
  Hits          531     531              
- Misses       8396    8397       +1
Flag Coverage Δ
#unittests 5.94% <0%> (-0.01%) ⬇️
Impacted Files Coverage Δ
include/drawtypes/label.hpp 80% <ø> (ø) ⬆️
src/drawtypes/label.cpp 2.18% <0%> (-0.02%) ⬇️

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 4a494e6...1db192d. Read the comment docs.

@patrick96

Copy link
Copy Markdown
Member

Please add some documenting comments inside the code. Looks good though, will do some testing and merge this, after #1338 is merged

@memeplex

Copy link
Copy Markdown
Contributor Author

Ok, no problem:

  1. I'm not adding any comment to label.hpp since there is no other comment in that file, I'm just adding a descriptive comment in the site where zpad is parsed and initialized, is it ok?

  2. Do you prefer me to amend my previous commit or will you just squash everything at the end?

@NBonaparte

Copy link
Copy Markdown
Member

For this, amending would be fine.

@patrick96

Copy link
Copy Markdown
Member

@NBonaparte I would have squashed this anyway because of the format of the commit message

For the first part, do whatever you feel like is clearest

@patrick96 patrick96 merged commit 1d6d44f into polybar:master Jul 23, 2018
patrick96 pushed a commit to patrick96/polybar that referenced this pull request Jul 23, 2018
@patrick96 patrick96 mentioned this pull request Jul 23, 2018
patrick96 added a commit that referenced this pull request Dec 23, 2018
### Breaking Changes:
None

### Changelog

**New Config Options**
The following config options were added

In the `[settings]` section:
* `pseudo-transparency`

In `internal/cpu`:
* `ramp-coreload-spacing`, spacing between elements in `<ramp-coreload>` elements
* `%percentage-sum%` in `label`. Shows sum of all percentages

In `custom/script`:
* `double-click-left`
* `double-click-middle`
* `double-click-right`

In `internal/pulseaudio`:
* `interval`, volume increase and decrease steps in percentage points

In `internal/memory`:
* `bar-swap-used`
* `bar-swap-free`
* `ramp-swap-used`
* `ramp-swap-free`

**Deprecations**
* `tray-transparent` in the bar section is no longer used. The tray now always uses pseudo-transparency

**Features**
* volume: adjustable volume steps (#1364), see #1361
* memory: Add progress bars and ramps for swap (#1325)
* render: pseudo-transparency (always used in the tray)  (#595)
* cpu: 
    * Configurable spacing in between coreloads (#1472), see #1389
    * Non-normalized cpu load (#1517), see #1516
* script: Double click actions (#1442)
* net: Support for bridge interfaces (#1528), see #1522
* token: zero pad tokens when min-width value starts with a `0` (#1341), see #1332

**Fixes**
* config: Use font patterns detected on system in sample config (#1350)
* net: Disconnected module when SSID is only an emoji (#1371), see #1367
* tray: Tray overlapping with bar (#1392), see #591
* build: Several build errors (#1395, #1454, #1470, polybar/xpp#12, #1493), see #1447, #1469, #750, #1492
* mpd: format-online-prefix not working (#1402), see #1399
* font: Crash for some font-patterns (#1451), see #1435
* script: 
    * Only `%counter%` or `%pid%` token usable, never both (#1456)
    * click actions with colons in them (#1457), see #984
* renderer: Bar locks up when xorg composite extension is disabled (#1430), see #1425
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