Skip to content

The AutoCarousel Is Done it's in the component folder#17

Open
EmmanuelIbiyemi wants to merge 2 commits intocaxtonacollins:mainfrom
EmmanuelIbiyemi:autoscrollcarousel
Open

The AutoCarousel Is Done it's in the component folder#17
EmmanuelIbiyemi wants to merge 2 commits intocaxtonacollins:mainfrom
EmmanuelIbiyemi:autoscrollcarousel

Conversation

@EmmanuelIbiyemi
Copy link
Copy Markdown

  • Made a fade animation for the button to route to the next page .
  • When the auto scrolling reaches the last index or should i say last image it will do animation and stop .
  • If the way the image is placed is okay by you no problem but if you don't like it you can tell me
    **SO SORRY I COULD NOT SEND A PULL REQUEST HAD SOME ISSUES AND OTHER TASK I WAS HANDLING **

Copy link
Copy Markdown
Owner

@caxtonacollins caxtonacollins left a comment

Choose a reason for hiding this comment

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

implement change and make sure project builds before pushing

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

you are to use the previous images there

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

ok i will work on that now

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

i wasn't expecting a new component.
you where to work on the existing onboarding.tsx component

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

oh sorry for that taught i will make a new one let me fix that quickly

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

revert the expo dep upgrade

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

i reverted it back i stopped that upgrade and put the one i found their but lemme try and fix it am coming if it's giving you issue

@EmmanuelIbiyemi
Copy link
Copy Markdown
Author

Am done with it should i convert all the styling to nativewind type or leave it ?

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.

2 participants