Skip to content
This repository has been archived by the owner on Oct 29, 2019. It is now read-only.

Fix delegates section link. #237

Closed

Conversation

davidlehn
Copy link
Contributor

@davidlehn davidlehn commented Jul 9, 2019

Unsure how to handle this. The section number listed is old and incorrect. But also the entire delegates section is currently commented out. If this <li> here is also commented out then there is no need for a list so the surrounding text should be reworked. This issue may just be a reminder for someone to revisit and fix this text later.


Preview | Diff

Copy link
Member

@peacekeeper peacekeeper left a comment

Choose a reason for hiding this comment

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

Thanks, I agree this is a good reminder to figure out what to do about "delegates". Hardcoded section numbers should be avoided, so if we decide to keep the text about delegates, then this is a good fix.

@rhiaro rhiaro added editorial Editorial changes to the specification markup/infra Does not pertain to spec contents, but markup or supporting infrastructure (eg. ReSpec, CSS, JS) and removed editorial Editorial changes to the specification labels Jul 18, 2019
Copy link
Contributor

@msporny msporny left a comment

Choose a reason for hiding this comment

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

We should remove this text entirely as the delegate section is now gone.

@msporny
Copy link
Contributor

msporny commented Aug 8, 2019

Close this in preference for PR #249.

@msporny msporny closed this Aug 8, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
markup/infra Does not pertain to spec contents, but markup or supporting infrastructure (eg. ReSpec, CSS, JS)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants