Skip to content
This repository has been archived by the owner on Sep 16, 2019. It is now read-only.

Made it easier to understand #584

Merged
merged 3 commits into from
Dec 8, 2015
Merged

Made it easier to understand #584

merged 3 commits into from
Dec 8, 2015

Conversation

alexbohariuc
Copy link
Contributor

I made a small change (closed the php tag and got the images with echo) - it is easier to understand the code this way.

@olefredrik
Copy link
Owner

Thanks! Can you fix the Travis-CI build error and add a new commit? https://travis-ci.org/olefredrik/FoundationPress/jobs/95248578

@Aetles
Copy link
Contributor

Aetles commented Dec 7, 2015

It's also a new div with the title that was not there before, right?

@olefredrik
Copy link
Owner

Well spotted. Why would we need that extra div @alexbohariuc ?

@alexbohariuc
Copy link
Contributor Author

I did some design changes for a project. I will remove it and then add a new commit.

olefredrik added a commit that referenced this pull request Dec 8, 2015
Made it easier to understand
@olefredrik olefredrik merged commit 11be847 into olefredrik:master Dec 8, 2015
@Aetles
Copy link
Contributor

Aetles commented Dec 8, 2015

Not so hard to spot if you read the diff, was only five new lines… 😉

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants