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

Including css declarations of flex specific prefixes for IE10 to postion correctly #7572

Merged
merged 1 commit into from
Feb 25, 2016

Conversation

sg00dwin
Copy link
Member

- This is a temporary fix until PostCSS Autoprefixer is included to address
  https://github.com/openshift/origin/issues/7559

@spadgett
Copy link
Member

Should we remove the problematic ones from the mixin as well?

@jwforres
Copy link
Member

yeah good call @spadgett

removed merge for now. actually want to wait to merge till the other big CSS PR goes in otherwise this might cause a conflict

@sg00dwin
Copy link
Member Author

@jwforres ok, I updated the pr and removed the incorrect prefixes from _flexbox.less

@jwforres jwforres added this to the 1.2.0 milestone Feb 24, 2016
@jwforres jwforres added the lgtm Indicates that a PR is ready to be merged. label Feb 24, 2016
@jwforres
Copy link
Member

waiting for #7475 before merge

@openshift-bot openshift-bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 25, 2016
…tion correctly

	- This is a temporary fix until PostCSS Autoprefixer is included to address
	  openshift#7559
@sg00dwin
Copy link
Member Author

@jwforres rebased

@openshift-bot openshift-bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 25, 2016
@spadgett
Copy link
Member

[merge]

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/1625/) (Image: devenv-rhel7_3543)

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to aa993e0

@openshift-bot
Copy link
Contributor

[Test]ing while waiting on the merge queue

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to aa993e0

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/1625/)

openshift-bot pushed a commit that referenced this pull request Feb 25, 2016
@openshift-bot openshift-bot merged commit 9bb19ef into openshift:master Feb 25, 2016
@sg00dwin sg00dwin deleted the flex-ie-prefix branch February 25, 2016 20:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/web lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants