-
Notifications
You must be signed in to change notification settings - Fork 4.8k
Add anchors for easier subheading linking #292
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
Conversation
@@ -30,7 +32,7 @@ Multiple rules may be delimited using either a "pipe" character, or as separate | |||
array('name' => array('required', 'min:5')) | |||
); | |||
|
|||
Once a `Validator` instance has been created, the `fails` (or `passes`) method may be used to perform the validation. | |||
Once a `Validator` instance has been created, the `fails` (or `passes`) method may be used to perform the validation. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is this massive space?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's where my positive attitude used to be.
Other than the space, which will only damage Taylor's OCD, it looks great! Thanks :) |
… the docs easier.
Haha nice Dayle. I fixed the spacing issue introduced in the commit. |
I like the general idea, but shouldn't this be done in the tool that parses the MarkDown files and displays them? |
Hi @piotrowski , could you target the 4.1 branch for this please? Thanks! :) |
Sorry, wat? Not my PR, so I can't do anything, right? |
This was my PR, but I'm not at a place where I can update and resubmit this. If someone else wants to tackle it, that'd be awesome. |
Sorry @piotrowski , I tagged the wrong username :) |
I thought it would be useful to add anchors for sub-headings so people could link to sub-sections of the docs more precisely.