-
Notifications
You must be signed in to change notification settings - Fork 43
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
Icon Configuration for Languages #860
Conversation
This pull request is deployed at test.editor.opencast.org/860/2022-10-28_17-01-49/ . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree with this PR. Correctly guessing all the language flags was a pipe dream anyway.
This also allows us to avoid the discussion whether country flags are sensible indicators for languages, which is nice.
Comments below are justodds and ends.
@@ -81,6 +81,13 @@ | |||
"captions/source+en" = "English" | |||
"captions/source+es" = "Spanish" | |||
|
|||
[subtitles.icons] | |||
# A list of icons to be displayed for languages defined above. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We could mention that these are totally optional? Arguably that might be obvious, but it might also not be. And in that case, no one feels pressured into thinking up icons when they don't want.
Alternatively comment the list out?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll update the description
This pull request is deployed at test.editor.opencast.org/860/2022-11-11_22-31-11/ . |
This pull request is deployed at test.editor.opencast.org/860/2022-11-11_22-32-12/ . |
I kind of feel like saying that if people want to be stupid, let them be stupid ;-) |
Yes, thinking about exactly that. Wouldn't spent any major on time this though if |
c722f7a
to
5d3bf00
Compare
This patch allows users to specify icons for languages selectors displayed in the subtitle editor. Users have to specify the selectable languages anyway and the algorithm used before guessed wrong for popular languages like English. If no icon is specified for a language, the selector will simply omit the icon and display just the label. This fixes opencast#825
5d3bf00
to
02940b1
Compare
This pull request is deployed at test.editor.opencast.org/860/2022-11-23_18-55-28/ . |
This patch allows users to specify icons for languages selectors displayed in the subtitle editor. Users have to specify the selectable languages anyway and the algorithm used before guessed wrong for popular languages like English.
If no icon is specified for a language, the selector will simply omit the icon and display just the label.
This fixes #825