ember-learn / ember-learn/ember-blog
Don't use links for headings in Ember Times
- Dominant language
- JavaScript
- Stars
- 36
- Forks
- 75
- Avg merge
- 3h 29m
- Merged PRs (30d)
- 2
Description
This is an issue that currently affects the production blog, so it could be done after the release :+1:
If you look at any of the Ember Times posts you will see that all of the headers are also links. This is not the best experience from an accessibility perspective and we need to change the structure of these headers
Right now the HTML is something like this:
```html
Using ember-concurrency with TypeScript 🤝
```
and we should really change it to something more like:
```html
Using ember-concurrency with TypeScript 🤝
External Link (->)
```
Once we figure out how this should work in the Markdown then we should write some sort of script to update all of the posts that have this problem
Example:
- https://blog.emberjs.com/2020/07/17/the-ember-times-issue-156.html
- https://deploy-preview-7--ember-blog-legacy.netlify.app/the-ember-times-issue-156
Contributor guide
Research direction
Start by inspecting the Markdown and rendered HTML for the linked Ember Times issue 156 examples. Determine how the heading and external link should be represented in Markdown, then identify all affected posts for a migration. Done means headings are no longer links and separate external links render consistently across existing posts.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- accessibility, content
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100