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 issue with broken two column layout in outlook #7287

Merged

Conversation

@danielcherrington
Copy link
Contributor

danielcherrington commented Feb 27, 2019

Fix issue where 2 column layout is broken in outlook

See issue #3871

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

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

Description:

Fix issue where 2 column layout is broken in outlook

Fix issue where 2 column layout is broken in outlook

See issue #3871
@alanhartless alanhartless added this to the 2.16.0 milestone Mar 12, 2019
@npracht npracht modified the milestone: 2.16.0 Jan 23, 2020
@RCheesley RCheesley added this to the 2.16.1 milestone Mar 9, 2020
@npracht npracht added this to Ready to test in Mautic 2 Mar 10, 2020
Copy link
Member

RCheesley left a comment

Code review only and looking at the discussion in the issue, this fix should resolve the problem.

@RCheesley RCheesley moved this from Ready to test to Needs a second test/review in Mautic 2 Mar 10, 2020
@RCheesley RCheesley linked an issue that may be closed by this pull request Mar 10, 2020
@RCheesley RCheesley added the L1 label Mar 18, 2020
@dennisameling

This comment has been minimized.

Copy link
Member

dennisameling commented Mar 18, 2020

Just tested and works as expected:

BEFORE:
image

AFTER:
image

Copy link
Member

dennisameling left a comment

Works as expected

@dennisameling dennisameling merged commit d9b12f6 into mautic:staging Mar 18, 2020
2 checks passed
2 checks passed
Scrutinizer Analysis: No new issues – Tests: passed
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Mautic 2 automation moved this from Needs a second test/review to Merged Mar 18, 2020
@mautibot

This comment has been minimized.

Copy link

mautibot commented Mar 20, 2020

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

https://forum.mautic.org/t/announcing-mautic-2-16-1-beta/13438/1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Mautic 2
  
Merged
Linked issues

Successfully merging this pull request may close these issues.

6 participants
You can’t perform that action at this time.