-
Notifications
You must be signed in to change notification settings - Fork 95
[stable8] feat(NcUserStatusIcon): update status icons #7223
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
Conversation
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
6026fa6 to
2c1b1f9
Compare
|
How complicated would it be to read a capability/template var or something to not use these icons on old Server? |
IMO, the change is not so dramatic to consider it a problem, especially if this is about different icons on different clients and not different parts of a single page. We can provide icons via server-defined CSS variables to make sure they are the same everywhere. |
susnux
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we did similar in the past also for old server - so should be fine
You will see things in the calendar, mail and a lot of other apps that support multiple versions, before you can see it for yourself in the top right and it spills the refreshing redesign to old versions.
|
2c1b1f9 to
1ef9f40
Compare
|
Added legacy check through loadState (so it works with Talk Desktop) There are two more components with |
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
1ef9f40 to
72f6d94
Compare
Backport of #7218
Warning, This backport's changes differ from the original and might be incomplete⚠️