Skip to content
This repository was archived by the owner on Jan 27, 2025. It is now read-only.

Conversation

masticore252
Copy link
Contributor

This adds customization support to the action buttons

this can help with #19 until proper localization support is implemented

this adds customization support to the action buttons
Copy link
Contributor

@scaredcat scaredcat 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 taking the time to go through the open issues and helping out.

Please have a quick look at the comments and see if you can fix them up.

Copy link
Contributor

@scaredcat scaredcat left a comment

Choose a reason for hiding this comment

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

great work

Comment on lines +168 to +172
if (actions.length) {
actions = (<ul className="item-actions">{actions}</ul>)
} else {
actions = (<div className="item-actions">&nbsp;</div>)
}
Copy link
Contributor

Choose a reason for hiding this comment

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

this could be moved right before returning actions which means you could delete the other identical block.

@scaredcat scaredcat merged commit 62ecc52 into uptick:master Nov 28, 2019
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.

2 participants