search

stuartlangridge / magnus

A very simple screen magnifier for Ubuntu
MIT License
38 stars 13 forks source link

Added refresh method to Main class #22

Open 1-0 opened 2 years ago

1-0 commented 2 years ago

Added refresh method to Main class callable for repaint region regardless of changing or saving the cursor position

stuartlangridge commented 2 years ago

um, ok? I mean, I can't see a reason to not do this, but equally I can't see a reason to do it either. Are you planning for there to be some other way to force a refresh, or something to be added which will want to call that method?

1-0 commented 2 years ago

This method is called 2 times (so I use DRY pattern) - cursor position changed and at timer cursor position saved. I do not change options used - all parameters stay as is.

stuartlangridge commented 2 years ago

Ah, I understand; it makes the code cleaner! I'll look at merging this as soon as I can.