Skip to content

Conversation

LastKing
Copy link

I am English is poor.
Some modifications
fixed icon and menu

this is view.
image
tanks!!

@willklein
Copy link
Owner

Thank you for the pull request! Let me take a look and test in a few browsers. I might make some changes to what you submitted, but I'll try to merge this if it works, and make other changes after merging. 😄

@willklein willklein self-requested a review April 17, 2019 17:25
markdownTarget.innerHTML += link;
}
var link = '<span class="github-markdown-contents select-menu js-menu-container js-select-menu"><span class="github-markdown-contents-btn js-select-menu js-menu-target btn btn-sm tooltipped-s' + '" role="button" aria-label="Show Table of Contents">' + threeBarSvg + '</span>' +
'<div class="select-menu-modal-holder github-markdown-contents-modal-holder js-menu-content js-navigation-container" style="width:300px;left:-100px;top:20px;">' +
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for making this multiple lines!

};

var insertContents = function(contents) {
var insertContents = function (contents) {
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After this pull request I'll setup Prettier so this auto-formats consistently going forward.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok,My code is idea format .
It's different from your way.

@grantwinney grantwinney mentioned this pull request Mar 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants