-
-
Notifications
You must be signed in to change notification settings - Fork 2
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
Turning OFF 'person' does not hide 'person' emoji variants #13
Comments
I see we just do a rudimentary scan for "person": Lines 26 to 30 in 096a7b6
but a "person emoji" is apparently any "X" for which also "man X" and "woman X" exist: Lines 845 to 857 in 096a7b6
🤔 |
Is it Unicode version:
If so, maybe it needs to add setting for "Unicode version" in "Options". N.B. In Android app "UnicodePad" there is Unicode version setting for compatibility issues: |
Those are just Unicode versions that introduced the symbol. Unreliable and not future-compatible. I'd rather not (have the user) filter by Unicode version. We simply enumerate all the glyphs from the latest (vendored) Unicode standard, and the user might see some of the emoji icons blank, depending on the platform/font availability. |
Describe the bug
Disabling
person
does not hideperson
emoji variantsTo Reproduce
Steps to reproduce the behavior:
man
and/orwoman
to turns ONperson
to turns OFFExpected behavior
Hide
person
variants if isperson
is turned OFF, whileman
and/orwoman
is turned ON.Versions
Specify versions:
OS: Linux (Debian 11)
Qt: 5.15.2
EFCK: 1.0
Additional context, screenshots
The text was updated successfully, but these errors were encountered: