Various docstring fixes, removing unused variables

[darrenburns]

I found a bunch of issues with docstrings/unused variables during my time on #3180.
I gathered all of the diffs into this PR to keep #3180 focused.

[codecov-commenter]

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (aabfd16) 98.30% compared to head (cb211d5) 98.28%.
Report is 26 commits behind head on master.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #3191      +/-   ##
==========================================
- Coverage   98.30%   98.28%   -0.03%     
==========================================
  Files          74       74              
  Lines        8038     8043       +5     
==========================================
+ Hits         7902     7905       +3     
- Misses        136      138       +2     

Flag	Coverage Δ
unittests	98.28% <92.85%> (-0.03%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files	Coverage Δ
rich/align.py	100.00% <ø> (ø)
rich/console.py	98.04% <100.00%> (+<0.01%)	⬆️
rich/containers.py	100.00% <ø> (ø)
rich/markdown.py	99.45% <100.00%> (ø)
rich/markup.py	100.00% <ø> (ø)
rich/progress.py	92.92% <ø> (ø)
rich/syntax.py	98.19% <100.00%> (-0.29%)	⬇️
rich/text.py	99.00% <66.66%> (-0.17%)	⬇️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.