-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
[UXUI-170] Defining code mode as last option in themes #14983
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
[UXUI-170] Defining code mode as last option in themes #14983
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## 7.x #14983 +/- ##
=========================================
Coverage 66.25% 66.25%
Complexity 35144 35144
=========================================
Files 2314 2314
Lines 141585 141585
=========================================
Hits 93805 93805
Misses 47780 47780 🚀 New features to boost your workflow:
|
laurielim
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed code and tested. Works as described, code mode is at the end of the list of themes
escopecz
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not a fan of this, but if this gets another approval I'll merge it.
kingsedem
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Description
This PR moves Code mode as last option when picking Email themes to make UI-editable themes higher in priority as they make Mautic's adoption easier for beginners
📋 Steps to test this PR: