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 751542 - New messages line is ambiguous
New messages line is ambiguous
Status: RESOLVED OBSOLETE
Product: polari
Classification: Applications
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Polari maintainers
Polari maintainers
Depends on:
Blocks:
 
 
Reported: 2015-06-26 13:11 UTC by Allan Day
Modified: 2021-06-10 21:41 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
mockup (125.66 KB, image/png)
2015-06-26 13:11 UTC, Allan Day
  Details
chatView: Update new-message indicator (5.26 KB, patch)
2015-07-22 10:14 UTC, Florian Müllner
none Details | Review
chatView: Update new-message indicator (5.39 KB, patch)
2015-07-23 13:50 UTC, Florian Müllner
none Details | Review
chatView: Update new-message indicator (5.46 KB, patch)
2015-07-30 10:16 UTC, Florian Müllner
none Details | Review
chatView: Update new-message indicator (5.54 KB, patch)
2015-07-31 15:28 UTC, Florian Müllner
none Details | Review
chatView: Update new-message indicator (5.54 KB, patch)
2015-07-31 22:01 UTC, Florian Müllner
committed Details | Review

Description Allan Day 2015-06-26 13:11:58 UTC
Created attachment 306172 [details]
mockup

The conversation history shows a thin black line to indicate where the unread part of a conversation begins.

The meaning of the line isn't clear - I didn't find it clear, and a discussion on IRC today revealed that others didn't understand it either. Some people had assumed that the line was a bug.

I've attached a mockup which suggests a different visual treatment for the line.

We might also want to look at the logic behind the new messages indicator. For me it is displayed even if I have only switched away from Polari for a few seconds. A longer timeout, and pairing the indicator with a timestamp might help.
Comment 1 Jakub Steiner 2015-07-13 22:07:20 UTC
The explicit new message separator is still subtle. I like it.
Comment 2 Florian Müllner 2015-07-22 10:14:31 UTC
Created attachment 307898 [details] [review]
chatView: Update new-message indicator

The plain line currently used to indicate the start of unread messages
in the chat log is not immediately obvious. Update its visuals to clarify
the purpose by including a "New Messages" label.

I did play around with including the timestamp as well, but left it out for now as it requires more work - it's really odd if there's already a timestamp on the preceding line or nearby ...
Comment 3 Florian Müllner 2015-07-23 13:50:44 UTC
Created attachment 307991 [details] [review]
chatView: Update new-message indicator

Pixel-align the indicator properly
Comment 4 Florian Müllner 2015-07-30 10:16:50 UTC
Created attachment 308447 [details] [review]
chatView: Update new-message indicator

The plain line currently used to indicate the start of unread messages
in the chat log is not immediately obvious. Update its visuals to clarify
the purpose by including a "New Messages" label.
Comment 5 Florian Müllner 2015-07-31 15:28:00 UTC
Created attachment 308560 [details] [review]
chatView: Update new-message indicator

Some more updates after comments on IRC:
 - fix vertical alignment
 - fix right margin
 - use solid line instead of dashed one
 - use 'borders' color for line (which is less heavy than 'dim-label')
Comment 6 Florian Müllner 2015-07-31 22:01:33 UTC
Created attachment 308584 [details] [review]
chatView: Update new-message indicator

Fix clip check
Comment 7 Florian Müllner 2015-08-05 20:28:32 UTC
Comment on attachment 308584 [details] [review]
chatView: Update new-message indicator

Attachment 308584 [details] pushed as ff0229e - chatView: Update new-message indicator

Pushing this now, but leaving the bug open in case we want to also tweak when to show the indicator as suggested in comment #0.
Comment 8 GNOME Infrastructure Team 2021-06-10 21:41:17 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to GNOME's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.gnome.org/GNOME/polari/-/issues/173.