Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

R&D SWIPE DRAFT Feature/60 #179

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from
Draft

R&D SWIPE DRAFT Feature/60 #179

wants to merge 5 commits into from

Conversation

CJLucido
Copy link
Collaborator

Description

This allows swiping through the tutorial on screens smaller than 800px and clicking "next" on screens larger. It has been deemed unnecessary and there currently no resolutions for why a user would click next if they were swiping so further work has been postponed/paused. Current issues include stepper and button positioning, and potentially unnecessary code complexity.

Pull and make personal branch for any further development.

Fixes #60

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

Change Status

  • Incomplete/work-in-progress, PR is for discussion/feedback

How Has This Been Tested?

  • npm run startDev

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • My code has been reviewed by at least one peer
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • There are no merge conflicts

…or permissions, maybe buttons on each page. seems to be working in dev tools, need to do a draft PR
…ttempting to move stepper and next button to render function- displaying but not allowing repositioning
…ile view, also triggering permissions at proper times. next button only appears if screen width over 800px. stepper and button display not styled properly for mobile but currently UX didn't have fix for simultaneously having buttons and swipe to go 'next' so issue discontinued
@CJLucido CJLucido added the enhancement New feature or request label Feb 28, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Enhancement: Add swipe capability to tutorial
1 participant