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 746125 - Redundant "No conversations selected" in right headerbar
Redundant "No conversations selected" in right headerbar
Status: RESOLVED FIXED
Product: geary
Classification: Other
Component: ux
master
Other Linux
: Normal normal
: 0.10.0
Assigned To: Geary Maintainers
Geary Maintainers
Depends on:
Blocks:
 
 
Reported: 2015-03-13 01:25 UTC by Jim Nelson
Modified: 2015-03-13 19:39 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Don't set conversation title for zero or multiple conversations (1.74 KB, patch)
2015-03-13 19:32 UTC, Robert Schroll
committed Details | Review

Description Jim Nelson 2015-03-13 01:25:53 UTC
I'm seeing two issues with Geary in current master:

* Sometimes I'm not seeing "No conversations selected" in the conversation viewer when no conversation is selected.  The label on the top headerbar shows the message.  I don't know how to reproduce this state.  It seems like they're both shown when the app is first executed but some time after I don't see the message in the conversation viewer.

* I'm now unsure if we want "No conversations selected" in both locations anyway -- it feels redundant.  I think we should pick one and stick to it.  I lean toward the conversation viewer and leaving the top headerbar blank in that situation, as it helps keep the conversation viewer from looking like a big blank field.
Comment 1 Robert Schroll 2015-03-13 04:15:31 UTC
I think point one is bug 744066.  As I note there, we may be able to solve it with some CSS trickery.

Point two is easy.  The header bar title code is actually completely separate.
Comment 2 Jim Nelson 2015-03-13 19:05:09 UTC
> I think point one is bug 744066.

Ah, ok.  I've changed the title of this ticket to reflect the second point only.

I'm targeting this for 0.10, since we think it'll be straightforward.
Comment 3 Robert Schroll 2015-03-13 19:32:39 UTC
Created attachment 299355 [details] [review]
Don't set conversation title for zero or multiple conversations

I've also removed the title when there are multiple conversations.  But 
we can easily keep one and not the other if you prefer.
Comment 4 Jim Nelson 2015-03-13 19:37:40 UTC
Review of attachment 299355 [details] [review]:

No, that's perfect.  Commit!
Comment 5 Robert Schroll 2015-03-13 19:38:58 UTC
Attachment 299355 [details] pushed as 1d0e58b - Don't set conversation title for zero or multiple conversations