mirror of
https://github.com/cypht-org/cypht.git
synced 2026-04-25 04:56:03 +03:00
[GH-ISSUE #795] Menu icons broken in dark mode themes #483
Labels
No labels
2fa
I18N
PGP
Security
Security
account
advanced_search
advanced_search
announcement
api_login
authentication
awaiting feedback
blocker
bug
bug
bug
calendar
config
contacts
core
core
devops
docker
docs
duplicate
dynamic_login
enhancement
epic
feature
feeds
framework
github
github
gmail_contacts
good first issue
help wanted
history
history
imap
imap_folders
inline_message
installation
keyboard_shortcuts
keyboard_shortcuts
ldap_contacts
mobile
need-ssh-access
new module set
nux
pop3
profiles
pull-request
question
refactor
release
research
saved_searches
smtp
strategic
tags
tests
themes
website
wordpress
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/cypht#483
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @jasonmunro on GitHub (Oct 19, 2023).
Original GitHub issue: https://github.com/cypht-org/cypht/issues/795
Originally assigned to: @kambereBr on GitHub.
🐛 Bugreport
Trash, Junk, and Drafts menu icons broken in dark themes
Version & Environment
master as of today
Steps to reproduce
Dark themes inject a path attribute into icon SVG strings to alter the color to match the theme. Icons added to lib/output.php need to be added to the list in modules/themes/modules.php to have their colors altered (this is because we don't want to blindly alter all of the icons). For 2 of these icons that are currently broken they are defined as pngs which means they will never work properly (they must be non-binary svgs). The third (trash) is an svg but still is not working they way the rest do.
@jasonmunro commented on GitHub (Oct 19, 2023):
@marclaporte commented on GitHub (Oct 19, 2023):
Thank you @jasonmunro
These are 3 new item elements and we never tested with a dark theme
We will fix soon
@marclaporte commented on GitHub (Nov 4, 2023):
@jasonmunro Is the fix OK?
@jasonmunro commented on GitHub (Dec 21, 2023):
yep this is fixed, thanks!