Skip to content
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

Embedded subtitle soft #8915

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

ivandrofly
Copy link
Member

Do not block UI / Windows explorer when big file is dropped
Add progress bar which will be visible when dropping large file / several files

Enhance the process of adding subtitles to the ListView by refactoring the code and introducing asynchronous file processing. This includes consolidating redundant methods into a single method that generates the VideoPreviewGeneratorSub object and ensuring null values are handled. Improved user experience by setting the cursor to a wait state during background operations.

Signed-off-by: Ivandro Jao <[email protected]>
Display a "please wait" message and change the cursor to a wait cursor when files are being processed in the drag-and-drop operation. Clear the message and reset the cursor once processing is complete.

Signed-off-by: Ivandro Jao <[email protected]>
Implemented a visible progress bar to give user feedback during drag-and-drop operations in the GenerateVideoWithSoftSubs form. This enhances the UI by indicating processing status and improving user experience.

Signed-off-by: Ivandro Jao <[email protected]>
This commit deletes the unused variable _inputVideoFileName from the GenerateVideoWithSoftSubs class. This cleanup improves code readability and reduces potential confusion during future maintenance.

Signed-off-by: Ivandro Jao <[email protected]>
Wrap subtitle list operations within BeginUpdate and EndUpdate calls to reduce UI flicker and improve performance during batch item insertions.

Signed-off-by: Ivandro Jao <[email protected]>
Extracted wait state activation and deactivation into separate methods: `ActivateWaitState` and `DeactivateWaitState`. This improves code readability and reduces repetition.

Signed-off-by: Ivandro Jao <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant