RedFantom

Results 73 comments of RedFantom

@reynhout Looking at them again I am not quite happy enough with them to share them as background images :sweat: .

Personally, I do not see any added benefit of having another complex `Calendar` widget, so I would indeed agree that either the current `Calendar` widget should be removed, or at...

@Dogeek I agree with you that separating data from widgets is a good design practice. I have not always done so in the past and it is easier to just...

a490457 contains changes to four lines of code that implement overrides for the state images of `CheckboxTreeview`, I think that is a good addition. However, I don't quite understand how...

### TODO List: - [x] Extend the tests to properly cover the functionality of the widget - [x] Test the widget on Windows - [x] Test the widget on Linux...

Thank you for these commits, @Dogeek . I have cherry-picked them, which yields an easier and cleaner solution than merging into this branch. In the future, just mention the branch...

Thank you for taking a look at it! When I cherry-picked your commit and tested the solution against the unit tests, the issue still occurred however. I believe this is...

> I checked your commit for the get_widget_options thing, and my implementation is way better (dict comprehensions are about 10X faster than dict mutations, and I actually check if the...

My apologies, I must have cherry-picked and then merged the changes wrongly, as you are correct: Your implementation does indeed work. However, I decided to profile the function performance, and...

The tests added in commit [`a69f41f`](https://github.com/TkinterEP/ttkwidgets/pull/48/commits/a69f41fe029e08d5e8379d8d7ebd8be6dd69160d) illustrate my concerns about the `move_widget` function, and why I thought more testing was needed. Due to the way that commands and bound functions...