mirror of
https://github.com/cypht-org/cypht.git
synced 2026-04-25 04:56:03 +03:00
[PR #649] [CLOSED] Icons - theme compatibility - use Bootstrap icons instead of svg ones #956
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#956
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?
📋 Pull Request Information
Original PR: https://github.com/cypht-org/cypht/pull/649
Author: @johnsantosDev
Created: 10/30/2022
Status: ❌ Closed
Base:
master← Head:theme-compatibility-with-icons📄 Description
Pullrequest
PROBLEM & LINK TO TASK DESCRIPTION

https://avan.tech/item41850-Cypht-in-Tiki-Change-icon-colors-by-inserting-a-fill-attribute-into-the-SVGs
The mission is to make Cypht icons look acceptable with all the Tiki themes, using the method suggested by Jason (or propose better).
After checking about this issue, I found that the better way is to allow using the existing bootstrap icons instead of SVG ones, why? because they're compatible with all all themes.
I am making it optional so that user can decide to either keep using svg icons or switch to bootstrap icons.
Issues
Icons color impactibility with the existing themes
Checklist
With this change, COMPOSER was updated, here's the version
"twbs/bootstrap-icons": "^1.9"How2Test
You need to run composer update in order to make tests
For your test, update the file hm3.ini by adding the line
default_icons_to_bootstrap_icons=yTodo
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.