-
Notifications
You must be signed in to change notification settings - Fork 523
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add "Join as dialogue" concatenation option #323
base: master
Are you sure you want to change the base?
Conversation
Agreed! |
Can you elaborate on what inspired the name of this new command? To me, "as dialogue" is not exactly telling about the functionality since – coming from ASS terminology – all non-commented lines are "dialogue" lines. From the functionality, I would lean more towards a name like "Join (different speakers)" or similar. What inspired the 4 different options for the join patterns? I mostly see Is there a use case for joining more than two lines or rather joining an already joined line with an unjoined line? |
Join as Dialogue loosely comes from SubtitleEdit's conventions, e.g. here. I can agree that it may appear as confusing terminology considering its meaning in the ASS spec, but for most users, I think it makes more sense and is short.
The 4 different conventions for joining were also wholly inspired from SubtitleEdit. The usecase for dash only to the second line is needed occasionally, e.g. for Bulgarian and Dutch. Initially, I also thought of having a single formatting string that the user may provide themselves, but I thought that it was too annoying to make a formatting string work using something in the standard library like
Not very likely. Occasionally, one may want to merge one or (ideally not more than) two additional subsequent lines if they are short, but that's about it. Edit: I added a branch called |
My initial reaction is that this is somewhat niche and ought to be an automation script, not built-in functionality. I'm open to being convinced otherwise. Regardless, I feel very strongly that it should not be called "Join as dialogue", regardless of SubtitleEdit conventions, due to the confusion with ASS. |
While I mostly agree, I think it would be nice if this could be put in the right-click menu, like the other join modes are. This is perhaps more a comment on wanting the ability to add automations in that menu, though... |
This is somewhat of a whataboutism, but the join (as karaoke) command feels the same to me. Different people use Aegisub in different ways. Personally, after using the program for years, I only finally understood how to do automation a few months ago. To my knowledge, there is no official tutorial for this, and the unofficial ones are not easily accessible and digestible.
Automations in the menu would be dope. |
I agree that it's currently harder than I'd like to get set up with automation scripts. We ship DepCtrl in the installer now at least - it's just not enabled by default. Once the performance problems are dealt with, I'd like to make it default and tutorialize it more properly. Also, I agree that join (as karaoke) is similar. I'd like to move more functionality like that to Moonscript in general, even for existing features, but this would definitely require having automations in the context menu. |
Add a fast way of joining two lines together as dialogue. See arch1t3cht#183