Closed (won't fix)
Project:
Vartheme Claro
Version:
4.0.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
25 Oct 2023 at 14:31 UTC
Updated:
28 Jul 2024 at 17:14 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
ibrahim tameme commentedComment #3
ibrahim tameme commentedComment #4
rajab natshahThanks, Ibrahim, for reporting and patching.
I reviews the code and tested
Let us switch to MR
Please, remove the
fas fa-homeicon classes.FA classes are used for now in the front-end theme
But the login page is using the Gin and Vartheme Claro admin themes
Let us follow with Gin Login templates
Only add the
Maybe ( if you file an issue to the Gin Login project, Sascha is a Senior Product Designer, He for sure knows which one is better, or place it in some other location. )
Steps:
1- Have the patch in the Gin Login module
2- Attache a patch to add changes over Gin Login templates
3- When your patch got committed ( or a better design is provided , we remove the patch )
Comment #5
rajab natshah