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

fix: jump to message highlights the element #748

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

thepiyush-303
Copy link

Brief Title

Acceptance Criteria fulfillment

  • Task 1
  • Task 2
  • Task 3

Fixes # (issue)
closes #747

Video/Screenshots

Screencast.from.2024-12-28.23-54-40.webm

PR Test Details

Note: The PR will be ready for live testing at https://rocketchat.github.io/EmbeddedChat/pulls/pr-<pr_number> after approval. Contributors are requested to replace <pr_number> with the actual PR number.

@dhairyashiil
Copy link

Hello @thepiyush-303, instead of hardcoding the color to yellow, could you set it using the colors from the 'theme'?

In the same file, MessageAggregator.js, you will find the theme - try searching for const { theme } = useTheme();

So that this highlighting feature looks consistent across all the design variants of Embedded Chat.

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.

Jump to message doesn't highlight the message
2 participants