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 684819 - "Connect to a Hidden Network" doesn't look actionable
"Connect to a Hidden Network" doesn't look actionable
Status: RESOLVED FIXED
Product: gnome-control-center
Classification: Core
Component: Network
3.6.x
Other Linux
: Normal normal
: ---
Assigned To: Control-Center Maintainers
Control-Center Maintainers
Depends on:
Blocks:
 
 
Reported: 2012-09-25 21:08 UTC by Cosimo Cecchi
Modified: 2012-10-09 08:23 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch (7.16 KB, patch)
2012-10-01 01:16 UTC, Matthias Clasen
none Details | Review
screenshot (56.01 KB, image/png)
2012-10-01 01:17 UTC, Matthias Clasen
  Details
Use a regular button for 'Connect to Hidden' (3.94 KB, patch)
2012-10-03 09:40 UTC, Bastien Nocera
accepted-commit_after_freeze Details | Review
network: Use a regular button for 'Connect to Hidden' (7.17 KB, patch)
2012-10-03 09:43 UTC, Bastien Nocera
committed Details | Review

Description Cosimo Cecchi 2012-09-25 21:08:52 UTC
The "Connect to a hidden Network" row doesn't look actionable, as it doesn't have a right arrow at the end. Also, the fact that it's displayed in bold before the out of range networks makes it look more like a heading for that section than a clickable row.

I think it would be nice to add some vertical padding around it, and also an arrow button.
Comment 1 Matthias Clasen 2012-09-27 02:39:18 UTC
Arrow button doesn't sound right to me. This doesn't go to a details page directly, but pops up a dialog. I think having noticable hover for the left sides of the row would address this. Maybe make it bold only on hover ? And then, do the same for APs that can be activated by clicking on the name ?
Comment 2 Allan Day 2012-09-28 10:47:22 UTC
This bug is largely a consequence of not having the correct widgetry to do this kind of UI. Ideally, list rows would have separators and styling that would invite interaction.

For now, I wonder if it would be better to simply make Connect to a Hidden Network a button outside the list (horizontally aligned with Use as Hotspot...).
Comment 3 Matthias Clasen 2012-10-01 01:16:33 UTC
Created attachment 225452 [details] [review]
patch
Comment 4 Matthias Clasen 2012-10-01 01:17:20 UTC
Created attachment 225453 [details]
screenshot

The button text is a bit long, otherwise looks ok to me
Comment 5 Allan Day 2012-10-01 07:28:15 UTC
(In reply to comment #4)
> Created an attachment (id=225453) [details]
> screenshot
> 
> The button text is a bit long, otherwise looks ok to me

The text seems OK to me. I'd maybe right align the button to the other edge of the list though.
Comment 6 Bastien Nocera 2012-10-01 10:14:18 UTC
With better arrows, we could avoid having to use a button for this functionality, which I would prefer to this current patch.
Comment 7 Matthias Clasen 2012-10-01 23:00:12 UTC
I still don't think an arrow is right for this at all. It opens a dialog, and doesn't switch to a different page, like all the other arrows do.
Comment 8 Bastien Nocera 2012-10-03 09:40:04 UTC
Created attachment 225670 [details] [review]
Use a regular button for 'Connect to Hidden'

The treeview is not really up to this level of custom UI.
Comment 9 Bastien Nocera 2012-10-03 09:43:20 UTC
Created attachment 225671 [details] [review]
network: Use a regular button for 'Connect to Hidden'

The treeview is not really up to this level of custom UI.
Comment 10 Bastien Nocera 2012-10-03 09:44:35 UTC
Comment on attachment 225671 [details] [review]
network: Use a regular button for 'Connect to Hidden'

Attachment 225671 [details] pushed as ea0f359 - network: Use a regular button for 'Connect to Hidden'
Comment 11 Bastien Nocera 2012-10-03 09:46:12 UTC
Comment on attachment 225670 [details] [review]
Use a regular button for 'Connect to Hidden'

This is for 3.8, because of the new string.
Comment 12 Bastien Nocera 2012-10-03 09:46:40 UTC
Removing from the whiteboard as it's fixed in master for GNOME 3.6.
Comment 13 Bastien Nocera 2012-10-09 08:23:06 UTC
Comment on attachment 225670 [details] [review]
Use a regular button for 'Connect to Hidden'

Would have been better if I didn't forget the UI file in the patch.
Comment 14 Bastien Nocera 2012-10-09 08:23:45 UTC
Fixed the string in master.