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

Add background-position and background-image to section content and wrapper #7211

Merged
merged 19 commits into from May 15, 2021

Conversation

Enc3phale
Copy link
Contributor

@Enc3phale Enc3phale commented Feb 4, 2019

Please be sure you are submitting this against the staging branch.

Q A
Bug fix? N
New feature? Y
Automated tests included? N
Related user documentation PR URL N
Related developer documentation PR URL N
Issues addressed (#s or URLs) N
BC breaks? N
Deprecations? N

Description:

This add background image position and all necessary fields to add image into content in Email builder and Landing builder.
Moreover this add Outlook tag for background-image when the builder is Email builder.

Steps to test this PR:

  1. Load up this PR
  2. Clear cache and regenerate assets
  3. Open Email Builder, Select a data-section element
  4. Then you'll see additional field in Customize Section menu to change the background image position, and all field necessary to add image into content. Input some value to it, then save.

screenshot_2019-02-04 editer l email - double opt-in webmecanik automation

@Enc3phale Enc3phale changed the title Add section background position Add background-image into section wrapper and content Feb 4, 2019
@Enc3phale Enc3phale changed the title Add background-image into section wrapper and content Add background-position and background-image to section content and wrapper Feb 4, 2019
@npracht npracht added ready-to-test PR's that are ready to test code-review-needed PR's that require a code review before merging enhancement Any improvement to an existing feature or functionality labels Feb 4, 2019
npracht
npracht previously approved these changes Feb 6, 2019
Copy link
Member

@npracht npracht left a comment

Choose a reason for hiding this comment

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

except some version of outlook <3 it works perfectly.

@npracht npracht 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 Feb 6, 2019
@alanhartless alanhartless added this to the 2.15.1 milestone Feb 6, 2019
@alanhartless alanhartless modified the milestones: 2.15.1, 2.16.0 Mar 12, 2019
@npracht
Copy link
Member

npracht commented May 3, 2019

Using in production environment.

@npracht npracht added this to Ready to Test (confirmation) in Mautic 2 Aug 15, 2019
@npracht npracht modified the milestone: 2.16.0 Jan 23, 2020
@npracht npracht added this to the 2.16.1 milestone Mar 6, 2020
@npracht npracht added this to Needs a second test/review in Mautic 2 Mar 10, 2020
@RCheesley
Copy link
Sponsor Member

@npracht this is a feature rather than a bugfix so it can't go into 2.16.1 - at least that is my understanding of semantic versioning. Let's move it to 3.1?

@RCheesley RCheesley modified the milestones: 2.16.1, 3.1.0 Mar 10, 2020
@RCheesley RCheesley removed this from Needs a second test/review in Mautic 2 Mar 10, 2020
@npracht
Copy link
Member

npracht commented Mar 10, 2020 via email

@npracht npracht added builder-legacy Anything related to the legacy email or landing page builders needs-rebase PR's that need to be rebased 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 Apr 6, 2020
@codecov
Copy link

codecov bot commented Nov 21, 2020

Codecov Report

Merging #7211 (20f7113) into features (f81ae01) will increase coverage by 0.01%.
The diff coverage is 100.00%.

Impacted file tree graph

@@              Coverage Diff               @@
##             features    #7211      +/-   ##
==============================================
+ Coverage       41.19%   41.20%   +0.01%     
  Complexity      34556    34556              
==============================================
  Files            2060     2060              
  Lines          111477   111497      +20     
==============================================
+ Hits            45926    45946      +20     
  Misses          65551    65551              
Impacted Files Coverage Δ Complexity Δ
...undles/CoreBundle/Form/Type/BuilderSectionType.php 100.00% <100.00%> (ø) 2.00 <0.00> (ø)

@kuzmany
Copy link
Member

kuzmany commented Nov 22, 2020

@RCheesley rebased on actual staging

@npracht npracht modified the milestones: 3.2.0, 3.2.1, 3.3 Nov 23, 2020
@npracht npracht changed the base branch from staging to features December 1, 2020 07:48
@npracht npracht removed needs-rebase PR's that need to be rebased pending-feedback PR's and issues that are awaiting feedback from the author labels Dec 1, 2020
@npracht npracht closed this Dec 20, 2020
@npracht npracht reopened this Dec 20, 2020
@npracht npracht modified the milestones: 3.3.0, Mautic 4.0 Feb 15, 2021
@RCheesley RCheesley modified the milestones: 4.0-alpha, 4.0-beta Mar 29, 2021
@npracht npracht modified the milestones: 4.0-beta, 4.0-rc May 14, 2021
Copy link
Sponsor 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.

Subject to tests passing given this is in use in production for some time at Webmecanik and had positive tests, LGTM! 🚀

@RCheesley RCheesley merged commit 8699293 into mautic:features May 15, 2021
ts-navghane added a commit to ts-navghane/mautic that referenced this pull request May 17, 2021
…riteria-ab-tests

* 'features' of github.com:mautic/mautic: (269 commits)
  Add background-position and background-image to section content and wrapper (mautic#7211)
  Fix API set multiselect empty value (mautic#9506)
  docs: add hluchas as a contributor (mautic#10039)
  Sync leaking memory (mautic#9299)
  docs: add incentfit as a contributor (mautic#10038)
  docs: add gabepri as a contributor (mautic#10037)
  form actions that register gotowebinar registrants now save join urls (mautic#9477)
  docs: add kuzmany as a contributor (mautic#10036)
  Add readme for Tag Manager repo (mautic#10035)
  Safe join company table (mautic#9973)
  docs: add fedys as a contributor (mautic#10034)
  docs: add anton-vlasenko as a contributor (mautic#10033)
  docs: add rcheesley as a contributor (mautic#10032)
  docs: add luguenth as a contributor (mautic#10031)
  docs: add snoblucha as a contributor (mautic#10030)
  docs: add stevedrobinson as a contributor (mautic#10029)
  Add stage name and stage date added to contacts report (mautic#8173)
  Fix DNC report channel link (mautic#10010)
  Revert the incorrect changes from previous commits.
  Fix PR comments.
  ...
@RCheesley RCheesley modified the milestones: 4.0-rc, 4.0-beta May 25, 2021
@npracht npracht deleted the add-section-background-position branch July 12, 2021 09:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
builder-legacy Anything related to the legacy email or landing page builders cla-signed The PR contributors have signed the contributors agreement enhancement Any improvement to an existing feature or functionality pending-test-confirmation PR's that require one test before they can be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants