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 754419 - "Connecting to '%s'" shown in collection views
"Connecting to '%s'" shown in collection views
Status: RESOLVED FIXED
Product: gnome-boxes
Classification: Applications
Component: general
3.17.x
Other Linux
: Normal normal
: --
Assigned To: GNOME Boxes maintainer(s)
GNOME Boxes maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2015-09-01 21:34 UTC by Zeeshan Ali
Modified: 2016-09-20 08:15 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Restore status label in topbar where it was original meant to be (17.99 KB, image/png)
2015-09-02 17:48 UTC, Zeeshan Ali
  Details
Restore status label under the thumbnail (here is where it looks odd) (111.68 KB, image/png)
2015-09-02 17:49 UTC, Zeeshan Ali
  Details
Same thing, just in list view (69.48 KB, image/png)
2015-09-02 17:49 UTC, Zeeshan Ali
  Details
machine: Don't show name in status (3.79 KB, patch)
2015-09-14 18:28 UTC, Zeeshan Ali
none Details | Review
machine: Set launch status directly to topbar (2.07 KB, patch)
2015-09-15 23:07 UTC, Zeeshan Ali
committed Details | Review
machine: Only bind name to Topbar.status if current (1.10 KB, patch)
2015-09-15 23:07 UTC, Zeeshan Ali
committed Details | Review
machine: Bind name directly to topbar status (1.57 KB, patch)
2015-09-15 23:07 UTC, Zeeshan Ali
committed Details | Review

Description Zeeshan Ali 2015-09-01 21:34:34 UTC
Unifying 'status' of machines caused some issues and the one that's not yet resolved is us showing "Connecting to '%s'" and "Restoring '%s'" in collection views, where it doesn't completely fit in the available space usually and also it looks odd given that name of the machine is already shown in the context and is clear to user.

We either need to modify these strings somehow or ensure that these status messages don't get shown in collection views.
Comment 1 Zeeshan Ali 2015-09-02 17:48:23 UTC
Created attachment 310535 [details]
Restore status label in topbar where it was original meant to be
Comment 2 Zeeshan Ali 2015-09-02 17:49:06 UTC
Created attachment 310536 [details]
Restore status label under the thumbnail (here is where it looks odd)
Comment 3 Zeeshan Ali 2015-09-02 17:49:34 UTC
Created attachment 310537 [details]
Same thing, just in list view
Comment 4 Zeeshan Ali 2015-09-14 18:28:12 UTC
Created attachment 311308 [details] [review]
machine: Don't show name in status

The status is also shown now in listview and showing the name of the
machine is not only redundant in that view but also looks ugly. Let's
add the name where and when it's needed.
Comment 5 Zeeshan Ali 2015-09-15 23:07:37 UTC
Created attachment 311421 [details] [review]
machine: Set launch status directly to topbar

The launch status strings are only useful for topbar. Moreover, the way
they currently are, they look ugly in collection views and don't even
fit in the space available. So let's just set these labels directly on
the topbar, instead of going through Machine.status property.
Comment 6 Zeeshan Ali 2015-09-15 23:07:43 UTC
Created attachment 311422 [details] [review]
machine: Only bind name to Topbar.status if current

Just cause the UI state is DISPLAY, doesn't mean every machine should
bind it's name to topbar status. Only current machine should do that.
Comment 7 Zeeshan Ali 2015-09-15 23:07:49 UTC
Created attachment 311423 [details] [review]
machine: Bind name directly to topbar status

The name has nothing to do with status anyway and going through
Machine.status property makes it appear in collection views as well and
that is just redundant and looks wrong and ugly.

So let's just bind the name directly to the topbar status.
Comment 8 Zeeshan Ali 2015-09-15 23:10:01 UTC
Attachment 311421 [details] pushed as 1cf5647 - machine: Set launch status directly to topbar
Attachment 311422 [details] pushed as 437f1fa - machine: Only bind name to Topbar.status if current
Attachment 311423 [details] pushed as b194ac3 - machine: Bind name directly to topbar status