Critique This! A Simple Qt Dock Widget which enables the whole widget to be 'folded' to use less screen space
Critique This! A Simple Qt Dock Widget which enables the whole widget to be 'folded' to use less screen space
I'm working on a project that needs lots of toolbars on screen at once, even though not all of them will be used at the same time. So, I'm modelling this 'foldable' dock widget after what I remember Photoshop panels used to be like.
It's a work in progress, but would like to hear constructive suggestions.
https://blocks.programming.dev/0101100101/42c5d67f86c049baa3500aa38e439f8a
Easily above average code for Python. I'm going to pick on one method:
First, Python does have ternary expressions so you can
Second, what does this code do?
Kind of surprising that it sets the icon to
icon_dock
right? There are two easy fixes:*, is_floating: bool
so you have to name the parameter when you call it._update_float_icon()
or something.Also use Black or Ruff to auto-format your code (it's pretty well formatted already but those will still improve it and for zero effort).
Thanks for the compliment! Python isn't my first language and it's difficult to be able to switch style from one language to another!
I always find it difficult to choose when to use ternary statements. Sometimes, for something quick and simple, I will, otherwise I'll be explicit. This is more of a readability issue than anything else. And I find the ternary statements quite verbose compared to other languages by using the words if/else rather than shorthand symbols.
You're absolutely right about the set_float_icon and corresponding method. Coding's an iterative process and that's a byproduct. I think
set_float_icon()
along with complimentary methods likeset_docked_icon(), set_minimize_icon(), set_restore_icon()
etc may be easier to use / remember wtf it does in six months time!Thanks for the black / ruff suggestion. I've never heard fo them, but I'm about to go look for them.
Recommend against the ternary expression.
coverage
might not detect the two code blocks. Would also not be able to apply# pragma: no cover
comment to a code block that isn't important enough to justify testing it. Often usedo nothing
code blocks.self.float_button.setIcon(self.icon_dock if is_floating else self.icon_float)
lets say while testing something goes wrong and trying to debug what happened.Will not see the value that gets past into
self.float_button.setIcon
I like your idea of having the param be keyword only. Makes it more readable.
If you branch coverage tool can't handle branches on the same line I would suggest you use a different one! Does it handle
if foo or bar
?Uhm, yes you will? Just step into the function.