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

Update head components section at creating_themes.rst #248

Merged
merged 2 commits into from
Mar 5, 2024

Conversation

markusVJH
Copy link
Contributor

@markusVJH markusVJH commented Mar 5, 2024

Related issue #247. I since tested all of the mj-head components by sending real emails through Mautic.

This pr would update the head components section at creating_themes.rst to state that most head components do work, including mj-style.

Previously it stated that none of the head components work, and that all styling has to be inline. This has caused confusion in the past (Example 1, Example 2), as it did for me as a new Mautic user

Regarding issue mautic#247

Updating the head components section at creating_themes.rst to state most head components work, including mj-style
Copy link
Member

@RCheesley RCheesley left a comment

Choose a reason for hiding this comment

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

Thanks for making the PR @markusVJH and welcome to the community!

Here are a few fixes to bring this in line with our styleguide, should be quick to implement!

docs/builders/creating_themes.rst Outdated Show resolved Hide resolved
docs/builders/creating_themes.rst Outdated Show resolved Hide resolved
@RCheesley RCheesley added enhancement New feature or request pending-feedback labels Mar 5, 2024
Copy link
Member

@RCheesley RCheesley left a comment

Choose a reason for hiding this comment

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

LGTM!

@RCheesley
Copy link
Member

@all-contributors please add @markusVJH for docs

Copy link
Contributor

@RCheesley

I've put up a pull request to add @markusVJH! 🎉

@RCheesley RCheesley merged commit a38ad61 into mautic:5.x Mar 5, 2024
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request pending-feedback
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants