Skip to content
This repository was archived by the owner on Dec 4, 2017. It is now read-only.

docs(style-guide): make each style clickable for easier referencing #1339

Closed
wants to merge 1 commit into from

Conversation

olore
Copy link
Contributor

@olore olore commented May 9, 2016

This will make it easier to capture the link to a specific style so that it can be shared.

@KyleBastien
Copy link

KyleBastien commented May 16, 2016

Just noticed this pull request, there is more discussion on this topic going on in #1397 and #1400, for those interested.

@olore
Copy link
Contributor Author

olore commented May 17, 2016

Thanks @KyleMcNutt. Our commits are very similar. One thing mine does different (which I prefer) is that I moved the anchor to the name of the rule (i.e. "Rule of One") That way when the page loads, it shows the rule name at the top, then the rule number below it. I find the name very useful.

@KyleBastien
Copy link

@olore I can see that perspective, I was going for the minimal change just to make it more obvious what can be linked to. It looks like from discussion in the other issues that an on-hover icon seems to be the preferred approach, but I'll leave that to the Angular teams designers to decide.

@johnpapa
Copy link
Contributor

johnpapa commented Jun 1, 2016

@wardbell I am OK with this.. Are you?

@wardbell
Copy link
Contributor

Thanks. Merged as ce05f9b

@wardbell wardbell closed this Jun 13, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants