Skip to content
This repository was archived by the owner on Oct 29, 2020. It is now read-only.

Fix issue with column overflow in Chrome 45.#5545

Merged
DFurnes merged 1 commit intoDoSomethingArchive:devfrom
DFurnes:finder-chrome-column-bugginess
Oct 19, 2015
Merged

Fix issue with column overflow in Chrome 45.#5545
DFurnes merged 1 commit intoDoSomethingArchive:devfrom
DFurnes:finder-chrome-column-bugginess

Conversation

@DFurnes
Copy link
Contributor

@DFurnes DFurnes commented Oct 19, 2015

Changes

Fixes #5409. Resolves issue with columns overflowing in Chrome 45. (Get it together, Google!)

How should I test this?

Check it out in all sorts of browsers and screen sizes. Tested across a few screen sizes in Chrome 45, Safari 9, Firefox 41 and thing seem to be working well.

Screenshots

Before (in Chrome 45):
before

After (in Chrome 45):
after

For review: @DoSomething/front-end

@sbsmith86
Copy link
Contributor

looks good to me :) what's the deal? Google no like display:inline anymore?

@DFurnes
Copy link
Contributor Author

DFurnes commented Oct 19, 2015

We had some prior issues with CSS columns in the Chrome 45 that were fixed by switching things from display: block to inline. It turns out that causes some other problems (again, only in Chrome though... 😩), but inline-block seems like the happy medium.

@sbsmith86
Copy link
Contributor

cool!

DFurnes added a commit that referenced this pull request Oct 19, 2015
Fix issue with column overflow in Chrome 45.
@DFurnes DFurnes merged commit b48a750 into DoSomethingArchive:dev Oct 19, 2015
@DFurnes DFurnes deleted the finder-chrome-column-bugginess branch October 19, 2015 17:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants