Bug 383213 - Menu from battery icon in tray shows huge icon
Summary: Menu from battery icon in tray shows huge icon
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Battery Monitor (show other bugs)
Version: 5.8.0
Platform: Other Linux
: NOR normal
Target Milestone: 1.0
Assignee: Kai Uwe Broulik
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-08-06 22:32 UTC by Patrick Silva
Modified: 2017-08-07 14:11 UTC (History)
1 user (show)

See Also:
Latest Commit: https://commits.kde.org/plasma-workspace/03f9ab0f9149050541d2852f94b508f746aac955
Version Fixed In: 5.8.8

Attachments
screenshot (41.87 KB, image/png)
2017-08-06 22:32 UTC, Patrick Silva 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Patrick Silva 2017-08-06 22:32:01 UTC 
Created attachment 107118 [details]
screenshot

See the screenhot. I use oranchelo icon theme on neon dev unstable.
Comment 1 Kai Uwe Broulik 2017-08-07 08:13:23 UTC 
Can you please provide the output of upower -d so I can replicate the battery configuration you have that triggers it. I tried to simulate a battery but here I just get a Plasma battery icon. I downloaded "Oranchelo" from KDE store.
Comment 2 Patrick Silva 2017-08-07 13:53:37 UTC 
Device: /org/freedesktop/UPower/devices/line_power_ACAD
  native-path:          ACAD
  power supply:         yes
  updated:              Seg 07 Ago 2017 10:49:37 BRT (174 seconds ago)
  has history:          no
  has statistics:       no
  line-power
    warning-level:       none
    online:              yes
    icon-name:          'ac-adapter-symbolic'

Device: /org/freedesktop/UPower/devices/battery_BAT1
  native-path:          BAT1
  vendor:               LENOVO
  model:                PABAS0241231
  serial:               41167
  power supply:         yes
  updated:              Seg 07 Ago 2017 10:51:41 BRT (50 seconds ago)
  has history:          yes
  has statistics:       yes
  battery
    present:             yes
    rechargeable:        yes
    state:               charging
    warning-level:       none
    energy:              19,79 Wh
    energy-empty:        0 Wh
    energy-full:         34,78 Wh
    energy-full-design:  37,58 Wh
    energy-rate:         0 W
    voltage:             15,373 V
    percentage:          56%
    capacity:            92,5492%
    technology:          lithium-ion
    icon-name:          'battery-good-charging-symbolic'

Device: /org/freedesktop/UPower/devices/DisplayDevice
  power supply:         yes
  updated:              Seg 07 Ago 2017 10:49:37 BRT (174 seconds ago)
  has history:          no
  has statistics:       no
  battery
    present:             yes
    state:               charging
    warning-level:       none
    energy:              19,79 Wh
    energy-full:         34,78 Wh
    energy-rate:         0 W
    percentage:          56%
    icon-name:          'battery-good-charging-symbolic'

Daemon:
  daemon-version:  0.99.4
  on-battery:      no
  lid-is-closed:   no
  lid-is-present:  yes
  critical-action: PowerOff
Comment 3 Kai Uwe Broulik 2017-08-07 14:11:41 UTC 
Git commit 03f9ab0f9149050541d2852f94b508f746aac955 by Kai Uwe Broulik.
Committed on 07/08/2017 at 14:11.
Pushed by broulik into branch 'Plasma/5.8'.

[Battery Monitor] Fix icon size for InhibitionHint

In QtQuick Layouts implicit size, which is taken from the SVG's preferred size, takes
precedence potentially destroying the layout.

Also, it should have been medium to be consistent with the rest.
FIXED-IN: 5.8.8

M  +2    -2    applets/batterymonitor/package/contents/ui/InhibitionHint.qml

https://commits.kde.org/plasma-workspace/03f9ab0f9149050541d2852f94b508f746aac955