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

iPages Review #1

Closed
AlexFine opened this issue Oct 6, 2020 · 0 comments
Closed

iPages Review #1

AlexFine opened this issue Oct 6, 2020 · 0 comments

Comments

@AlexFine
Copy link
Collaborator

AlexFine commented Oct 6, 2020

Swiping is dope. Super natural spring. I really fuck with that. Similar to the other repositories, all comments here are ease of use or aesthetic related. The engine runs well!

  1. Page number confusion. I wanted to see if currentPage was zero indexed or 1 indexed and it wasn't in the documentation. Easy thing to add.

  2. The app crashes when you enter a negative current page. Consider making current page an unsigned int, so change Int to UInt. I think that makes more sense in this context bc you could never have a negative page count.

  3. The app crashes when you enter a non-valid current page. Thoughts on just making it instead go to the nearest page? If you enter page 9 and there are 6 pages just send it to page six.

  4. An aesthetic point, but I did very much expect some dots at the bottom of the view. At least something to indicate the type of element I was dealing with. My first step was to I look at the init's & the modifiers to try to see if I was simply missing that option and didn't see anything.

  5. It was really difficult to figure out the default height and width of the view. It was also difficult to figure out if that height and width was inherited from the subviews. Unsure the best way to communicate that information, but definitely confusing.

  6. We should toss in a sexy dummy view that'd be cool.

@blsage blsage closed this as completed Oct 7, 2020
AlexFine added a commit that referenced this issue Feb 14, 2021
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

No branches or pull requests

2 participants