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

feat: Display total number and room unread messages in room title #32304

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

MichalFoksa
Copy link

@MichalFoksa MichalFoksa commented Apr 24, 2024

TLTR
PR replaces room title from:
20240424-develop-branch-unread_messages_obscures_rooms_name
to:
324708497-2c42eddc-9620-4d17-bb1d-d6f21a1a672b


PR changes room title to:

  • total amount of unread messages in less intrusive way
  • number of unread room messages
  • room name - of course

Format:

  • number of total unread messages in square brackets - if the number is zero then it is not displayed
  • number of room unread messages in brackets - if the number is zero, it is not displayed.

Examples:

  • [10] (5) Mark Watney - workspace name
  • [10] general - workspace name - AFAIK channels do not have number of unread messages, so room unread not show
  • general - workspace name - no unread messages

image
Note: 32 is total number of unread, 17 "Johny Smith" room unread, 15 "Mark Watney" room unread

Issue(s)

RocketChat/feature-requests#885

Steps to test or reproduce

Further comments

Copy link

changeset-bot bot commented Apr 24, 2024

🦋 Changeset detected

Latest commit: 8594c56

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 32 packages
Name Type
@rocket.chat/meteor Minor
@rocket.chat/core-typings Minor
@rocket.chat/rest-typings Minor
@rocket.chat/apps Patch
@rocket.chat/core-services Patch
@rocket.chat/cron Patch
@rocket.chat/gazzodown Major
@rocket.chat/livechat Patch
@rocket.chat/model-typings Patch
@rocket.chat/ui-contexts Major
@rocket.chat/account-service Patch
@rocket.chat/authorization-service Patch
@rocket.chat/ddp-streamer Patch
@rocket.chat/omnichannel-transcript Patch
@rocket.chat/presence-service Patch
@rocket.chat/queue-worker Patch
@rocket.chat/stream-hub-service Patch
@rocket.chat/api-client Patch
@rocket.chat/license Patch
@rocket.chat/omnichannel-services Patch
@rocket.chat/pdf-worker Patch
@rocket.chat/presence Patch
rocketchat-services Patch
@rocket.chat/ddp-client Patch
@rocket.chat/fuselage-ui-kit Major
@rocket.chat/models Patch
@rocket.chat/ui-avatar Major
@rocket.chat/ui-client Major
@rocket.chat/ui-video-conf Major
@rocket.chat/uikit-playground Patch
@rocket.chat/web-ui-registration Major
@rocket.chat/instance-status Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@CLAassistant
Copy link

CLAassistant commented Apr 24, 2024

CLA assistant check
All committers have signed the CLA.

@MichalFoksa MichalFoksa force-pushed the feature/develop/display-total-unread-and-room-unread-in-room-title branch from 57a049d to 3431f94 Compare April 24, 2024 08:42
Text "xyz unread messages" in document title replaced by room title format [total unread] (room unread) Room Name Where:

    [total unread] - is total number of unread messages - hidden when equal to zero
    (room unread) - is number of room unread messages - hidden when equal to zero
@MichalFoksa MichalFoksa force-pushed the feature/develop/display-total-unread-and-room-unread-in-room-title branch from 1a287ba to 8594c56 Compare April 24, 2024 20:18
@MichalFoksa MichalFoksa marked this pull request as ready for review April 26, 2024 19:18
@MichalFoksa MichalFoksa requested a review from a team as a code owner April 26, 2024 19:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants