Fixes #81 - Keep article images. #82
Conversation
|
It seems like the basic idea here was to remove items which contain more images than paragraphs. I don't know why that'd be desirable. Has anyone looked at the archaeology here to figure out why? |
|
It was already there in arc90's first released version. Hard to figure out what's the exact reasoning behind this… |
|
Hm. At the point where we remove the container, can we check if either it or its parent is a |
|
Good idea, will do. |
|
Now that's annoying, tests are green on my box while one of them breaks on Travis. Both envs are up to date. Oh dear. |
|
Ohh that's nice, only one of the two Travis builds is failing. That means intermittent I guess? Oh dear. |
|
Bah, that was just because I didn't rebase. |
n1k0
force-pushed
the
keep-images
branch
4 times, most recently
from
96c4ccb
to
b0cdad6
Compare
|
\o/ |
Fixes #81 - Keep article images.
Not super sure with this one, though that fixes #81 and doesn't break the rest of tests.
r? @leibovic @gijsk