-
Notifications
You must be signed in to change notification settings - Fork 100
feat: add caption card log feature to subtitle overlay #87
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
base: master
Are you sure you want to change the base?
Conversation
chaixshot
commented
Mar 9, 2025
- Follow CaptionLogEnabled setting
- Follow CaptionLogMax setting
- Follow overlay control panel changing.
- Follow SubtitleWindow.IsTranslationOnly changing.
I thought for a while, and for my usage, it might be better to show more sentences in a textbox like the vanilla app does. I don't know if you are happy with what I said? |
I believe the main advantage of the overlay mode is its lightweight design. Introducing multiple cards might make it appear overly bloated, negatively affecting usability and causing feature overlap with the main window. |
I think the multiple cards and overlay mode might not be inherently mutually exclusive in terms of functionality. Given that the main window's opacity isn't adjustable and it can feel somewhat cumbersome, I actually prefer using the overlay mode. The multiple card feature, on the other hand, ensures that I don't miss previous subtitles in certain situations, such as when encountering a series of short, consecutive sentences. If combining the two impacts performance, could this be mitigated by limiting the number of cards displayed, perhaps to a maximum of two? |
I like this solution. I will try displaying multiple sentences in Overlay Mode later. |
This is the first attempt to make caption log from the beginning. But sentence is changing to fast not like in this example, due to short sentence handler that makes me keep focusing on the subtitle instead of movie. Then I make it have two rows with the same order original and translated, but that wasn't enough; that way I made 6 rows total. |
This method, I think, the better way is we intro a build in transcribe instead of M$‘. But currently I dont have time for this |
8f36446
to
71938d4
Compare
92fe1e2
to
dbaa5fe
Compare