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: app bar and back button in all sensor activity and add missing MPU925X sensor in sensor list index #2158

Merged

Conversation

CodeWithSouma
Copy link
Member

@CodeWithSouma CodeWithSouma commented Jan 9, 2021

Add app bar and back button in all sensor activity and add missing MPU925X sensor in sensor list index, Fix #2157

@CloudyPadmal

  1. Add app bar in all sensor activity
  2. Add missing MPU925X sensor in sensor list index
     Before change UI look like this                         After change UI look like this

Before change UI looks like this After change UI look like this

PSLab apk (before the change):before change PSLab.zip

PSLab apk (after the change):after change PSLab.zip

CloudyPadmal and others added 30 commits October 18, 2019 19:54
@CodeWithSouma
Copy link
Member Author

@CloudyPadmal sir, please check this PR

@CodeWithSouma
Copy link
Member Author

@CloudyPadmal @mariobehling please review this PR

Copy link
Collaborator

@CloudyPadmal CloudyPadmal left a comment

Choose a reason for hiding this comment

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

Looks good. Nice work @CodeWithSouma . Can you clean up the branch? There seem to be a lot of commits from development

@CloudyPadmal CloudyPadmal added Enhancement Improvement to an existing feature Status: Review Required Requested reviews from peers and maintainers labels Apr 1, 2021
@CloudyPadmal CloudyPadmal changed the title Add app bar and back button in all sensor activity and add missing MPU925X sensor in sensor list index fix: app bar and back button in all sensor activity and add missing MPU925X sensor in sensor list index Apr 1, 2021
@CodeWithSouma
Copy link
Member Author

Looks good. Nice work @CodeWithSouma . Can you clean up the branch? There seem to be a lot of commits from development

Sir, I don't how to squash commit please do something

@CloudyPadmal
Copy link
Collaborator

Aha. They look like small commits for this one PR. I'll squash it. Something similar to https://github.com/fossasia/pslab-android#how-to-git-squash will help squashing commits in future.

@CloudyPadmal CloudyPadmal merged commit 4d8c9df into fossasia:development Apr 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Improvement to an existing feature Status: Review Required Requested reviews from peers and maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add app bar and back button in all sensor activity and add missing MPU925X sensor in sensor list index
2 participants