Skip to content

Conversation

chloerice
Copy link
Member

@chloerice chloerice commented Nov 9, 2018

WHY are these changes introduced?

Closes #590

WHAT is this pull request doing?

Fixes misplacement of the Markdown horizontal rule so that the last example gets parsed properly.

Before After
2018-11-09 13 48 13 2018-11-09 13 51 08

How to 🎩

🖥 Local development instructions
🗒 General tophatting guidelines
📄 Changelog guidelines

  • git checkout fix-autocomplete-readme && yarn run build-consumer polaris-styleguide
  • The empty state example should parse properly and be listed as the last option in the Examples select input.

🎩 checklist

@ghost
Copy link

ghost commented Nov 9, 2018

👋 Thanks for opening your first pull request. A contributor should give feedback soon. If you haven’t already, please check out the contributing guidelines. You can also join #polaris on the Shopify Partners Slack.

@BPScott BPScott temporarily deployed to polaris-react-pr-592 November 9, 2018 21:40 Inactive
@chloerice chloerice changed the title fix section break placement [Autocomplete] Fix section break placement in README Nov 9, 2018
Copy link
Member

@BPScott BPScott left a comment

Choose a reason for hiding this comment

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

Not tested, but I trust ya

Copy link
Contributor

@amrocha amrocha left a comment

Choose a reason for hiding this comment

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

Tested and it looks good

@chloerice chloerice merged commit b26a9a6 into master Nov 9, 2018
@ghost
Copy link

ghost commented Nov 9, 2018

🎉 Thanks for your contribution to Polaris React!

@tmlayton tmlayton deleted the fix-autocomplete-readme branch November 9, 2018 23:09
@solonaarmstrong-zz
Copy link

Oh, I'm sorry, I thought I had included the horizontal divider. Thank you, @chloerice, that explains why it works for me now. ❤️

@AndrewMusgrave AndrewMusgrave temporarily deployed to production November 14, 2018 20:23 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Autocomplete] Last example is listed in related components section

5 participants