Closed ry5n closed 10 years ago
@ry5n I think lots of these components have really simple direct uses but the idea is to avoid rewriting stuff often. This looks good to me.
OK, cool. @kpeatt @jeffkamo can one of you check the tests for this and merge if it all checks out?
@ry5n Might get a chance to poke at this today otherwise it's my first priority on Monday.
Remove unsupported Align component features, support horizontal alignments and update code style. Incorporates the changes in #54. This makes the component pretty small; we should consider deprecating it in favour of direct use of flexbox.
Status: Ready to merge
Reviewers: @kpeatt @jeffkamo Ticket: Fixes #45
Changes
align-selft
. In fact, thec-align__item
sub-component is completely removed in favour of alignment modifiers on the parent.How to test-drive this PR