sakithb / media-controls

A media indicator for the Gnome shell.
MIT License
228 stars 33 forks source link

V17 | Padding Issue #16

Closed mBilG closed 2 years ago

mBilG commented 2 years ago

Thanks for the amazing extension.

Just updated to the latest version (17). Noticed 2 issues with padding (issue not present in previous version):

  1. Player icons are too small (probably due to larger padding).
  2. Title appears to be bottom-aligned: text gets cut-off when letters go below the line (g/y/q/p etc).

I'm using a different shell theme. However, even if i change to adwaita, the buttons still appear smaller than the rest of the icons.

sakithb commented 2 years ago

Thanks for reporting the issue. Can you send me a screenshot and the name of your theme? I tested it on Pop theme, Materia theme, Orchis theme and the White Sur theme but the issue didn't exist.

sakithb commented 2 years ago

Hi, I figured out the issue and commited the fix. To apply the fix you can use this command, cd ~/.local/share/gnome-shell/extensions/mediacontrols@cliffniff.github.com/ && curl -OL https://raw.githubusercontent.com/cliffniff/media-controls/main/player.js

mBilG commented 2 years ago

I'm currently using Fluent_compact theme by Vinceliuice. I set my panel size to 26 px. The icons appear bigger now, but still tiny compared to other icons (and previous version)

This is before applying fix: image

This is after fix mentioned above: image

This is from v14: image

Maybe you need to remove top and bottom paddings/margins and let the theme control that?

sakithb commented 2 years ago

Maybe you need to remove top and bottom paddings/margins and let the theme control that?

Yes, that's what I did. In v14, all the elements were on the panel on their own. Now they are in a separate container. I'll find a workaround for this.

sakithb commented 2 years ago

image image I installed the theme and tested it on my machine and it shows the icons fine.

mBilG commented 2 years ago

I found a fix:

I use this for the buttons' style:

style: "padding: 0px 2px; margin: 0px 2px;"

Here's the result: image

sakithb commented 2 years ago

I think it is a issue specific to the theme. The weird thing is it looks fine in the same theme when I installed it.

mBilG commented 2 years ago

Yes, that's what I did. In v14, all the elements were on the panel on their own. Now they are in a separate container. I'll find a workaround for this.

Maybe the padding/margin of the container also needs to be adjusted?

I think it is a issue specific to the theme. The weird thing is it looks fine in the same theme when I installed it.

I also noticed that the player icon has the same bug. I don't usually use the icon though, so i disabled it.

Just tested, I think the issue is caused when using "Just perfection" extension and setting my panel size to lower than 30px.

Panel 26px: image

Panel 32px: image

sakithb commented 2 years ago

Fixed the bug properly. Rerun the command cd ~/.local/share/gnome-shell/extensions/mediacontrols@cliffniff.github.com/ && curl -OL https://raw.githubusercontent.com/cliffniff/media-controls/main/player.js && curl -OL https://raw.githubusercontent.com/cliffniff/media-controls/main/stylesheet.css to apply it. Thanks for reporting!

Edit: Updated the command.

mBilG commented 2 years ago

Excellent.. Works like a charm.. Thanks for your help and for the great extension.. keep it up..