Gremlins it is.
That's definitely not where our code places that element.
Would it be possible to get the complete HTML of one of your diff viewer
pages? Preferably one with content you don't mind me having.
It's past 2AM here, so I'll probably have to resume this tomorrow. Thanks
for all your quick responses and detailed info!
Christian Hammond - christ...@beanbaginc.com
Review Board - https://www.reviewboard.org
Beanbag, Inc. - https://www.beanbaginc.com
On Sat, Feb 13, 2016 at 2:06 AM, <tra...@apptimize.com> wrote:
> I attached two screen shots this time around. It looks like you are
> expecting one of the parents of the .diff-collapse-btn to be a tbody
> element, but it does not appear that the expanded code is being wrapped in
> a tbody when the "parentOffset" variable is used.
> In my screenshots I expanded the table element that contains the new
> expanded code. The "top" screenshot shows the start of the table element
> and the first few lines of newly expanded code. The "scrolled down"
> screenshot shows the last few lines of expanded code before the normal code
> that was already being displayed by the diff (which is correctly wrapped in
> tbody elements).
> Supercharge your Review Board with Power Pack:
> Want us to host Review Board for you? Check out RBCommons:
> Happy user? Let us know! https://www.reviewboard.org/users/
> You received this message because you are subscribed to the Google Groups
> "reviewboard" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to reviewboard+unsubscr...@googlegroups.com.
> For more options, visit https://groups.google.com/d/optout.
Supercharge your Review Board with Power Pack:
Want us to host Review Board for you? Check out RBCommons:
Happy user? Let us know! https://www.reviewboard.org/users/
You received this message because you are subscribed to the Google Groups
To unsubscribe from this group and stop receiving emails from it, send an email
For more options, visit https://groups.google.com/d/optout.