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

Font Size preview doesn't change on changing the message layout #21782

Closed
goelesha opened this issue Apr 13, 2022 · 3 comments · Fixed by matrix-org/matrix-react-sdk#10170
Closed
Labels
A-Appearance A-User-Settings good first issue Good for newcomers Help Wanted Extra attention is needed O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect

Comments

@goelesha
Copy link

Steps to reproduce

  1. Where are you starting? What can you see?
    Open appearance in the setting on the element web in the browser. When I change the message layout the font size preview doesn't get changed. Instead, it changes on refreshing the page.

Outcome

What did you expect?

If initially I have IRC message layout set in the appearance settings and now I change it to a modern message layout, then on changing the message layout the font size preview message doesn't get changed on the click instead it gets changed after refreshing.
Initially when layout is set to IRC it looks like this-
Screenshot 2022-04-13 at 11 25 58 PM
Now as soon as I change the message layout by clicking on modern, I expect the font size preview message to get changed on the click. So I expect this after clicking on Modern now.
Screenshot 2022-04-13 at 11 26 20 PM

What happened instead?

On changing the message layout by clicking on Modern, the font size preview message doesn't get changed at the same time. The changes occur after refreshing the page. Something like this occurs-
Screenshot 2022-04-13 at 11 26 07 PM

Operating system

macOS

Browser information

Chrome

URL for webapp

No response

Application version

No response

Homeserver

No response

Will you send logs?

No

@robintown robintown added S-Minor Impairs non-critical functionality or suitable workarounds exist A-User-Settings A-Appearance O-Occasional Affects or can be seen by some users regularly or most users rarely good first issue Good for newcomers labels Apr 14, 2022
@bolu-tife
Copy link

Hello,
Please I would like to work on this. However, I'm not able to reproduce this error.

@goelesha
Copy link
Author

Hello, Please I would like to work on this. However, I'm not able to reproduce this error.

Actually I am Outreachy applicant and I have opened the issue and already working on it.

@bolu-tife
Copy link

Alright, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Appearance A-User-Settings good first issue Good for newcomers Help Wanted Extra attention is needed O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants