-
-
Notifications
You must be signed in to change notification settings - Fork 419
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Icon request] battery-profile-performance & battery-profile-powersave #3666
Comments
@andrewgrasman can you please confirm that the icons are now correct? I don't have a laptop with Plasma. |
@morganist It seems to be pretty good, only difference is it's staying at a larger size for me than before using the default battery icon. |
@andrewgrasman is it the the baterry profile icons that are staying at a larger size? can you please attach a screenshot, if possible? |
oh, i see. can you please check if the power applet is inside the margins separator, like the system tray? if it's not, Plasma will stretch it to panel width, which seems like it does on your screenshot. doesn't seem like a scaling issue. |
It is the standalone aplet, I prefer it over the one in the tray. |
@andrewgrasman let me give you an example: in the edit mode, you can see the panel margins and how applets set their width based on which side of the separator they are on. what I'm saying is you should drag your standalone power applet inside the narrow side of the margin separator, like I should drag my media player applet to the right in my example. if you don't have a margins separator on your panel, add one - it's supposed to solve exactly this issue. |
@andrewgrasman look, the icon is now correctly sized at 22px. the only problem is that half of it is obscured by percentage, isn't it? but margin separators solve the sizing issue like they should, so... can you customise the percentage display? for example, for it to be next to the icon, not in it? i haven't used a laptop with plasma before. also, you colud put the system tray inside these two separators to avoid a large gap between it and the battery indicator. |
It doesn't bother me, I just think knowing that this icon change changed how sizing works for this theme is notable for development purposes. Sorry for coming off a little rude in my last reply. |
These seem to be mentioned in passing in this issue: 3595 but that seems to be a different root issue and not an icon request.
At the moment these Icons seem to be replaced with the default full battery icon (arch papirus and plasma 6 from arch repos), which dons't give the information it should and looks kinda bad. This is visible when you click the default battery applet with power-profiles-daemon installed and enabled.
What it currently looks like:
What it should look like (from breeze):
The text was updated successfully, but these errors were encountered: