Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

message view: Use inline date separators. #10820

Merged
merged 1 commit into from Feb 11, 2019

Commits on Feb 11, 2019

  1. message_list: Don't split message groups for a date divider.

    This adds date dividers within a single message group when the only
    reason we had previously been splitting apart two message groups is a
    change of date.  The overall effect is a cleaner message list user
    experience.
    
    The downside of this change would be that the recipient bars no longer
    will always show a new date for date changes; to fix that, we rewrite
    how the floating recipient bars both set the date field on the
    floating recipient bar itself, as well as ensure that non-floating
    recipient bars don't show duplicate dates.
    
    In a future design update where we modify how message recipient bars
    look, we may very well be able to simplify this logic by removing some
    of the dynamic nature of the recipient bar calculations.  But this is
    a good implementation of what remains.
    
    Tweaked significantly by tabbott from Steve Howell's original, both to
    extract these changes from a larger PR as well as to modify the
    first_visible_message logic to handle some tricky corner cases.
    
    Fixes zulip#10171.
    timabbott committed Feb 11, 2019
    Configuration menu
    Copy the full SHA
    f63e579 View commit details
    Browse the repository at this point in the history