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

feat: new Wave Generator UI #1943

Merged
merged 1 commit into from
Aug 18, 2019

Conversation

neel1998
Copy link
Member

@neel1998 neel1998 commented Aug 15, 2019

Fixes #1935

Changes: new Wave Generator UI

Screenshot/s for the changes:

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:
wave.zip

TODO for subsequesnt PRs

  1. Add preview Chart
  2. Remove wave headings from bottom black screen
  3. Update seekbar

@mariobehling @CloudyPadmal since this PR is getting too large, I will make the above changes is subsequent PRs. For now wave generator is working totally fine. I have mmoved view and save buttons to to action bar.... Please review

@neel1998 neel1998 self-assigned this Aug 15, 2019
@neel1998 neel1998 added the In Progress Developer is working on the problem label Aug 15, 2019
@neel1998 neel1998 requested a review from mariobehling August 16, 2019 18:10
@neel1998 neel1998 added Enhancement Improvement to an existing feature and removed In Progress Developer is working on the problem labels Aug 16, 2019
@neel1998
Copy link
Member Author

@CloudyPadmal please review

@CloudyPadmal CloudyPadmal merged commit 08c8c3e into fossasia:development Aug 18, 2019
@neel1998 neel1998 deleted the new_wave_gen_ui branch August 27, 2019 13:57
makoteq pushed a commit to makoteq/pslab-android that referenced this pull request Jan 16, 2020
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Wave Generator: Simplify design and Fix issues
2 participants