Skip to content
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

Make WTP more modular #107

Open
eniraa opened this issue Aug 14, 2021 · 1 comment
Open

Make WTP more modular #107

eniraa opened this issue Aug 14, 2021 · 1 comment
Labels
enhancement New feature or request

Comments

@eniraa
Copy link
Contributor

eniraa commented Aug 14, 2021

As it stands currently, WTP seems to have code that is rather specific to StackOverflow. For instance, there is code that is specifically for navigating questions in display.py: https://github.com/what-the-python/wtpython/blob/022785fa710f3bf69becd0b2a429e4926f37ba2d/wtpython/display.py#L147-L161

Perhaps this code would be better located in some file which is specific to SO like stackoverflow.py.

I think it might be a good idea to have the application be more modular (i.e. allow someone to replace SO with RTD without too much work). This will also help in implementing features like #106.

@eniraa eniraa added the enhancement New feature or request label Aug 14, 2021
@briancohan
Copy link
Contributor

This is one of my main goals. I want to make it so the display class manages as little as possible and has the stackoverflow object track which question it's displaying as well as the displays for the sidebar, main window, and even the --no-display option. Then I hope to incorporate a search through the Reddit API and other sources of information.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants