-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
[RFC][ENH] OWWidget: Input/output summary #2556
[RFC][ENH] OWWidget: Input/output summary #2556
Conversation
f4220f2
to
3231d90
Compare
Codecov Report
@@ Coverage Diff @@
## master #2556 +/- ##
==========================================
+ Coverage 83.17% 83.23% +0.05%
==========================================
Files 362 362
Lines 63832 64088 +256
==========================================
+ Hits 53094 53344 +250
- Misses 10738 10744 +6 |
831a658
to
a8d85ae
Compare
e2caffb
to
fee6450
Compare
e5b07a0
to
3777d58
Compare
3777d58
to
d1e655f
Compare
12b2d6b
to
f79fd99
Compare
@markotoplak, @BlazZupan, @janezd, this has been sitting here for too long. If rebased, could it be included in the next release? |
f79fd99
to
6050809
Compare
I/O Summaries | ||
------------- | ||
|
||
.. versionadded:: 3.7 |
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.
This line will probably have to updated. :)
332b65e
to
e1bf6c3
Compare
This didn't make it into the 3.15.0 release, so where are we at? |
c4adb6d
to
3024ad2
Compare
@BlazZupan Can we merge this before 3.16 or at least decide what to do with it? @lanzagar |
This looks really cool, @ales-erjavec can you please resolve conflicts and then we merge. |
:alt: Inline status bar summary | ||
|
||
Predefined constants indicating no input/output are available as | ||
``self.info.NoInput`` and ``self.info.NoOutput`` respecively |
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.
respectively
Once merged, this functionality would also require a note in a widget development documentation. @ales-erjavec, can you please add an entry there as well. |
c63b07a
to
60b74f7
Compare
Allow setting custom (text) style sheet
Note: uses implicit data sharing.
Workaround QTBUG-60018 and remove style provided frames around the status bar items.
60b74f7
to
107ac89
Compare
Issue
Provide a consistent widget input/output descriptions to the user.
Description of changes
Works on top of #2514
Add an input and output message areas in the widget's status bar.
The widget authors set the summaries using
for basic inline short descriptions. A more detailed (including rich text) summaries are also possible:
Ignore stylistic deficiencies in the last commit. Its flailing around to find some common 'description language'.Includes