-
Notifications
You must be signed in to change notification settings - Fork 802
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: fixed the In-Built MIC to work properly after toggling the PSLab MIC #2422
Conversation
@@ -10,6 +10,7 @@ | |||
import androidx.core.content.ContextCompat; | |||
import androidx.fragment.app.Fragment; | |||
|
|||
import android.util.Log; |
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.
Looks like this is unused?
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.
Oh yes! My bad, I had added these for debugging. I'm removing them.
@CloudyPadmal Done:) |
I have tested the app without the fix and with the fix. I was able to reproduce the problem and I can confirm that I can toggle between both MICs now and the build-in microphone works whenever it is selected in the version which contains the fix. |
@AsCress, just a general remark: I just noticed that you created this pull request on your development branch. I don't know if this was intentional or not. I like to (and sometimes forget to) make an extra branch for every issue I work on (named e.g. issue2421). That way I can switch between working on different issues. Even more convenient: Waiting for the merge of a pull request does not block me from working on other issues. |
@marcnause Thank you very much for testing the fix and your comments!! I'll surely make extra branches for different issues from now on. |
Fixes #2421 and #2173
This issue was arising primarily due to the following two reasons:-
Changes
Screenshots / Recordings
This is a screenshot of the OscilloscopeActivity, after following the required steps to reproduce the issue, as mentioned in #2421.
As one can see, the issue is resolved.
Checklist:
strings.xml
,dimens.xml
andcolors.xml
without hard coding any value.strings.xml
,dimens.xml
orcolors.xml
.