Skip to content

Stanleyy/fillin#4351

Merged
stanleyymsft merged 27 commits intomicrosoft:masterfrom
stanleyymsft:stanleyy/fillin
Mar 23, 2018
Merged

Stanleyy/fillin#4351
stanleyymsft merged 27 commits intomicrosoft:masterfrom
stanleyymsft:stanleyy/fillin

Conversation

@stanleyymsft
Copy link
Copy Markdown
Collaborator

Pull request checklist

  • Addresses an existing issue: Fixes #0000
  • Include a change request file using $ npm run change

Description of changes

ComboBox: Add some code comments. Minor bug fixes to the example page.

Focus areas to test

Just minor changes in code comment and example page.

"changes": [
{
"packageName": "office-ui-fabric-react",
"comment": "Add multiSelect capability for ComboBox",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I don't think this changefile is needed since the one below seems to be the correct file. Can you remove this one?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

You are right. Will delete.

Copy link
Copy Markdown
Contributor

@jspurlin jspurlin left a comment

Choose a reason for hiding this comment

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

Remove the extra/outdated changefile

@stanleyymsft stanleyymsft merged commit 7e3911a into microsoft:master Mar 23, 2018
@microsoft microsoft locked as resolved and limited conversation to collaborators Aug 31, 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