rich icon indicating copy to clipboard operation
rich copied to clipboard

fix(syntax): include padding with background color override

Open TomJGooding opened this issue 1 year ago • 1 comments

Type of changes

  • [x] Bug fix
  • [ ] New feature
  • [ ] Documentation / docstrings
  • [ ] Tests
  • [ ] Other

Checklist

  • [x] I've run the latest black with default args on new code.
  • [x] I've updated CHANGELOG.md and CONTRIBUTORS.md where appropriate.
  • [x] I've added tests for new code.
  • [x] I accept that @willmcgugan may be pedantic in the code review.

Description

Fixes #3295. I've tested manually but I'm afraid not sure how to add a proper test for this.

Before:

image

After:

image

TomJGooding avatar Feb 29 '24 20:02 TomJGooding

I've now added a test based on the others, where I assume you just copy paste the result from print(repr(result)) and then check that? (If Will has actually written these by hand I'm seriously impressed!)

If that's not the proper way to test this, please let me know!

TomJGooding avatar Feb 29 '24 22:02 TomJGooding