More tweaks - #19
More tweaks#19
Conversation
| Normally the unit tests skip over any permanent changes to your system, but if you | ||
| want to enable destructive tests (which make changes without any confirmation) set | ||
| the environment variable `DESTRUCTIVE_TEST=T`. **Use with caution!** Normally you | ||
| the environment variable `DESTRUCTIVE_TESTS=T`. **Use with caution!** Normally you |
| Normally the unit tests skip over any permanent changes to your system, but if you | ||
| want to enable destructive tests (which make changes without any confirmation) set | ||
| the environment variable `DESTRUCTIVE_TEST=T`. **Use with caution!** Normally you | ||
| the environment variable `DESTRUCTIVE_TESTS=T`. **Use with caution!** Normally you |
|
Thank you for another round of good changes. That includes helpful improvements to my new commit 6a3c2ec . My only concerns were two things about rendering code blocks. code blocks with multiple commandsBefore the change, no line is prefixed with 
After the change, only first line is prefixed with 
.bat lexerbefore change 
With change, it flags % as unknown token. 
|
|
I'll try to see if the first issue is fixable in an easy way, otherwise I'll drop that patch. As for the the |
25cf1c5 to
6f810b9
Compare
|
I dropped the The |
- Wrap main.js in an IIFE with strict mode - Only add copy buttons when the Clipboard API is available - Use a Range to get console commands instead of walking text nodes - Show "Failed" on copy errors and reset the label timer on repeated clicks - Move `position: relative`` to CSS and show the button on keyboard focus
I am open to keeping the bat change |
|
Something is weird with Cloudflare... I keep getting |
You're right about the |
|
You still seem to be loading zaraz, whatever that is. If you don't really need it, better drop that too. |
|
True. I disabled Zaraz |
2 participants
Non-whitespace diff: https://github.com/bleachbit/docs/pull/19/changes?w=1
Preview: https://super-salamander-00b8ec.netlify.app