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

Implemented #511 Receive UI: 2.0 , details are as follows: #553

Closed
wants to merge 9 commits into from

Conversation

zaxayn
Copy link

@zaxayn zaxayn commented Dec 12, 2019

Issue:Receive UI 2.0 #511
1.Modified ReceiveViewController.swift and ReceiveViewController scene of Main.storyboard
2.Added ReceiveAccountListView.swift, ReceiveAccountListCell.swift, ReceiveAccountListCell.xib
3.Modified LocalizedStrings.swift, Localizable.strings in order to add some strings
There are a problem that unsolved:Data Display of Receive UI:2.0, Receive UI:2.0 Need the following data structure:
WX20191212-142542

1.Modified ReceiveViewController.swift and ReceiveViewController scene of Main.storyboard
2.Added ReceiveAccountListView.swift, ReceiveAccountListCell.swift, ReceiveAccountListCell.xib
3.Modified LocalizedStrings.swift, Localizable.strings in order to add some strings
Copy link
Contributor

@macsleven macsleven left a comment

Choose a reason for hiding this comment

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

code review

Decred Wallet/en.lproj/Localizable.strings Outdated Show resolved Hide resolved
Decred Wallet/es.lproj/Localizable.strings Outdated Show resolved Hide resolved
Decred Wallet/table_view_cell/ReceiveAccountListCell.swift Outdated Show resolved Hide resolved
Decred Wallet/view_controller/ReceiveViewController.swift Outdated Show resolved Hide resolved
Decred Wallet/view_controller/ReceiveViewController.swift Outdated Show resolved Hide resolved
Decred Wallet/view_util/ReceiveAccountListView.swift Outdated Show resolved Hide resolved
Decred Wallet/view_controller/ReceiveViewController.swift Outdated Show resolved Hide resolved
Decred Wallet/view_controller/ReceiveViewController.swift Outdated Show resolved Hide resolved
Decred Wallet/view_controller/ReceiveViewController.swift Outdated Show resolved Hide resolved
Decred Wallet/view_controller/ReceiveViewController.swift Outdated Show resolved Hide resolved
Copy link
Contributor

@itswisdomagain itswisdomagain left a comment

Choose a reason for hiding this comment

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

I noticed that your icons only have 2x and 3x versions, and may not be the exact/right qualities. The official design assets are here: https://drive.google.com/drive/folders/1wW5UJwFn2JmOTVOYapxMjF08ilLt6VWR

Decred Wallet.xcodeproj/project.pbxproj Outdated Show resolved Hide resolved
Decred Wallet.xcodeproj/project.pbxproj Outdated Show resolved Hide resolved
Decred Wallet.xcodeproj/project.pbxproj Outdated Show resolved Hide resolved
Decred Wallet/Assets.xcassets/receive/Contents.json Outdated Show resolved Hide resolved
Decred Wallet/Main.storyboard Outdated Show resolved Hide resolved
Decred Wallet/view_controller/ReceiveViewController.swift Outdated Show resolved Hide resolved
Decred Wallet/view_controller/ReceiveViewController.swift Outdated Show resolved Hide resolved
Decred Wallet/view_controller/ReceiveViewController.swift Outdated Show resolved Hide resolved
Decred Wallet/view_controller/ReceiveViewController.swift Outdated Show resolved Hide resolved
Copy link
Contributor

@bgptr bgptr left a comment

Choose a reason for hiding this comment

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

Thank you for this PR.
Overall it works nicely, although I suggest some cleanup.

Decred Wallet/Custom Views/AccountTableView.swift Outdated Show resolved Hide resolved
Decred Wallet/Custom Views/AccountTableView.swift Outdated Show resolved Hide resolved
Decred Wallet/Custom Views/AccountTableView.swift Outdated Show resolved Hide resolved
Decred Wallet/Custom Views/AccountTableView.swift Outdated Show resolved Hide resolved
Decred Wallet/Custom Views/AccountTableView.swift Outdated Show resolved Hide resolved
Decred Wallet/Custom Views/AccountTableView.swift Outdated Show resolved Hide resolved
Decred Wallet/Custom Views/AccountTableView.swift Outdated Show resolved Hide resolved
Decred Wallet/Custom Views/AccountTableView.swift Outdated Show resolved Hide resolved
Decred Wallet/Custom Views/AccountTableView.swift Outdated Show resolved Hide resolved
Decred Wallet/Custom Views/AccountTableView.swift Outdated Show resolved Hide resolved
Decred Wallet/Custom Views/AccountTableView.swift Outdated Show resolved Hide resolved
Decred Wallet/Custom Views/AccountTableView.swift Outdated Show resolved Hide resolved
Decred Wallet/Custom Views/AccountTableView.swift Outdated Show resolved Hide resolved
Decred Wallet/Custom Views/AccountTableView.swift Outdated Show resolved Hide resolved
Decred Wallet/Custom Views/AccountTableView.swift Outdated Show resolved Hide resolved
Decred Wallet/Features/Receive/ReceiveViewController.swift Outdated Show resolved Hide resolved
Decred Wallet/Features/Receive/ReceiveViewController.swift Outdated Show resolved Hide resolved
Decred Wallet/Features/Receive/ReceiveViewController.swift Outdated Show resolved Hide resolved
Decred Wallet/Features/Receive/ReceiveViewController.swift Outdated Show resolved Hide resolved
Decred Wallet/Features/Receive/ReceiveViewController.swift Outdated Show resolved Hide resolved
@zaxayn zaxayn requested a review from bgptr January 7, 2020 01:42
Decred Wallet/Custom Views/AccountTableViewCell.swift Outdated Show resolved Hide resolved
Decred Wallet/Features/Receive/ReceiveViewController.swift Outdated Show resolved Hide resolved
Decred Wallet/Features/Receive/ReceiveViewController.swift Outdated Show resolved Hide resolved
Decred Wallet/Features/Receive/ReceiveViewController.swift Outdated Show resolved Hide resolved
Decred Wallet/Features/Receive/ReceiveViewController.swift Outdated Show resolved Hide resolved
Decred Wallet/Custom Views/AccountTableView.swift Outdated Show resolved Hide resolved
Decred Wallet/Custom Views/AccountTableView.swift Outdated Show resolved Hide resolved
Decred Wallet/Custom Views/AccountTableView.swift Outdated Show resolved Hide resolved
Decred Wallet/Custom Views/AccountTableView.swift Outdated Show resolved Hide resolved
Decred Wallet/Custom Views/AccountTableView.swift Outdated Show resolved Hide resolved
@bgptr
Copy link
Contributor

bgptr commented Jan 7, 2020

I just noticed the AccountTableView showing/hiding animation is not working properly. It is just showing up without duration.

@zaxayn zaxayn closed this Jan 8, 2020
@zaxayn zaxayn reopened this Jan 8, 2020
@zaxayn
Copy link
Author

zaxayn commented Jan 8, 2020

I just noticed the AccountTableView showing/hiding animation is not working properly. It is just showing up without duration.

have resolved!

@zaxayn zaxayn requested a review from bgptr January 8, 2020 03:36
Copy link
Contributor

@macsleven macsleven left a comment

Choose a reason for hiding this comment

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

i can't seem to build this branch. there is an issue with the file "Decred Wallet.xcodeproj/project.pbxproj". it wasn't merged properly

@itswisdomagain
Copy link
Contributor

This PR has been stale for almost a month. This feature is currently being implemented a new PR: #583.

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.

5 participants