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

[UI] Message Box icons don't have hover effect. #21068

Closed
apurv-1 opened this issue Mar 10, 2021 · 5 comments · Fixed by #27566
Closed

[UI] Message Box icons don't have hover effect. #21068

apurv-1 opened this issue Mar 10, 2021 · 5 comments · Fixed by #27566

Comments

@apurv-1
Copy link

apurv-1 commented Mar 10, 2021

Description:

If the user hovers over icons like Emoji picker or microphone or '+' icon, the cursor changes to pointer but the icon color remains the same.

Steps to reproduce:

  1. Open any room, and Try hovering over any of the mentioned icons.

Expected behavior:

We can add a hover effect for changing the colour of the icon.

Actual behavior:

Only cursor changes to pointer.

Server Setup Information:

  • Version of Rocket.Chat Server: 3.13.0-develop
  • Operating System: linux
  • Deployment Method:
  • Number of Running Instances:
  • DB Replicaset Oplog:
  • NodeJS Version: 12.18.4 - x64
  • MongoDB Version: 4.2.8

Client Setup Information

  • Desktop App or Browser Version: Chrome 88.0
  • Operating System: Windows 10

Additional context

bandicam.2021-03-11.03-39-32-418.mp4
@maansiii
Copy link

Use Font Awesome and add this to your html element.

@maansiii
Copy link

<i class="fas fa-plus"></i>

@apurv-1
Copy link
Author

apurv-1 commented Mar 13, 2021

@maansiii You can check the PR I've mentioned in this issue.
And if you think you have a better solution. Please go ahead and open a new PR for this. 😃

@Vishv07
Copy link

Vishv07 commented Mar 13, 2021

@apurv-1 can I work upon this?

@apurv-1
Copy link
Author

apurv-1 commented Mar 14, 2021

I've already opened a PR and mentioned it, But if you have a better solution please go ahead! @Vishv07 .

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 a pull request may close this issue.

4 participants