-
Couldn't load subscription status.
- Fork 97
Expose deeplinks. #275
Expose deeplinks. #275
Conversation
This is a nice change, @Lukas-Stuehrk!
Before we merge this in, I'd like to give this a quick once-over to get the design a bit closer to what I had in mind with #110 (comment).
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.
Actually, scratch that — this looks fantastic as-is 🤩
For whatever reason, the screenshots initially looked off to me. But seeing them in a live render, I wouldn't change a thing. Great work, @Lukas-Stuehrk!
Adding a changelog entry now and merging after that.
Thanks! 🙇
Initially, I wanted to build the version as described in #110 and had the current solution as intermediate step. But then I liked the visuals so much that I decided to keep it and go with this solution. I think it looks great and matches the visual language of the index page.
This change adds links to the name of all members, so it's easy to copy the direct link. This implements #110.
I chose a different approach than adding a chain symbol when hovering the name. The entire name is a link. This leads to some visual changes in the generated documentation. I can change this behavior to the behavior described in the original issue if wanted.
Before the changes:
Screenshot 2021年05月08日 at 00 12 21
After the changes when the entire name becomes the link to the anchor:
Screenshot 2021年05月08日 at 00 12 59