<div dir="ltr">Hi,<div><br></div><div>I'd like to have a go at bug fixing, tidying up and adding features to the x2goClient and to start with I have spent a bit of time looking at the sessionbutton code to try to get to know it a bit. Having done so I think it would be useful to submit some refactoring patches. Once I've done this, I would then like to take the same approach with some of the other classes (onmainwindow being the next obvious candidate)</div><div><br></div><div>This is the sort of thing that I can imagine isn't high priority for the core devs but it will, I think, make future improvements much easier to code, review and test.</div><div><br></div><div>I don't want to stand on any toes so thought I would ask the following before submitting anything...</div><div><br></div><div>1: Is the above proposal acceptable?</div><div>2: If so is there a particular approach you would like me to take? (my plan is to start by extracting code into smaller functions in a series of small commits)</div><div>3: Do the dev team have any other thoughts/suggestions as to things they would like to see as part of a refactoring that I might be able to pick up</div><div><br></div><div>Just to be clear, to start with my intention is to make the code easier to understand and follow without changing any functionality. <br></div><div><br></div><div>Cheers</div><div>Dave </div></div>