Skip to content

Conversation

@andersonjeccel
Copy link
Contributor

Q A
Bug fix? (use the a.b branch)
New feature/enhancement? (use the a.x branch) ✔️
Deprecations?
BC breaks? (use the c.x branch)
Automated tests included?
Related user documentation PR URL mautic/user-documentation#...
Related developer documentation PR URL mautic/developer-documentation-new#...
Issue(s) addressed Fixes #...

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:

  1. Open this PR on Gitpod or pull down for testing locally (see docs on testing PRs here)
  2. Open Emails > New > Pick any type > Check how Code mode is now the last option

@andersonjeccel andersonjeccel changed the title [UX] Defining code mode as last option [UXUI-170] Defining code mode as last option May 6, 2025
@andersonjeccel andersonjeccel self-assigned this May 6, 2025
@andersonjeccel andersonjeccel added T1 Low difficulty to fix (issue) or test (PR) code-review-needed PR's that require a code review before merging pending-test-confirmation PR's that require one test before they can be merged enhancement Any improvement to an existing feature or functionality user-experience Anything related to related to workflows, feedback, and navigation labels May 6, 2025
@andersonjeccel andersonjeccel changed the title [UXUI-170] Defining code mode as last option [UXUI-170] Defining code mode as last option in themes May 6, 2025
@codecov
Copy link

codecov bot commented May 6, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 66.25%. Comparing base (418051c) to head (ca051d1).
Report is 4 commits behind head on 7.x.

Additional details and impacted files

Impacted file tree graph

@@            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:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Member

@laurielim laurielim left a 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

@laurielim laurielim added code-review-passed PRs which have passed code review user-testing-passed PRs which have been successfully tested by the required number of people. and removed code-review-needed PR's that require a code review before merging pending-test-confirmation PR's that require one test before they can be merged labels May 7, 2025
@andersonjeccel andersonjeccel added the ready-to-commit PR's with 2 successful tests, 1 approval, automated tests and docs and is ready to be merged label May 7, 2025
Copy link
Member

@escopecz escopecz left a 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.

@escopecz escopecz added pending-test-confirmation PR's that require one test before they can be merged and removed ready-to-commit PR's with 2 successful tests, 1 approval, automated tests and docs and is ready to be merged labels May 20, 2025
Copy link

@kingsedem kingsedem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have tested this PR and it works as expected.

Before
image

After
image

@kingsedem kingsedem removed the pending-test-confirmation PR's that require one test before they can be merged label May 23, 2025
@escopecz escopecz added this to the 7.0.0-alpha milestone May 23, 2025
@escopecz escopecz added the ready-to-commit PR's with 2 successful tests, 1 approval, automated tests and docs and is ready to be merged label May 23, 2025
@escopecz escopecz merged commit 659cced into mautic:7.x May 23, 2025
20 checks passed
@andersonjeccel andersonjeccel deleted the UXUI-170-move-code-mode-as-last-theme-option branch May 23, 2025 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

code-review-passed PRs which have passed code review enhancement Any improvement to an existing feature or functionality ready-to-commit PR's with 2 successful tests, 1 approval, automated tests and docs and is ready to be merged T1 Low difficulty to fix (issue) or test (PR) user-experience Anything related to related to workflows, feedback, and navigation user-testing-passed PRs which have been successfully tested by the required number of people.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants