<div dir="ltr">Hi Joaquín!<div> good point!!</div><div> After looking at your code I realized there were other places that could be improved so I did it.</div><div> To avoid cluttering CodeProvider and to put everything in one place I used a method object to calculate and prompt for the categories based on your code.</div><div><br></div><div> Thank you again!</div><div> Cheers!</div><div> Hernan.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jul 16, 2021 at 4:16 AM Joaquín Singer <<a href="mailto:josinger@fi.uba.ar">josinger@fi.uba.ar</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Hi Hernan,<div><br></div><div>I'm glad you made those changes, as I mentioned in the first email, I was a bit worried about that problem.<br><br>Did you consider taking advantage of these changes to be able to use them in CodeProvider, which has a very similar method to the one we were modifying? (Used for when you want to change the category of a method)<br><br>In case of what i said interests you, I'm attaching a changeset that pushes "addCategoriesOf: to: separatingWith:" to CodeProvider and changes "categoryFromUserWithPrompt: for:" for it to use this new method, removing quite a bit of duplicate code. <br><br>The only thing that ends up changing is that for classes, this will show "instance creation" on top of everything just like in the debugger, I suppose that if for some reason you want to avoid that, there are several simple ways to change it.<br><br><div><span style="color:rgb(0,0,0);white-space:pre-wrap">Cheers</span>, </div><font color="#888888"><font color="#888888"><div><br></div><div>Joaquín.</div></font></font></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">El jue, 15 jul 2021 a las 10:54, Hernan Wilkinson (<<a href="mailto:hernan.wilkinson@10pines.com" target="_blank">hernan.wilkinson@10pines.com</a>>) escribió:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Hi Joaquín!<div> Thank you for the contribution! It is a great one. It makes selecting the method category simpler. As you said, sorting the categories that way is much much better.</div><div> I made some changes to make it more declarative and testable. I also added tests to the way categories are sorted. I hope you don't mind.</div><div> It is on github now!</div><div><br></div><div>Cheers!</div><div>Hernan.</div><div><br></div><div>PD: I'm back for just one day :-)</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jul 15, 2021 at 2:38 AM Gerald Klix via Cuis-dev <<a href="mailto:cuis-dev@lists.cuis.st" target="_blank">cuis-dev@lists.cuis.st</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Joaquín,<br>
<br>
See below.<br>
<br>
On 7/14/21 10:16 PM, Joaquín Singer wrote:<br>
> Hi gerald,<br>
> <br>
> I noticed this behavior but, if I didn't misunderstand, it doesn't allow<br>
> you to see the categories that are in the following pages. What this change<br>
> does is show the list in the same way that it is shown when you want to<br>
> change the category of a message, having in the top all the categories in<br>
> the current class, then in the superclass, and so on.<br>
Yep, the following page is AFIR just a submenu.<br>
> <br>
> I like showing the last assigned or created on top of everything, at first<br>
> what I wanted to do was something similar, but then I realized that doing<br>
> it in the same hierarchical way as the option to change the category, was<br>
> useful in almost all cases and more consistent with the other<br>
> functionality. But if you think it's better to show it this way, I can<br>
> modify it.<br>
Ok, I will try to use our changes for some time.<br>
> <br>
> Did you tested it?<br>
Nope! I did touch my computer for only some minutes and did<br>
not use Cuis at all. Me sorry.<br>
> <br>
> Cheers,<br>
> <br>
> Joaquín.<br>
> <br>
> El mar, 13 jul 2021 a las 4:12, Gerald Klix (<<a href="mailto:csh.00@klix.ch" target="_blank">csh.00@klix.ch</a>>) escribió:<br>
> <br>
>> Hi Joaquin,<br>
>><br>
>> I have not yet tested your change set(s), I will do so today.<br>
>><br>
>> But it should be pointed out, that you can type the first N letters<br>
>> when a menu is displayed, which disables all menu items<br>
>> that do not start with that prefix. Additionally you can<br>
>> use the up- and down-keys to select an item.<br>
>> In this way it is possible to select a category<br>
>> quickly using the keyboard.<br>
>><br>
>> I frequently miss the categories I used recently in the menu,<br>
>> therefore IHMO a useful change would be to offer the last 4 (or N)<br>
>> categories assigned or created.<br>
>> When I add "instance creation' methods, I often miss the<br>
>> "instance creation" category at the instance side of class.<br>
>> Maybe it would make sense to present the class side categories<br>
>> in the instance side categories menu (and vice versa).<br>
>><br>
>><br>
>> Best Regards,<br>
>><br>
>> Gerald<br>
>><br>
>><br>
>><br>
>> On 7/11/21 12:13 PM, Joaquín Singer via Cuis-dev wrote:<br>
>>> Hi everyone,<br>
>>><br>
>>> I'm using Cuis in a class at my university and these last few weeks I've<br>
>>> been making small changes to the image with the intention of solving some<br>
>>> things that personally bothered me a little / I thought could be<br>
>> improved.<br>
>>><br>
>>> If you don't mind, I'd like to share some of the changes with you so you<br>
>>> can discuss whether they are worth incorporating directly into the base<br>
>>> image.<br>
>>><br>
>>> For example, in the debugger, when the create button is clicked, after<br>
>>> selecting a class, a list of the categories present in the hierarchy<br>
>>> appears in alphabetical order, and it is usually easier for me to retype<br>
>> it<br>
>>> with the option "new..." than to go through the list using "more..." in<br>
>>> search of the one I want. The way this works seems much more practical to<br>
>>> me when you want to change the category of a message, where it appears,<br>
>> in<br>
>>> addition to alphabetically, hierarchically ordered.<br>
>>><br>
>>> I am attaching a changeset that modifies this.<br>
>>><br>
>>> On the other hand, since the allMethodCategoriesIntegratedThrough:<br>
>> message<br>
>>> gives you the categories ordered alphabetically, I kept using it but<br>
>>> sending as an argument the same class that receives it. Is it okay to do<br>
>> it<br>
>>> this way? Or it would be more declarative to get them with the<br>
>>> "organization categories" messages and then sort them, in the same way as<br>
>>> categoryFromUserWithPrompt: for :? Another option that might make sense<br>
>> is<br>
>>> to create a new message called allMethodCategoriesSortedAlphabetically<br>
>> that<br>
>>> does not have to receive any arguments and also use it in<br>
>>> categoryFromUserWithPrompt: for:<br>
>>><br>
>>> I have changesets with either of these last two options as well, so if<br>
>> you<br>
>>> decide to make the change and any of these seem more appropriate than the<br>
>>> implementation I'm submitting now, I can submit them.<br>
>>><br>
>>> Cheers,<br>
>>><br>
>>> Joaquín.<br>
>>><br>
>>> PS: Thanks for the work you do maintaining Cuis, I'm having a really good<br>
>>> time with it.<br>
>>><br>
>>><br>
>><br>
> <br>
<br>
Best Regards,<br>
<br>
Gerald<br>
-- <br>
Cuis-dev mailing list<br>
<a href="mailto:Cuis-dev@lists.cuis.st" target="_blank">Cuis-dev@lists.cuis.st</a><br>
<a href="https://lists.cuis.st/mailman/listinfo/cuis-dev" rel="noreferrer" target="_blank">https://lists.cuis.st/mailman/listinfo/cuis-dev</a><br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr"><div dir="ltr"><span style="font-size:xx-small;border-collapse:collapse"><div style="font-size:small"><a href="https://10pines.com/" style="font-family:Roboto,Helvetica,Arial,sans-serif;font-size:medium" target="_blank"><img width="108" src="https://10pines.github.io/email-signature/10pines-firma@2x.png" style="margin-bottom: 0.5em;"></a><span style="color:rgb(0,0,0);font-family:Roboto,Helvetica,Arial,sans-serif;font-size:medium"></span><h1 style="margin:0px;font-size:14px">Hernán Wilkinson</h1><h2 style="margin:0px 0px 1em;font-size:14px;color:rgb(100,100,100)">Software Developer, Teacher & Coach</h2><p style="margin:0px;color:rgb(100,100,100);font-size:12px">Alem 896, Floor 6, Buenos Aires, Argentina</p><p style="margin:0px;color:rgb(100,100,100);font-size:12px">+54 11 6091 3125</p><p style="margin:0px;color:rgb(100,100,100);font-size:12px">@HernanWilkinson</p></div></span></div></div>
</blockquote></div>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><span style="font-size:xx-small;border-collapse:collapse"><div style="font-size:small"><a href="https://10pines.com/" style="font-family:Roboto,Helvetica,Arial,sans-serif;font-size:medium" target="_blank"><img width="108" src="https://10pines.github.io/email-signature/10pines-firma@2x.png" style="margin-bottom: 0.5em;"></a><span style="color:rgb(0,0,0);font-family:Roboto,Helvetica,Arial,sans-serif;font-size:medium"></span><h1 style="margin:0px;font-size:14px">Hernán Wilkinson</h1><h2 style="margin:0px 0px 1em;font-size:14px;color:rgb(100,100,100)">Software Developer, Teacher & Coach</h2><p style="margin:0px;color:rgb(100,100,100);font-size:12px">Alem 896, Floor 6, Buenos Aires, Argentina</p><p style="margin:0px;color:rgb(100,100,100);font-size:12px">+54 11 6091 3125</p><p style="margin:0px;color:rgb(100,100,100);font-size:12px">@HernanWilkinson</p></div></span></div></div>