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

Skip to content

fix: Move return value of parse_layout_symbol#1470

Merged
NBonaparte merged 1 commit into
polybar:masterfrom
patrick96:issue/1469
Oct 16, 2018
Merged

fix: Move return value of parse_layout_symbol#1470
NBonaparte merged 1 commit into
polybar:masterfrom
patrick96:issue/1469

Conversation

@patrick96

@patrick96 patrick96 commented Oct 15, 2018

Copy link
Copy Markdown
Member

Fixes #1469

I wasn't sure if I should use std::forward here but decided against it since the function is not templated. rvalue references still give me headaches ^^

@codecov-io

Copy link
Copy Markdown

Codecov Report

Merging #1470 into master will not change coverage.
The diff coverage is 0%.

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #1470   +/-   ##
======================================
  Coverage    5.78%   5.78%           
======================================
  Files         162     162           
  Lines        9030    9030           
======================================
  Hits          522     522           
  Misses       8508    8508
Flag Coverage Δ
#unittests 5.78% <0%> (ø) ⬆️
Impacted Files Coverage Δ
src/x11/extensions/xkb.cpp 1.08% <0%> (ø) ⬆️

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 abdd5f9...3fbb2a7. Read the comment docs.

@NBonaparte NBonaparte merged commit 3b32f58 into polybar:master Oct 16, 2018
@patrick96 patrick96 deleted the issue/1469 branch October 16, 2018 18:56
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.

3 participants