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
Initial mode picker #45
Conversation
@MatthewBeaudouinLafon and I talked about some next steps: making |
Rebased off of new master |
I refactored some more, 132004a has the details. |
Ready for a look-through, those last two are just formatting changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. I added a comment
Lots of changes here: - mode functions are now called only with a "reason" and the state - reasons are explained more on Line 18 of fe_modes.c - the only function that was using canvas_win or status_win was mode_brush, which was using it as a hacky way of clearing the status window - declare new functions with mode_function_t in fe_modes.h - mode_master renamed to master_handler and is now less of a mode. - run continuously in main loop - grabs keys with wgetch - handles global keypresses and UI updates - mode calling and switching is a little more complex but now factored out into call_mode and switch_mode
Added some changes from comments, this should be good to go |
I made a rough mode picker interface to use brush mode with, and refactored some of the mode code in the process.
Closes #15.
TODO:
add mode-level configssee Add mode-level configs #52