Skip to content

[UXUI-132] Implement tile components #14869

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 14 commits into
base: 7.x
Choose a base branch
from
Open

[UXUI-132] Implement tile components #14869

wants to merge 14 commits into from

Conversation

andersonjeccel
Copy link
Contributor

@andersonjeccel andersonjeccel commented Apr 10, 2025 •
edited
Loading

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 refactor some items across the platform to use tiles

Dependency (needs to be merged first):

Screencast.From.2025-04-10.10-55-16.webm

image

image

image

image

📋 Steps to test this PR:

Requirements (needed to follow steps)

  • Segments created
  • Tags created
  • Form created
  • Focus item created
  • Contact assigned to several segments and tags
  1. Open this PR on Gitpod or pull down for testing locally (see docs on testing PRs here)
  2. Open Contacts > Click on any to open the profile > Check tags and segments
  3. Open Forms > Click to open any > See the Form implementation tile and interact
  4. Repeat step 3 for Focus items

@andersonjeccel
[Component] Tile Mini
affb092
@andersonjeccel
[Component] Tile expandable
c310403
@andersonjeccel
Translations for expandable
7c1a52c
@andersonjeccel
[Component] Tile expandable interactive
0f0b454
@andersonjeccel
Applying tile to show contact tags and segments
7dcd2f7
@andersonjeccel
Squashed commit of the following:
8f73163 
commit 2bd6052
Author: Anderson J. Eccel <[email protected]>
Date:   Mon Mar 31 21:32:50 2025 -0300

    Adding animation to illustrations by default

commit 786dec4
Author: Anderson J. Eccel <[email protected]>
Date:   Mon Mar 31 21:32:31 2025 -0300

    Animation utilities

commit 91d0064
Author: Anderson, from Dropsolid <[email protected]>
Date:   Mon Dec 16 14:40:19 2024 -0300

    illustration template

commit 8016f6d
Author: Anderson, from Dropsolid <[email protected]>
Date:   Mon Dec 16 14:40:13 2024 -0300

    illustration classes

commit 9bed61b
Author: Anderson, from Dropsolid <[email protected]>
Date:   Mon Dec 16 14:11:27 2024 -0300

    adding illustration package svgs
@andersonjeccel
Improving modal transition
2a3b815
@andersonjeccel
Update variable name in illustration template
a714912
@andersonjeccel
Applying tile to show form installation
0b9b87d
@andersonjeccel
Applying tile to show focus implementation
7749d17
@andersonjeccel
Updating tile mini to accept custom attributes
bee9fc8
@andersonjeccel andersonjeccel self-assigned this Apr 10, 2025
@andersonjeccel andersonjeccel added T1 Low difficulty to fix (issue) or test (PR) user-interface Anything related to appearance, layout, and interactivity ready-to-test PR's that are ready to test code-review-needed PR's that require a code review before merging forms Anything related to forms focus-items Anything related to focus items contacts Anything related to contacts labels Apr 10, 2025
@andersonjeccel andersonjeccel moved this to 🦸🏻 Needs 2 tests in Open Source Fridays Apr 10, 2025
@andersonjeccel andersonjeccel moved this to In Progress in UX/UI Tiger Team Apr 10, 2025
@jos0405
Copy link
Contributor

jos0405 commented Apr 11, 2025 •
edited
Loading

Tags and segmens are listed one per line, no matter how long they are. Am I missing something?
image

@jos0405 jos0405 added the pending-feedback PR's and issues that are awaiting feedback from the author label Apr 11, 2025
@codecov Codecov
Copy link

codecov bot commented Apr 15, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 65.72%. Comparing base (84c9707) to head (a7015a3).
Report is 44 commits behind head on 7.x.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##                7.x   #14869      +/-   ##
============================================
+ Coverage     65.70%   65.72%   +0.01%     
- Complexity    34966    34990      +24     
============================================
  Files          2300     2302       +2     
  Lines        140658   140708      +50     
============================================
+ Hits          92420    92474      +54     
+ Misses        48238    48234       -4

see 14 files with indirect coverage changes

🚀 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.

@andersonjeccel andersonjeccel mentioned this pull request Apr 15, 2025
Open
@andersonjeccel andersonjeccel removed the pending-feedback PR's and issues that are awaiting feedback from the author label Apr 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@andersonjeccel andersonjeccel

Labels
code-review-needed PR's that require a code review before merging contacts Anything related to contacts focus-items Anything related to focus items forms Anything related to forms ready-to-test PR's that are ready to test T1 Low difficulty to fix (issue) or test (PR) user-interface Anything related to appearance, layout, and interactivity
Projects
Open Source Fridays
Status: 🦸🏻 Needs 2 tests
UX/UI Tiger Team
Status: In Progress
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@andersonjeccel @jos0405