Skip to content
This repository was archived by the owner on Jun 16, 2020. It is now read-only.

feat: TransactionRow#110

Open
kielbarry wants to merge 149 commits intoethereum:devfrom
kielbarry:TransactionRow
Open

feat: TransactionRow#110
kielbarry wants to merge 149 commits intoethereum:devfrom
kielbarry:TransactionRow

Conversation

@kielbarry
Copy link
Contributor

What does it do?

Provides a row describing transactions to be in a table. Conditions and css apply for unconfirmed and confirmed transactions, and for even and odd counts.

Any helpful background information?

TransactionConfirmations.jsx largely taken from updateNodeEvery3Blocks for css animation in storybook. User can define clicks for different data cells.

New dependencies? What are they used for?

CDN Link for simple-line-icons

Relevant screenshots?

screen shot 2019-01-31 at 5 25 14 pm

wolovim and others added 30 commits December 3, 2018 15:40
* Standardizes Component imports and runs prettier

* Cleans up Button

* Cleans up Checkbox

* Cleans up ValidatedField

* Cleans up EthAddress and starts on NodeInfoDot
* fix: clean up identicon

* fix: adds empty state to Identicon

* fix: updates README with new Identicon API
* Add prop structure
* Add story states
ToDo:
* Styling
* fix: adds displayNames, other standardizations

* Introduces PR template
* fix - Identicon displayName

* fix: adds anonymous state to Identicon
* fix: WIP linting

* fix: NewBlock lints

* fix: refactors TxParties
@wolovim wolovim changed the title Transaction row feat: TransactionRow Feb 5, 2019
Copy link
Member

@wolovim wolovim left a comment

Choose a reason for hiding this comment

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

Thanks for another PR! Few notes below:

@evertonfraga evertonfraga requested a review from wolovim February 25, 2019 19:15
@evertonfraga evertonfraga changed the base branch from master to dev March 17, 2019 14:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants