bczsalba / pytermgui

Python TUI framework with mouse support, modular widget system, customizable and rapid terminal markup language and more!
https://ptg.bczsalba.com
MIT License
2.21k stars 54 forks source link

Add gitignore and do some code splitting #28

Closed developomp closed 2 years ago

developomp commented 2 years ago

b30b2d89b91dc4e417f2a617ed8f3bf6f0334cce: added .gitignore f094ee6dcf5e54e689e0690ec91b4081bee8a3f8: separated interactive widgets to separate files for readability

bczsalba commented 2 years ago

Hey there! I'm fine and probably happier than anything with these changes, but they aren't really linting proof at the moment.

One thing though: I'm not sure about the underscore in input_field.py. The PascalCase in the class name has been more about looks if anything, so inputfield.py would look and work better IMO. Feel free to dispute this part, as I'm not very attached to either.

Running make lint in the root of the project gives me these new errors: (there is an extra on the top that is present in upstream as well, so we will ignore it)

Screenshot 2022-01-29 at 8 42 31

Could you fix these? Since we don't have a contribution guide at the moment I'll just tell you here, for docstrings we use Google's styleguide.

Thank you!

developomp commented 2 years ago

Will follow up with a fix in a couple of hours after I go for a walk.

As for the filename, I intentionally chose to use an underscore, so some IDEs won't mark it as a typo. It can be imported without mentioning the filename once (like this: from pytermgui.widgets.interactive import InputField) so I think it's okay as it is.

bczsalba commented 2 years ago

As for the filename, I intentionally chose to use an underscore, so some IDEs won't mark it as a typo. It can be imported without mentioning the filename once (like this: from pytermgui.widgets.interactive import InputField) so I think it's okay as it is.

Fair point, I use a pretty raw Vim for my editing so that isn't something I run into.

developomp commented 2 years ago

Fixed all warnings introduced by the changes. Review please!

bczsalba commented 2 years ago

Looks good, thank you!