Skip to content

Fix: deep TextEnum was improperly resolved #609

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

Merged
merged 1 commit into from
Sep 22, 2023

Conversation

penguinolog
Copy link
Collaborator

  • Fix WHSettings.CLIP & WHSettings.FLOW
  • Export Sextant2x2Font and Sextant3x3Font
  • Add __class_getitem__ to FontRegistry for fonts access
  • Add related unittests

Fixes: #606

Checklist
  • I've ensured that similar functionality has not already been implemented
  • I've ensured that similar functionality has not earlier been proposed and declined
  • I've branched off the master or python-dual-support branch
  • I've merged fresh upstream into my branch recently
  • I've ran tox successfully in local environment
  • I've included docstrings and/or documentation and/or examples for my code (if this is a new feature)

* Fix `WHSettings.CLIP` & `WHSettings.FLOW`
* Export `Sextant2x2Font` and `Sextant3x3Font`
* Add `__class_getitem__` to `FontRegistry` for fonts access
* Add related unittests

Fixes: urwid#606
@coveralls
Copy link

Coverage Status

coverage: 73.531% (+0.03%) from 73.503% when pulling 1534d15 on penguinolog:regression_enums into cce5326 on urwid:master.

@penguinolog penguinolog merged commit 583e8a0 into urwid:master Sep 22, 2023
@penguinolog penguinolog deleted the regression_enums branch September 22, 2023 13:31
@ethanfurman
Copy link

This is the correct fix for the enum issue -- see this comment for details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"clip" no longer works for Padding width in 2.2.0
4 participants