Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

chore(travis): update browsers to the latest version #10620

Closed
wants to merge 1 commit into from

Conversation

lgalfaso
Copy link
Contributor

@lgalfaso lgalfaso commented Jan 2, 2015

Update the used browsers to the latest versions available

Update the used browsers to the latest versions available
@petebacondarwin
Copy link
Contributor

@lgalfaso - there appear to be two fixes in this commit.
One is related to a bug in Safari I believe? (The old 2px thing). I think this should be in a separate commit.

@lgalfaso
Copy link
Contributor Author

Newer versions of Safari do not update the calculated properties (after a programatic css change) synchronously. The change to the test is needed to make it pass in Safari (I can make it 2 commits, but just the browser update without the change on the test will make the test fail in Safari)

@petebacondarwin
Copy link
Contributor

You can always have commits that fail followed by commits that fix them as long as they are pushed together. But in this case, it is not needed. We just put the "fix" commit first.

@petebacondarwin
Copy link
Contributor

I guess we backport to 1.3.x

petebacondarwin added a commit that referenced this pull request Jan 19, 2015
Update the used browsers to the latest versions available

Closes #10620
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants