Skip to content
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

fix(mrml-core): add missing attributes in mj-accordion-text and mj-button #513

Merged
merged 4 commits into from
Mar 1, 2025

Conversation

austinanguyen
Copy link
Contributor

@austinanguyen austinanguyen commented Feb 1, 2025

mj-accordion-text was missing letter-spacing and font-weight. mj-button was missing font-weight. Now supports rendering these attributes. https://documentation.mjml.io/#mj-button, https://documentation.mjml.io/#mj-accordion

@jdrouet jdrouet force-pushed the my-patched-version branch from 59c4e3c to e8fe78b Compare March 1, 2025 07:37
@jdrouet jdrouet changed the title Patched mj-accordion-text and mj-button fix(mrml-core): add missing attributes in mj-accordion-text and mj-button Mar 1, 2025
Copy link

codecov bot commented Mar 1, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.43%. Comparing base (5311e58) to head (7311a97).
Report is 114 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #513      +/-   ##
==========================================
- Coverage   92.89%   92.43%   -0.46%     
==========================================
  Files         227      201      -26     
  Lines       12203    12334     +131     
==========================================
+ Hits        11336    11401      +65     
- Misses        867      933      +66     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Signed-off-by: austin <austinanguyenwork@gmail.com>
Signed-off-by: austin <austinanguyenwork@gmail.com>
@jdrouet jdrouet force-pushed the my-patched-version branch from e8fe78b to b9b96fd Compare March 1, 2025 07:40
jdrouet added 2 commits March 1, 2025 08:44
Signed-off-by: Jérémie Drouet <jeremie.drouet@gmail.com>
Signed-off-by: Jérémie Drouet <jeremie.drouet@gmail.com>
@jdrouet jdrouet merged commit cdbcc9c into jdrouet:main Mar 1, 2025
15 checks passed
@github-actions github-actions bot mentioned this pull request Mar 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants