Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixup Intro Separation for Ember Times Posts #284

Merged
merged 3 commits into from
Sep 23, 2019

Conversation

jayjayjpg
Copy link
Contributor

Closes #274

What it does

Related Issue(s)

Sources

before after
Screenshot 2019-09-24 at 00 24 34 Screenshot 2019-09-24 at 00 24 19

@jayjayjpg jayjayjpg changed the title WIP: Fixup Intro Separation for Ember Times Posts Fixup Intro Separation for Ember Times Posts Sep 23, 2019
@jayjayjpg jayjayjpg requested review from amyrlam and a team September 23, 2019 22:33
Copy link
Member

@amyrlam amyrlam left a comment

Choose a reason for hiding this comment

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

👍 Looks good, let's add it to https://github.com/ember-learn/ember-blog/blob/master/source/embertimes-template.md too (could be done separate)

I can address the markdown-lint things separately. Maybe we should audit the rules as a group and just move forward with the ones that are important for a11y if it's too interruptive

@jayjayjpg jayjayjpg merged commit 72412a3 into ember-learn:master Sep 23, 2019
jayjayjpg added a commit to jayjayjpg/ember-blog that referenced this pull request Sep 23, 2019
@amyrlam
Copy link
Member

amyrlam commented Sep 23, 2019

cc @mixonic this will solve your concern with the weird cutoff on the blog, thanks @jessica-jordan / @chancancode

@jayjayjpg
Copy link
Contributor Author

@amyrlam Good point, I just re-added the linting and the separator over here in these PRs: #285 & #286

@jayjayjpg jayjayjpg deleted the readmore branch September 23, 2019 22:47
jayjayjpg added a commit that referenced this pull request Sep 23, 2019
ci(markdown): re-enable markdown linting (see #284)
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.

Investigate how the preview cutoff point is generated
2 participants