Skip to content

Improve badge for click tracking and links#268

Merged
tcbegley merged 1 commit intodbc-team:masterfrom
miqh:feature/improve-badge
Oct 24, 2019
Merged

Improve badge for click tracking and links#268
tcbegley merged 1 commit intodbc-team:masterfrom
miqh:feature/improve-badge

Conversation

@miqh
Copy link
Contributor

@miqh miqh commented Oct 23, 2019

Hello,

These changes aim to address the improvements suggested in #173.

I'm aware n_clicks_timestamp wasn't requested, but I figure I'd include it for consistency since most other components supporting n_clicks also has the former.

I transplanted the documentation for the new prop types from existing component implementations.

I've also introduced (couldn't find an existing one) a test file for the badge component that covers the new behaviour introduced.

Copy link
Collaborator

@tcbegley tcbegley left a comment

Choose a reason for hiding this comment

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

Hey @miqh

Thanks so much for your contribution! Particularly for adding tests. I've somewhat neglected these to date as I've invested most of my time into improving documentation instead 😳 I definitely want to improve the state of the testing now that I'm mostly done with the docs for now.

I've made a couple of minor inline comments, really just for making this component consistent with other components that do similar things, specifically Button and DropdownMenuItem, and which I think helps readability a bit. If you are able to incorporate those I'd be happy to merge and include in the next release.

Thanks again for taking the time to contribute!

Extends the badge component to offer additional properties for tracking
click interactions. Links are also rendered appropriately.

Addresses #173
Copy link
Collaborator

@tcbegley tcbegley left a comment

Choose a reason for hiding this comment

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

This is great, thanks again for your contribution!

@tcbegley tcbegley merged commit cdf342f into dbc-team:master Oct 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants