-
-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
Added an illustration to the compass direction docs (issue 13664) #13788
Conversation
i forgot to add "ignore" to the documentation (bc rust unit tests documentation like a chad) |
Delightful :D |
FYI only "fixes" and "closes" work as magic keywords: I edited your post from "solves" to link the issue properly :) |
Wait, why did you close this? This was good stuff and I was about to merge it :) |
@alice-i-cecile what no i didnt close it on purpose |
Yep, it looks like it closed this out automatically when the branch was deleted :) I've reopened this now, merging! |
i based the design on @mgi388 in the discussion about the issue.
i added the illustration in such a way that it shows up when you hover your mouse over the type, i hope this is what was meant by the issue
no unit tests were added bc obviously
Fixes #13664