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

Visibility of PSLab-MIC button changes when PSLab device is not found #2180

Closed
wants to merge 2 commits into from

Conversation

arpoch
Copy link

@arpoch arpoch commented Jan 23, 2021

Fixes #2173

Changes:

  • The visibility of PSLab-MIC button is set to View.Gone when PSLab device is not found.

Screenshot/s for the changes:
Screenshot_20210122-225917_PSLab

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:

app-fdroid-debug.zip

Copy link
Member

@asaikarthikeya asaikarthikeya left a comment

Choose a reason for hiding this comment

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

Congrats for your first PR. Add spaces between two if loops.

@arpoch
Copy link
Author

arpoch commented Jan 23, 2021

Congrats for your first PR. Add spaces between two if loops.

Thanks. I have made the changes you asked. Can you please tell what does 'No newline at the end of file' means and what changes should I make now ?

Copy link
Member

@asaikarthikeya asaikarthikeya left a comment

Choose a reason for hiding this comment

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

Reason -- Not exactly sure but as per my understanding Generally when working with git or other Linux distros , They add '\n' at the end of each line by default sometime the miss to add. In this case \n is missing .

Solution - open terminal in your project folder and the checkout to your branch then try git diff --ignore-all-space .Might solve give a try

@CloudyPadmal
Copy link
Collaborator

That was a feature. Change is not necessary.

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.

Oscilloscope: After selecting PSLab MIC, In-Built-MIC does not work when PSLab device is not connected
3 participants