Skip to content

Navigation Menu

Sign in
Sign up
This repository was archived by the owner on May 21, 2026. It is now read-only.

fix: Min Value for Sample per Second , Bins , Number Channels - #598

Open
refactor-droidyy wants to merge 1 commit into
fossasia:development from
refactor-droidyy:setting
Open

fix: Min Value for Sample per Second , Bins , Number Channels #598
refactor-droidyy wants to merge 1 commit into
fossasia:development from
refactor-droidyy:setting

Conversation

@refactor-droidyy

@refactor-droidyy refactor-droidyy commented Jan 22, 2020

Copy link
Copy Markdown
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

<EditTextPreference
android:defaultValue="3"
android:digits="0123456789"

@dheerajkotwani dheerajkotwani Jan 24, 2020

Copy link
Copy Markdown
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 subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@jddeep jddeep Awaiting requested review from jddeep
@adityastic adityastic Awaiting requested review from adityastic
1 more reviewer
@dheerajkotwani dheerajkotwani dheerajkotwani requested changes
Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

fix Solution to a bug or an issue

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Min Value for Sample per Second , Bins , Number Channels

AltStyle によって変換されたページ (->オリジナル) /