-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
DRAFT use images from spec package for SAS verification #31571
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
base: develop
Are you sure you want to change the base?
Conversation
|
I think this would need a solution to the dark-on-dark problem, as in a dark theme certain icons (e.g. spanner) may fail contrast accessibility. Our custom build of Twemoji deals with this already. |
Very good point. What solution would you propose? |
|
iirc M did some discovery in the area and a light outline was a common solution across other apps. That should really be done on the source side rather than trying to special-case them here given the icons could change within the spec and the outlines become mismatched from the appropriate icons otherwise |
|
I think so, but perhaps Design would know best |
|
Current Element Web doesn't appear to do anything special to make emoji like the hat below more visible:
but I definitely think we should address this in MSC4347 |
|
I guess Hat didn't make it into Matthew's firing line yet, though he did not go for the solution of a white outline instead defaulted to using monochrome emoji for glyphs which were hard to see on dark |
|
@ara4n (context: I am prototyping MSC4347, hard-coding the same exact emoji images for verification) I am thinking the best solution to support dark themes would be to modify the actual SVGs to add a white circle behind each image, making the images usable on all themes, and identical on all themes. I've asked for input from design in case I'm missing something, but do you have comments about whether that is a sensible approach? |
|
OK, after chatting with @aaronraimist I decided to prototype white circles incorporated into the images. I've updated the description of this item with a preview. The images are processed via a script in matrix-org/matrix-spec#2273 and pulled from the package as before. |



DEPENDS ON A TEST VERSION OF THE SPEC PACKAGE.
This PR is intended to demonstrate how MSC4347 could work in Element Web and hopefully qualify as an implementation of that MSC.
It depends on a temporary version of the
@matrix-org/specpackage modified as proposed in MSC3447 and uses images from that package to render emoji for SAS verification, instead of using an emoji font to draw the characters.