Skip to content

Add tip counter - #1

Open
mmalykov wants to merge 2 commits into
masterfrom
add-tip-counter
Open

Add tip counter#1
mmalykov wants to merge 2 commits into
masterfrom
add-tip-counter

Conversation

@mmalykov

Copy link
Copy Markdown
Collaborator

No description provided.

@AliceAponasko

AliceAponasko commented Sep 14, 2016

Copy link
Copy Markdown
Member

Please connect linter! I will not be commenting typos on styleguide, please do that with linter.

@AliceAponasko

AliceAponasko commented Sep 14, 2016

Copy link
Copy Markdown
Member

unspecified

Not clear what to do in the initial screen, sumCountTextField should be focused so the keyboard will be up.

@AliceAponasko

AliceAponasko commented Sep 14, 2016

Copy link
Copy Markdown
Member

unspecified

iPhone 5
unspecified

@AliceAponasko

AliceAponasko commented Sep 14, 2016

Copy link
Copy Markdown
Member

The app doesn't show the bar in landscape mode (Command + ➡ in simulator).
Option 1: figure out an interface for landscape for every phone with constraints
Option 2: disable landscape rotation for the app (since the samvlu design does not specify landscape)

@AliceAponasko

Copy link
Copy Markdown
Member

App crashes if no value is specified and Done clicked


import Foundation

extension Double {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@AliceAponasko

Copy link
Copy Markdown
Member

Great idea with the Sum struct! 👏
How about adding couple unit tests to this project? I can help.

@AliceAponasko AliceAponasko left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Linter + design + all the changes

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