In the RevisionSlider popups I see the <bdi> tag used a lot.
It is useful for resolving certain RTL issues, but I'm not sure that it's needed in this code. For example, I see that in some instances, there is a <p> tag inside a <bdi> tag, which doesn't make sense, because <bdi> provides bidi isolation, but <p> is already isolated. Either I misunderstand the intention behind using this tag, or it helps fix some browser bugs with which I am not familiar, or it is just unnecessary :)
I'll be happy to help clean its usage up.