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: Min Value for Sample per Second , Bins , Number Channels #598

Open
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

refactor-droidyy
Copy link
Member

Fixes #597

Screenshot/s for the changes:

ezgif com-video-to-gif (1)

Checklist: [Please tick following check boxes with [x] if the respective task is completed]

  • I have used resources from strings.xml, dimens.xml and colors.xml without hard-coding them
  • No modifications done at the end of resource files strings.xml, dimens.xml or colors.xml
  • I have reformatted code in every file included in this PR [CTRL+ALT+L]
  • My code does not contain any extra lines or extra spaces
  • I have requested reviews from other members

APK for testing: [Compress the app-debug.apk file into a <feature>.rar or <feature>.zip file and upload it here]

@auto-label auto-label bot added the fix Solution to a bug or an issue label Jan 22, 2020
@@ -4,7 +4,7 @@

<EditTextPreference
android:defaultValue="3"
android:digits="0123456789"
Copy link
Member

Choose a reason for hiding this comment

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

This will prevent user from entering '0' from the input. What if user wants to enter '10'?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix Solution to a bug or an issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Min Value for Sample per Second , Bins , Number Channels
2 participants