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

Fully implement the "align descendants" rule for div. #2244

Merged
merged 1 commit into from Oct 12, 2015

Conversation

Ms2ger
Copy link
Contributor

@Ms2ger Ms2ger commented Oct 12, 2015

This adds -servo-left and -servo-right to complement -servo-center.

This intentionally doesn't try to address issue #7301.

This adds -servo-left and -servo-right to complement -servo-center.

This intentionally doesn't try to address issue #7301.
@Ms2ger
Copy link
Contributor Author

Ms2ger commented Oct 12, 2015

Code reviewed upstream.

Ms2ger added a commit that referenced this pull request Oct 12, 2015
…56e6cdaae7

Merge pull request #2244 from sync_b81c8717f3cfee77bcbdbc8bd10d3c56e6cdaae7
@Ms2ger Ms2ger merged commit d264561 into master Oct 12, 2015
@hoppipolla-critic-bot
Copy link

Critic review: https://critic.hoppipolla.co.uk/r/5884

This is an external review system which you may optionally use for the code review of your pull request.

In order to help critic track your changes, please do not make in-place history rewrites (e.g. via git rebase -i or git commit --amend) when updating this pull request.

@Ms2ger Ms2ger deleted the sync_b81c8717f3cfee77bcbdbc8bd10d3c56e6cdaae7 branch October 12, 2015 14:59
@tobie tobie added the html label Nov 19, 2015
<html>
<head>
<meta charset=utf-8>
<link rel="match" href="div-align-ref.html">
Copy link
Member

Choose a reason for hiding this comment

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

Shouldn't this line be in the div-align.html file instead?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

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

Successfully merging this pull request may close these issues.

None yet

5 participants