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

Refactor LocoTerm #23

Closed
synthdnb opened this issue Jul 30, 2014 · 2 comments
Closed

Refactor LocoTerm #23

synthdnb opened this issue Jul 30, 2014 · 2 comments

Comments

@synthdnb
Copy link
Contributor

#22

  1. TermApp::View
    • Every logical menu now owns its class, which inherits TermApp::View
    • Missing methods are delegated to TermApp::Application
    • These classes have to implement process method for displaying in main loop.
    • process method can have arguments. Possibly block in later?
    • process method have to return next menu's symbol, and arguments for its process method
    • Instance variables are preserved between navigation
  2. TermApp::Terminal
    • Previous LocoTerm class.
@khwon khwon mentioned this issue Aug 1, 2014
Closed
@yous
Copy link
Contributor

yous commented Aug 2, 2014

TermApp::View is now renamed to TermApp::Processor by f4dc837.

@khwon
Copy link
Owner

khwon commented Aug 24, 2014

@synthdnb ok for closing this issue?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@synthdnb @yous @khwon