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

fix: change DeArrow title submission default #5909

Merged
merged 1 commit into from
Apr 17, 2024

Conversation

ajayyy
Copy link
Contributor

@ajayyy ajayyy commented Apr 15, 2024

"Ok" button disabled until at least one option is checked

@ajayyy
Copy link
Contributor Author

ajayyy commented Apr 15, 2024

Getting

14:27:48.690 [main] ERROR com.pinterest.ktlint.cli.internal.KtlintCommandLine -- Parameter '--code-style=android_studio' is ignored. The code style should be defined as '.editorconfig' property 'ktlint_code_style='.

when trying to run klint

@ajayyy ajayyy closed this Apr 15, 2024
@ajayyy ajayyy reopened this Apr 15, 2024
@Bnyro
Copy link
Member

Bnyro commented Apr 15, 2024

No worries, you can ignore ktlint.

Comment on lines 66 to 68

binding.titleCheckbox.setOnCheckedChangeListener(onChecked)
binding.thumbnailTimeCheckbox.setOnCheckedChangeListener(onChecked)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We already set onCheckedChangeListener's above, it's not possible to add multiple checked change listeners. Hence this results in none of the text input fields being usable anymore.

@Bnyro Bnyro merged commit 0c42c12 into libre-tube:master Apr 17, 2024
2 of 3 checks passed
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