-
Notifications
You must be signed in to change notification settings - Fork 23
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
Refactor The File Store And GUI #143
base: develop
Are you sure you want to change the base?
Conversation
Absolutely agree. I will look into this. |
I think we should first try to ge the app to compile. So it would be easier for development |
Why dont we use glade to construct the core widgets ? |
The rewritten app is currently compiling into |
I prefer not to add glade as a dependency. If I was building the app using |
Purpose
This is a major refactor that started with my dissatisfaction with the close coupling of the GUI with the actual list store and file saving interface.
My intent is to separate the "backend" concerns from the "frontend" concerns by encapsulating the "backend" code in
lib/
and the "frontend" code insrc/
. Once the two concerns are separated, it should be easier in the future to apply features without needing to intimately know how every piece of the app works. Refactors should be easier with an agreed-upon backend interface, and (hopefully) online syncing with services like libecal.With this rewrite, I am switching to a
ListBox
for the GUI to allow for more customization with each task.Before Merging!
In order to reach feature parity with the current GUI, the new app GUI needs to be able to do the following things:
<Esc>
to quit<Ctrl> + q
to quit<Ctrl> + w
to quit<Ctrl> + z
to undo<Ctrl> + y
to redo<Ctrl> + <Shift> + z
to redo