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

fix(layout): fix IE11 layout #1227

Closed

Conversation

patrykkopycinski
Copy link
Contributor

Fixes for #536

@patrykkopycinski patrykkopycinski changed the title Fix ie layout fix(layout): fix IE11 layout #536 Jan 19, 2015
@patrykkopycinski patrykkopycinski changed the title fix(layout): fix IE11 layout #536 fix(layout): fix IE11 layout Jan 19, 2015
@marcysutton marcysutton self-assigned this Jan 19, 2015
marcysutton pushed a commit that referenced this pull request Jan 19, 2015
@patrykkopycinski
Copy link
Contributor Author

Hi @marcysutton,

could you explain me why you closed my PR without merge commits? And instead of this you created your own branch with the same changes and then merged your branch?

@marcysutton
Copy link
Contributor

There was an issue with the patch where the changes wouldn't apply, so I had to recreate them in a new branch. We have a specific merge process to keep our changelog clean. I'll try to find a way around it next time.

marcysutton pushed a commit that referenced this pull request Jan 23, 2015
Fix in 74fe3eb was too dangerous as a global change, reverting to minimize layout issues.
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.

2 participants