penguin-teal / gnome-openweather

A GNOME Shell extension to show the weather of any location on Earth.
GNU General Public License v3.0
58 stars 18 forks source link

bug: Right "Position in panel" looks off #17

Closed fiftydinar closed 8 months ago

fiftydinar commented 8 months ago

How it looks right now: Screenshot from 2024-03-30 16-10-12

How it should look (took from my other repo): output

This behavior happens since v132 version.

Gnome 45 Fedora Silverblue 39 RPM package

penguin-teal commented 8 months ago

Thanks for the bug report. Are you certain it is not another extension doing that? I also notice that your second screenshot has some text that says "sr" but that text appears to be missing in the first one; if you hover over it does it highlight like there is an additional missing button between the weather button and the network/sound/power buttons?

TynInks commented 8 months ago

I am also experiencing this behavior after updating to version 132. The temperature is significantly further away from the other icons than in previous versions. Only seems to affect Right-Aligned position.

Default Behavior, no other extensions:
Weather Text Location - Default

Additionally, sometimes after it refreshes the data, the temperature cuts off:
Weather Text Location - Number Cut Off

I've included my settings, but they're pretty close to the defaults:
app version 132 git version.txt

fiftydinar commented 8 months ago

Thanks for the bug report. Are you certain it is not another extension doing that? I also notice that your second screenshot has some text that says "sr" but that text appears to be missing in the first one; if you hover over it does it highlight like there is an additional missing button between the weather button and the network/sound/power buttons?

"sr" is the language switcher. I disabled 2nd language so that's why I don't have that. No additional button when switching between that empty space.

Tested with no other extensions & factory-reseted the extension to defaults, no change.

penguin-teal commented 8 months ago

In trying to fix it for GNOME 46 it broke GNOME 45! Now both versions work with the new commit.