<div dir="auto"><div>Hi Mike,<div dir="auto"><br></div><div dir="auto">It's been a while and I'm not clear what, if anything, is happening with gitlab. Is there any way to proceed despite this?</div><div dir="auto"><br></div><div dir="auto">Cheers</div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 9 Jun 2020, 00:20 Dave Chamberlin-Kidd, <<a href="mailto:dave@flamangoes.co.uk">dave@flamangoes.co.uk</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="auto"><div>Cheers Mike. No problem.<br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 8 Jun 2020, 08:54 Mike Gabriel, <<a href="mailto:mike.gabriel@das-netzwerkteam.de" target="_blank" rel="noreferrer">mike.gabriel@das-netzwerkteam.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Dave,<br>
<br>
On  So 07 Jun 2020 17:49:21 CEST, Dave Chamberlin-Kidd wrote:<br>
<br>
> Package: x2goclient<br>
> Version: 4.1.2.3<br>
> Severity: wishlist<br>
> Tag: patch<br>
><br>
> There are a number of minor issues, oddities and limitations with the<br>
> current x2goclient gui which would be good to fix, resolve and improve on.<br>
> Like all projects however the core team have more important issues to<br>
> resolve. The code in its current state is difficult to understand and amend<br>
> and as it has no tests it is risky for developers to make functional<br>
> changes unless they spend a lot of time getting to know the code and doing<br>
> extensive manual testing.<br>
><br>
> Attached are 10 small refactorings which start extracting out some of the<br>
> code into smaller functions. As the code is separated out it becomes easier<br>
> to understand and change.<br>
><br>
> There are no functional changes in these patches, nor do they attempt or<br>
> claim to fix any known issues. On their own the benefit of these patches is<br>
> limited however the hope is that over time I will be able to submit more<br>
> refactorings as I learn more and then start fixing issues and improving the<br>
> UI. Extracting out functions is just one of many techniques that can my experience it's a good one to start<br>
> with. It should also be noted that sometimes code is extracted "as is" even<br>
> when it is clear further refactoring can be done to improve things, this is<br>
> by design in order to keep risk and size of patches to a minimum.<br>
><br>
> Cheers<br>
> Dave<br>
<br>
Thanks for your patches. As the GitLab server for X2Go is in reach,  <br>
I'd like to postpone patch review until its there. I'd be happy if you  <br>
could file a merge request then and we dive into your proposals.<br>
<br>
Thanks+Greets,<br>
Mike<br>
-- <br>
<br>
DAS-NETZWERKTEAM<br>
c\o Technik- und Ökologiezentrum Eckernförde<br>
Mike Gabriel, Marienthaler Str. 17, 24340 Eckernförde<br>
mobile: +49 (1520) 1976 148<br>
landline: +49 (4351) 850 8940<br>
<br>
GnuPG Fingerprint: 9BFB AEE8 6C0A A5FF BF22  0782 9AF4 6B30 2577 1B31<br>
mail: <a href="mailto:mike.gabriel@das-netzwerkteam.de" rel="noreferrer noreferrer" target="_blank">mike.gabriel@das-netzwerkteam.de</a>, <a href="http://das-netzwerkteam.de" rel="noreferrer noreferrer noreferrer" target="_blank">http://das-netzwerkteam.de</a><br>
<br>
</blockquote></div></div></div>
</blockquote></div></div></div>