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

line-height for constraints #671

Merged
merged 1 commit into from Jan 9, 2014

Conversation

jrochkind
Copy link
Member

When contraints wrap to two lines, they needed more space.

constraints-vertical-collision

Fixed by increasing line-height of the .constraints-container, then had to adjust padding on other elements to leave spacing the same.

vertical-collision-fixed

(I'm also trying to figure out a solution to when the screen is so narrow and one individual constraint so lengthy that it goes off screen, but that's proving very tricky, so will do it as a separate PR, if at all)

@jcoyne
Copy link
Member

jcoyne commented Jan 7, 2014

I don't understand how this could have caused the failure:

  1) CatalogController index action with format :json facets should have items with labels and values
     Failure/Error: query_facet_items.first['label'].should == 'within 5 Years'
       expected: "within 5 Years"
            got: "within 10 Years" (using ==)
     # ./spec/controllers/catalog_controller_spec.rb:153:in `block (5 levels) in <top (required)>'

Perhaps the test data moved outside of a 5 year window?

@jrochkind
Copy link
Member Author

Thanks @jcoyne, me neither. Literally all I changed was styles in a .css file, as can be verified by the diff. (Didn't even change any classes in the HTML or anything).

i suspect the next commit that isn't me will trigger the same failure, and presumably people could trigger it running the tests locally too?

@cbeer
Copy link
Member

cbeer commented Jan 9, 2014

@jcoyne got it -- we only have test data up to 2008.

@jrochkind
Copy link
Member Author

thanks @cbeer! I see you fixed in master, thanks again. I'll merge or rebase this branch (is rebasing preferred? It always scares me so).

cbeer added a commit that referenced this pull request Jan 9, 2014
@cbeer cbeer merged commit 7cbb257 into projectblacklight:master Jan 9, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants