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
Add pull request manually merge instruction #13840
Conversation
If tea would work properly - we could also mention it ... |
Codecov Report
@@ Coverage Diff @@
## master #13840 +/- ##
==========================================
+ Coverage 42.11% 42.20% +0.08%
==========================================
Files 710 710
Lines 77227 77227
==========================================
+ Hits 32527 32596 +69
+ Misses 39350 39258 -92
- Partials 5350 5373 +23
Continue to review full report at Codecov.
|
62c0354
to
e060839
Compare
$('.show-instruction').on('click', () => { | ||
$('.instruct').toggle(); | ||
}); | ||
} |
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.
Could do this without any JS by using the <details>
and <summary>
elements.
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/details
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.
Yes, it is. But the arrow is not that suitable here.
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.
You mean the triangle? That can be hidden:
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/details#Customizing_the_disclosure_widget
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.
This is difficult to me and I will left this for other PRs.
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.
Yeah I guess it may be hard to have it only trigger from the link element.
3f35de8
to
85ed541
Compare
Can the 'You can view command line instructions' be moved a bit from the button? It seems cramped to me to have it so close. |
37c6770
to
3ec9353
Compare
@CirnoT @silverwind Done. |
Please pull in silverwind@d1fa043, I think it's a good compromise. |
Also pull in silverwind@35f7eed. It just removes that background on the search box. I think it looks better without it and we can use a deeper dark on those instructions. |
9519c39
to
56a9e68
Compare
@silverwind Done. |
default theme
arc-green theme