Skip to content

1.5.1-beta2#499

Merged
ilyhalight merged 12 commits intoilyhalight:devfrom
NullVerdict:dev
Jan 30, 2024
Merged

1.5.1-beta2#499
ilyhalight merged 12 commits intoilyhalight:devfrom
NullVerdict:dev

Conversation

@NullVerdict
Copy link
Copy Markdown
Collaborator

Оптимизация и упрощение процессов и функционала для повышения эффективности. (Отказ от проверки на инициализацию и функции формата строки)

@NullVerdict NullVerdict marked this pull request as ready for review January 28, 2024 12:56
@NullVerdict
Copy link
Copy Markdown
Collaborator Author

Протестируй, если всё хорошо, объедини. @ilyhalight

@ilyhalight
Copy link
Copy Markdown
Owner

Есть небольшой визуальный баг.

При смене видео не меняется выбранный язык видео в меню, пока не нажмешь на кнопку перевода. Например, при смене с англоязычного видео на русскоязычное в меню останется английский язык до того момента пока не нажмешь на кнопку перевода или не обновишь страницу.

firefox_UiBUDOKX0h

@NullVerdict
Copy link
Copy Markdown
Collaborator Author

Есть небольшой визуальный баг.

При смене видео не меняется выбранный язык видео в меню, пока не нажмешь на кнопку перевода. Например, при смене с англоязычного видео на русскоязычное в меню останется английский язык до того момента пока не нажмешь на кнопку перевода или не обновишь страницу.

firefox_UiBUDOKX0h firefox_UiBUDOKX0h

Проверяй, вроде исправил.

@NullVerdict
Copy link
Copy Markdown
Collaborator Author

Есть небольшой визуальный баг.

При смене видео не меняется выбранный язык видео в меню, пока не нажмешь на кнопку перевода. Например, при смене с англоязычного видео на русскоязычное в меню останется английский язык до того момента пока не нажмешь на кнопку перевода или не обновишь страницу.

firefox_UiBUDOKX0h firefox_UiBUDOKX0h

Теперь точно должно быть исправлено

@ilyhalight
Copy link
Copy Markdown
Owner

Теперь, смена работает исправно. Пока тестил заметил, что еще отвалилось возвращение громкости аудио после отключения перевода, если громкость видео поставить на 0% через слайдер в меню.

Воспроизведение:

  1. Включаем перевод
  2. Убираем громкость оригинала до 0% через слайдер в меню
  3. Отключаем перевод
  4. Видим, что звук не вернулся до того значения, которое было до перевода

Воспроизводится в Chrome и Firefox

chrome_ULeJ4hkQB8

@NullVerdict
Copy link
Copy Markdown
Collaborator Author

NullVerdict commented Jan 30, 2024

Теперь, смена работает исправно. Пока тестил заметил, что еще отвалилось возвращение громкости аудио после отключения перевода, если громкость видео поставить на 0% через слайдер в меню.

Воспроизведение:

1. Включаем перевод

2. Убираем громкость оригинала до 0% через слайдер в меню

3. Отключаем перевод

4. Видим, что звук не вернулся до того значения, которое было до перевода

Воспроизводится в Chrome и Firefox

chrome_ULeJ4hkQB8 chrome_ULeJ4hkQB8

Так и задумано, при смене громкости, она сохраняется. Могу откатить, если не очень удобно.

    addExtraEventListener(this.video, "volumechange", () => {
      debug.log("lipsync mode is volumechange");
      let videoVolume = this.getVideoVolume();
      if (videoVolume !== this.data.autoVolume) {
        this.volumeOnStart = videoVolume;
      }
    });

@NullVerdict
Copy link
Copy Markdown
Collaborator Author

Теперь, смена работает исправно. Пока тестил заметил, что еще отвалилось возвращение громкости аудио после отключения перевода, если громкость видео поставить на 0% через слайдер в меню.

Воспроизведение:

1. Включаем перевод

2. Убираем громкость оригинала до 0% через слайдер в меню

3. Отключаем перевод

4. Видим, что звук не вернулся до того значения, которое было до перевода

Воспроизводится в Chrome и Firefox

chrome_ULeJ4hkQB8 chrome_ULeJ4hkQB8

Надеюсь это последнее что я должен был откатить...

@NullVerdict
Copy link
Copy Markdown
Collaborator Author

@ilyhalight Теперь всё устраивает?

@ilyhalight ilyhalight merged commit 2693b71 into ilyhalight:dev Jan 30, 2024
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.

2 participants