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

update filter/sort heading sizes #86

Merged
merged 1 commit into from
Mar 1, 2017

Conversation

nickjvm
Copy link
Contributor

@nickjvm nickjvm commented Feb 21, 2017

What does this PR do?

Updates filter/sort headings to be more consistent. Also updated grommet dependency to 1.2.1 and fixed some flex-shrink issues with inline filters.

Codepen

https://codepen.io/nickjvm/pen/vgoPrO?editors=0010

Before
sidebar title ('filter by') and sort label are smaller than state & status filter labels.
google chromefeb 21 2017 2 33 15 pm
After
google chromefeb 21 2017 2 32 29 pm

<Box
{...boxProps}
flex={false}
pad={{...boxProps.pad, ...{between: 'small'}}}>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can this become pad={{...boxProps.pad, between: 'small'}} instead?

update grommet version and fix flex issues with inline filter boxes
@michaelgilley michaelgilley merged commit 725dc07 into grommet:master Mar 1, 2017
@nickjvm nickjvm deleted the sidebar-ui branch March 1, 2017 21:11
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