fossasia / eventyay-video

https://video-dev.eventyay.com
Apache License 2.0
1.49k stars 22 forks source link

Only show audio translation option if available #189

Closed odkhang closed 1 month ago

odkhang commented 1 month ago

This PR closes/references issue #171 . It does so by check if translation size of video to hide/unhide audio translation box

How has this been tested?

Checklist

Summary by Sourcery

This pull request enhances the user interface by conditionally displaying the audio translation dropdown only when more than one language is available, improving the user experience by hiding unnecessary options.

sourcery-ai[bot] commented 1 month ago

Reviewer's Guide by Sourcery

This pull request addresses issue #171 by adding a conditional check to the audio translation dropdown in the 'item.vue' file. The dropdown will now only be displayed if more than one language is available. This change ensures that the audio translation option is only shown when applicable, improving the user interface.

File-Level Changes

Files Changes
webapp/src/views/rooms/item.vue Added a conditional check to display the audio translation dropdown only when multiple languages are available.

Tips - Trigger a new Sourcery review by commenting `@sourcery-ai review` on the pull request. - Continue your discussion with Sourcery by replying directly to review comments. - You can change your review settings at any time by accessing your [dashboard](https://app.sourcery.ai): - Enable or disable the Sourcery-generated pull request summary or reviewer's guide; - Change the review language; - You can always [contact us](mailto:support@sourcery.ai) if you have any questions or feedback.
odkhang commented 1 month ago

This room did not have audio translation image

image

No translation box is showed