Open
Conversation
2802ea2 to
5e1ad7b
Compare
| <Link className="nav__link" href="/admin/site-compliance"> | ||
| Site Compliance | ||
| </Link> | ||
| </NavGroup> |
Contributor
There was a problem hiding this comment.
Suggested change
| </NavGroup> | |
| </NavGroup> | |
| <div className="nav__controls"> | |
| <Link className="nav__log-out" href="/admin/logout"> | |
| <svg className="icon icon--logout" fill="none" height="20" viewBox="0 0 20 20" width="20" xmlns="http://www.w3.org/2000/svg"> | |
| <path className="stroke" d="M12 16H14.6667C15.0203 16 15.3594 15.8595 15.6095 15.6095C15.8595 15.3594 16 15.0203 16 14.6667V5.33333C16 4.97971 15.8595 4.64057 15.6095 4.39052C15.3594 4.14048 15.0203 4 14.6667 4H12M7.33333 13.3333L4 10M4 10L7.33333 6.66667M4 10H12" stroke-linecap="square"></path> | |
| </svg> | |
| </Link> | |
| </div> |
There isn't a Logout in the Figma but do we want to add that?
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #266
Based on the mockup... with some liberties taken.
Changes proposed in this pull request:
Security considerations
None