Skip to content
This repository has been archived by the owner on Mar 4, 2020. It is now read-only.

refactor(Dropdown): rename DropdownLabel to DropdownSelectedItem #725

Merged
merged 11 commits into from
Jan 15, 2019

Conversation

layershifter
Copy link
Member

@layershifter layershifter commented Jan 15, 2019

Refs #560.

BREAKING

  • adds static for displayName and className
  • use correct type in handleIconOverrides
  • extracts DropdownSelectedItem styles to a separate file

Before

dropdownStyles.ts

{
  label: // label styles
}

After

dropdownSelectedItem.ts

{
  root: // selected item styles
}


static create: Function

className = 'ui-dropdown__label'
static className = 'ui-dropdown__label'
Copy link
Contributor

Choose a reason for hiding this comment

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

let's rename the className as well

Copy link
Member Author

Choose a reason for hiding this comment

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

👍

@@ -17,6 +17,9 @@ This project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.htm

## [Unreleased]

### Fixes
- Rename `DropdownLabel` to `DropdownSelectedItem` and extract styles @layershifter ([#725](https://github.com/stardust-ui/react/pull/725))
Copy link
Contributor

Choose a reason for hiding this comment

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

Please before merging move this to BREAKING CHANGES.

Copy link
Member Author

Choose a reason for hiding this comment

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

Yep, also added update notes 👍

@layershifter layershifter merged commit 476aa0c into master Jan 15, 2019
@layershifter layershifter deleted the refactor/dropdown-selectem-item branch January 29, 2019 08:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants