After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 646121 - network: signal strength icons in the menu are not aligned
network: signal strength icons in the menu are not aligned
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: network-indicator
2.91.x
Other Linux
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
[gnome3-important]
Depends on:
Blocks:
 
 
Reported: 2011-03-29 15:28 UTC by Cosimo Cecchi
Modified: 2011-04-04 13:31 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
screenshot (41.41 KB, image/png)
2011-03-29 15:28 UTC, Cosimo Cecchi
  Details
network: fix alignment of secure vs insecure wireless icons (1.64 KB, patch)
2011-03-29 15:48 UTC, Dan Winship
committed Details | Review
screenshot with icons (54.38 KB, image/png)
2011-03-29 20:11 UTC, William Jon McCann
  Details

Description Cosimo Cecchi 2011-03-29 15:28:04 UTC
It seems the signal strength icons are not properly aligned in the NM menu. See attached screenshot.
Comment 1 Cosimo Cecchi 2011-03-29 15:28:20 UTC
Created attachment 184601 [details]
screenshot
Comment 2 Dan Winship 2011-03-29 15:39:10 UTC
part of that is because you don't have the latest gnome-icon-theme-symbolic, and so you're not seeing the lock icons on the secure networks. But yeah, even with that, it's wrong, which is weird because I totally remember changing things to make things line up right...
Comment 3 Dan Winship 2011-03-29 15:48:46 UTC
Created attachment 184603 [details] [review]
network: fix alignment of secure vs insecure wireless icons

Since the icon area is end-aligned, the signal strength icon for
insecure networks was ending up aligned with the lock icon for secure
networks. Fix that by always including a _secureIcon, but having it be
blank for the insecure networks.
Comment 4 Florian Müllner 2011-03-29 17:10:21 UTC
Review of attachment 184603 [details] [review]:

Looks right and works as advertised.
Comment 5 William Jon McCann 2011-03-29 20:11:01 UTC
Created attachment 184630 [details]
screenshot with icons
Comment 6 Florian Müllner 2011-03-29 21:12:07 UTC
(In reply to comment #5)
> Created an attachment (id=184630) [details]
> screenshot with icons

That's without Dan's patch, right?
Comment 7 William Jon McCann 2011-03-31 00:39:29 UTC
Florian, correct.  Would be really nice to fix this.
Comment 8 Giovanni Campagna 2011-04-02 17:37:45 UTC
I'm a bit late, but I think that the right fix is to place secure icons on the left, so wireless icons are always right aligned at the end.
Otherwise this bug comes back with private connections and adhocs, that use plain PopupImageMenuItem.
Comment 9 Dan Winship 2011-04-04 00:48:14 UTC
(In reply to comment #8)
> I'm a bit late, but I think that the right fix is to place secure icons on the
> left, so wireless icons are always right aligned at the end.

I did it according to https://live.gnome.org/GnomeShell/Design/Guidelines/SystemStatus/Network.

> Otherwise this bug comes back with private connections and adhocs, that use
> plain PopupImageMenuItem.

Can't ad-hoc connections be either secure or insecure too?

I guess in the private case we can't tell whether the connection is secure or not? But we can tell its signal strength? That seems weird...
Comment 10 Owen Taylor 2011-04-04 13:19:06 UTC
(In reply to comment #9)
> (In reply to comment #8)
> > I'm a bit late, but I think that the right fix is to place secure icons on the
> > left, so wireless icons are always right aligned at the end.
> 
> I did it according to
> https://live.gnome.org/GnomeShell/Design/Guidelines/SystemStatus/Network.

The high and low fidelity mockups actually differ there... there may not be a lot of intent. I think for 3.0.0 we should just go as we have a patch here then ask the designers - I'm not worried if the ordering changes for 3.0.1.
Comment 12 Dan Winship 2011-04-04 13:31:17 UTC
Attachment 184603 [details] pushed as 7932585 - network: fix alignment of secure vs insecure wireless icons