Skip to content

Conversation

andersonjeccel
Copy link
Contributor

@andersonjeccel andersonjeccel commented Jun 24, 2024

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 adds context for settings fields and changes the page appearance to reduce information density.
Specifically re. context, while tooltips are more functionality-descriptive, these help texts aim to be more like a smart tip or just provide the necessary information so the user have idea of what this setting is about without searching docs.

It also adds a panel to help people set their date format.

image

Before After
Screenshot From 2024-12-21 13-47-11 Screenshot From 2024-12-21 13-46-12

📋 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 Settings > Have a look at all tabs

@andersonjeccel andersonjeccel requested review from a team, Mike-Dropsolid and Esthertests June 24, 2024 21:01
@andersonjeccel andersonjeccel self-assigned this Jun 24, 2024
@andersonjeccel andersonjeccel added T1 Low difficulty to fix (issue) or test (PR) ready-to-test PR's that are ready to test code-review-needed PR's that require a code review before merging user-experience Anything related to related to workflows, feedback, and navigation enhancement Any improvement to an existing feature or functionality configuration Anything related to the Mautic configuration section labels Jun 24, 2024
@andersonjeccel andersonjeccel requested a review from LordRembo June 24, 2024 21:02
@andersonjeccel andersonjeccel changed the title [UX] Configuration extra info [UX] Configuration tips and extra context Jun 24, 2024
@andersonjeccel andersonjeccel added the blocked Something blocks this PR/issue (e.g. waiting for another PR to be merged) label Jun 25, 2024
Copy link
Contributor

@LordRembo LordRembo left a comment

Choose a reason for hiding this comment

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

Okay, yeah, I'm all for more tooltips.

Copy link
Contributor

@oltmanns-leuchtfeuer oltmanns-leuchtfeuer left a comment

Choose a reason for hiding this comment

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

I like it, helps out newer users a ton.

@oltmanns-leuchtfeuer oltmanns-leuchtfeuer added user-testing-passed PRs which have been successfully tested by the required number of people. and removed ready-to-test PR's that are ready to test labels Jul 2, 2024
@andersonjeccel andersonjeccel removed the blocked Something blocks this PR/issue (e.g. waiting for another PR to be merged) label Jul 2, 2024
@andersonjeccel andersonjeccel added this to the 5.2 milestone Jul 2, 2024
@escopecz escopecz removed this from the 5.2 milestone Dec 1, 2024
@andersonjeccel andersonjeccel changed the base branch from 5.x to 6.x December 21, 2024 16:49
@andersonjeccel andersonjeccel added this to the 6.0 milestone Dec 21, 2024
@andersonjeccel andersonjeccel added ready-to-test PR's that are ready to test and removed user-testing-passed PRs which have been successfully tested by the required number of people. labels Dec 21, 2024
@andersonjeccel andersonjeccel requested review from a team, mchojrin and laurielim and removed request for Mike-Dropsolid and Esthertests December 21, 2024 16:57
Copy link
Contributor

@shinde-rahul shinde-rahul left a comment

Choose a reason for hiding this comment

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

Code changes look good to me!!

@shinde-rahul shinde-rahul added code-review-passed PRs which have passed code review and removed code-review-needed PR's that require a code review before merging labels Jan 4, 2025
Copy link
Contributor

@shinde-rahul shinde-rahul left a comment

Choose a reason for hiding this comment

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

It works as described.

@andersonjeccel andersonjeccel added pending-test-confirmation PR's that require one test before they can be merged and removed ready-to-test PR's that are ready to test labels Jan 4, 2025
@escopecz escopecz merged commit 1c27fff into mautic:6.x Jan 6, 2025
15 checks passed
@escopecz escopecz removed the pending-test-confirmation PR's that require one test before they can be merged label Jan 6, 2025
@andersonjeccel andersonjeccel deleted the ux-configuration-extra-info branch January 6, 2025 11:47
@mautibot
Copy link
Contributor

This pull request has been mentioned on Mautic Forums. There might be relevant details there:

https://forum.mautic.org/t/mautic-6-is-here-alpha-release-is-ready-for-you-to-start-testing/34874/1

@mautibot
Copy link
Contributor

This pull request has been mentioned on Mautic Forums. There might be relevant details there:

https://forum.mautic.org/t/mautic-6-is-here-alpha-release-is-ready-for-you-to-start-testing/34900/1

@mautibot
Copy link
Contributor

This pull request has been mentioned on Mautic Forums. There might be relevant details there:

https://forum.mautic.org/t/mautic-6-0-orion-edition-is-released/35360/1

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 configuration Anything related to the Mautic configuration section enhancement Any improvement to an existing feature or functionality T1 Low difficulty to fix (issue) or test (PR) user-experience Anything related to related to workflows, feedback, and navigation
Projects
Archived in project
Status: Done
Development

Successfully merging this pull request may close these issues.

6 participants