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.

Sign up for GitHub

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

Jump to bottom

Create interactive command for curses interface (for discussion purposes) #66

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Create interactive command for curses interface (for discussion purposes) #66

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
a2b1aac
Create interactive command for curses interface
Jan 3, 2017
01d7556
squashme: minor cleanups
Feb 18, 2017
c5524a3
Mass renaming
untitaker Feb 18, 2017
ed4dc01
Fix up docstring types
untitaker Feb 18, 2017
94f991e
Remove superfluous exit
untitaker Feb 18, 2017
123461e
More code cleanup
untitaker Feb 18, 2017
0a5d438
Replace list with tuple
untitaker Feb 18, 2017
271bd43
Replace leftover signatures
untitaker Feb 18, 2017
a04aa5a
Fixup
untitaker Feb 18, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
9 changes: 5 additions & 4 deletions todoman/interactive.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -530,7 +530,8 @@ def __init__(self, databases, formatter):
Create a Main instance based on the Database objects that
the regular Todoman cli module passes.

(Main, [Database], TodoFormatter) -> None
:type databases: list[Database]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We still need to change this, because with the current model it doesn't make sense to pass a list of Databases.

:type formatter: TodoFormatter
'''
self.databases = databases
# self.databases.sort(key = lambda db: db.name)
Expand All @@ -551,7 +552,7 @@ def unhandled_input(self, key):
'''
Handles all the key presses that are application-wide.

(Main, str) -> None
:type key: str
'''
if key in ('q', 'Q'):
raise urwid.ExitMainLoop()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When editing an entry, if a checkbox if focus and I press q, it immediately exists, without any confirmation, etc.

Expand All @@ -561,7 +562,7 @@ def _open_page(self, page_to_open):
Open a page over the existing stack of pages. page_to_open is the
Page object to display.

(Main, Page) -> None
:type page_to_open: Page
'''
self.loop.widget = urwid.Overlay(
page_to_open,
Expand All @@ -578,7 +579,7 @@ def close_page(self, page, **kwargs):

Usually called from the page to be closed.

(Main, Page, **kwargs) -> None
:type page: Page
'''
if self.pageCounter <= 1:
raise urwid.ExitMainLoop()
Expand Down