Live HTML highlighting is often in the wrong place with Reveal.js

[dangoor]

If you're working on a Reveal.js presentation, the live highlight is often in the wrong place (even when the cursor is in an element that is visible on the screen). It may be the case that there's no easy solution to this because of the way that Reveal.js does its layout, but I thought I'd open this for comment from the people who know the live highlighting better than I do.

[njx]

Reviewed. Medium priority to me to see if this is a more general problem.

[njx]

To @dangoor

[redmunds]

This may be fixed by #8922 by @marcelgerber. I tested a reveal presentation that I made for Sprint 30 Review -- it renders correctly in this branch, but also in master.

@dangoor Do you have the original reveal presentation that you can test against that PR?

[redmunds]

FBNC back to @dangoor .

[dangoor]

I just took a look at this. It is not fixed on current master (103e67d). I think this is likely a problem with the way Reveal does positioning rather than a more general problem, so I'm reducing this to low priority.

[marcelgerber]

@dangoor Could you upload a little Reveal presentation for me to take a look at that issue?

[redmunds]

@dangoor Note that highlighting of elements only on the current "page" of a Reveal presentation are expected to be correct. If that's the case, can you post the presentation for us to look at?

[dangoor]

I've uploaded the presentation. Just start live preview on index.html there.

[redmunds]

I didn't exhaustively test every page and element, but this looks to me. Can you provide a recipe for which page you are viewing in browser and which elements you are selecting in Brackets that have highlights out of place?

[marcelgerber]

I see the issue.
For the best effect, it's probably best to find some way to simplify jQuery's position as much as we can and use it there. That's basically what I did, but I oversimplified it (on purpose, I didn't want 100 lines hanging around).