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
Feature: Support FIXED and FLOW operations for Overlay
depends on options
#743
Conversation
…ptions * Add tests * Update documentation
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.
File will be re-generated from original vectors in another PR
@@ -304,17 +304,17 @@ new space. | |||
Overlay Widgets | |||
--------------- | |||
|
|||
The :class:`Overlay` widget is a box widget that contains two | |||
other box widgets. The bottom widget is rendered the full size of the Overlay |
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.
looks like totally outdated
Pull Request Test Coverage Report for Build 7505379911
💛 - Coveralls |
* * width amount and GIVEN width or min_width AND: | ||
* * * FLOW supported by the TOP widget AND PACK height type | ||
* * * BOX supported by the TOP widget AND height_amount and GIVEN height or min height | ||
""" |
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.
IMHO adding examples is overkill (too complex arguments sequences)
if Sizing.FIXED in top_sizing: | ||
sizing.add(Sizing.FIXED) | ||
else: | ||
warnings.warn( |
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.
negative cases with warnings and exceptions should be covered separately.
Provide information about owner widget
def __rich_repr__(inner_self) -> Iterator[tuple[str | None, typing.Any] | typing.Any]: | ||
yield from self.items() | ||
yield from inner_self.items() |
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.
Autocomplete fail
|
||
if typing.TYPE_CHECKING: | ||
from collections.abc import Iterator | ||
|
||
from typing_extensions import Literal | ||
|
||
|
||
class OverlayError(Exception): | ||
pass | ||
class OverlayError(WidgetError): |
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.
Proper exception class (need to do for others widgets too)
Checklist
master
orpython-dual-support
branchtox
successfully in local environment