Expanding reply chain with youtube video makes CSS not match up
-
So I expanded the in reply to indicator on this post to see which of my comments blakeyrat was replying to and i noticed something odd about the reply chain.
http://what.thedailywtf.com/t/r-cathighfive/8353/10
that first post of mine simply contained a youtube link that was oneboxed in. Now i get why it wasn't oneboxed in the expansion, but when it's omitted entirely odd things apparently happen...
-
Adding
.ember-view reply { min-height: 75px; }
Seems to be enough to fix it. I guess we can fix it in to local style (paging @PJH), but it should be fixed upstream as well. Anyone feels like going over to meta.d?
-
I guess we can fix it in to local style (paging @PJH)
While it looks 'off,' I think I'll just wait for it to be fixed up-stream...
-
While it looks 'off,' I think I'll just wait for it to be fixed up-stream...
off to meta.d to report then.
-
Just tried to repro on try.d - couldn't get it to display 3 previous posts..
-
did you have a reply chain or just a sequence of posts?
-
-
i see what's happened.... we turned off the hide the reply to indicator if the post in reply to is right above discobool thing. try didn't do that.
-
I'd've expected the first post shown to be repeated above the grey-boxed version of the second (and the OP above that.)
-
hmm...... something weird is going on then.... and i don't know what.
-
Yeah, I tried expanding on that topic at try.d, and it looks like the reply chain there is limited to just one reply.
-
Ah...
Default is one.
-
well then. wonder if i can make it repro on one reply?
EDIT: Yup. and it happens in discourse default too, although it is less obvious:
-
.ember-view
is about the most useless CSS selector you could use, don't use it.
-
Noted. Also, I got no notification of this post!
... or I just dismissed it while delirious. Too lazy to actually check.