-
Notifications
You must be signed in to change notification settings - Fork 29
Wrong order of dialogues #49
Comments
Yes, currently the renderer always appends new subs to the layer div, so new subs always appear in document order (i.e., below) existing subs. Is it okay to just reverse this for the layer1-3 divs - use insert instead of append? Or is there something else to consider? |
First dialogues ordered by "start time" (the earliest dialogue would be at the bottom). I don't think that thing with time important for common dialogues. But maybe it can be important for typesetting (for example when one dialogue "moves" another). In other words, "just reverse" would help in most cases. What to do with time - it is your choise :) |
Sorting by start time is technically not right. Eg: dialogue1 is from 2s to 7s and dialogue2 is from 1s to 5s. If seeking to 3s, then d1 should show before d2 (assuming \an2, d1 should be below d2). However if you play the video with vsfilter from 0s to 3s you will see it put d2 below d1, which is because of three reasons:
An interesting outcome of the third point above is that you can make it give different results. If you close and reopen the video and seek directly to 6s, now vsfilter will render d1 at the bottom. Then if you seek back to 3s, now it will render d2 above d1. Then if you seek to 0s and play normally, at 1s it will render d2 in the air, since that's where it's cached it now. I do understand that sorting by start time emulates the order vsfilter would've chosen if playing normally. At any rate, not moving dialogues when the current set of visible subs changes is another thing libjass can't do, since it relies on the browser to position the sub divs. Pausing and playing should not reorder the dialogues. They should still render in the same order as before pause. I'll investigate. |
Oh right, it reorders on pause and play for a similar reason to vsfilter's caching - libjass doesn't re-render subs that are already displayed. So in normal playback the |
In libjass:
In Aegisub (libass):
Test script:
P.S. Test almost the same as #48 but this issue about different thing so I decided to put it in separate issue.
The text was updated successfully, but these errors were encountered: