[Cuis-dev] Proposed change to StringRequestMorph

[Mauro Rizzi]

Hello Juan,

I discovered the renameCategory option uses the class side of
StringRequestMorph to make the request and made the appropriate changes so
it follows the same behaviour as every other dialogue.

I'm unsure what you need to add this so I'm attaching two changesets, one
that includes both this fix and the previous one and one that only includes
the fix.
Note: The fix changeset (The one made in Dec09) requires the first
changeset to be installed to work.

Cheers!
*Mauro Rizzi*

El mar, 8 dic 2020 a las 17:32, Juan Vuletich (<juan at jvuletich.org>)
escribió:

> On 12/7/2020 8:58 PM, Mauro Rizzi via Cuis-dev wrote:
>
> Oops!
>
> What I thought was a bug because of this changeset is actually standard
> behaviour so the changeset actually works as intended.
>
> I'll happily look into any errors you guys can find!
>
> Cheers!
> *Mauro Rizzi*
>
> El lun, 7 dic 2020 a las 18:30, Mauro Rizzi (<mrizzi at fi.uba.ar>) escribió:
>
>> Hello everyone,
>>
>> I didn't go deep enough testing it and now realized things like the
>> debugger will automatically focus the last created StringRequestMorph in
>> its composition so maybe this isn't the final solution.
>>
>> I'm looking into coming up with a fix.
>>
>> Cheers!
>> *Mauro Rizzi*
>>
>> El lun, 7 dic 2020 a las 16:38, Mauro Rizzi (<mrizzi at fi.uba.ar>)
>> escribió:
>>
>>> Hello everyone!
>>>
>>> I got tired of having to click on text boxes every time I tried to
>>> rename something (when using click to focus) and decided to do something
>>> about it.
>>>
>>> Attached you'll find a changeset that makes it so those string request
>>> dialogue boxes automatically gain focus when created (It seems to be
>>> standard behaviour for the rest of the system but not for this particular
>>> class).
>>>
>>> I would also like to request this change be introduced to the base image.
>>>
>>> If you check the changeset the changes to the base codebase are minimal,
>>> and judging by the left over " textPane focusText." comment they were
>>> intended to exist at some point.
>>>
>>> Only thing I'm unsure about is if I should check that we're on the click
>>> to focus setting or not.
>>>
>>> Cheers!
>>> *Mauro Rizzi*
>>>
>>
> Integrated.
>
> Thanks!
>
> --
> Juan Vuletichwww.cuis-smalltalk.orghttps://github.com/Cuis-Smalltalk/Cuis-Smalltalk-Devhttps://github.com/jvuletichhttps://www.linkedin.com/in/juan-vuletich-75611b3
> @JuanVuletich
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.cuis.st/mailman/archives/cuis-dev/attachments/20201209/ba65e305/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 4386-CuisCore-MauroRizzi-2020Dec07-16h29m-MJR.002.cs.st
Type: application/octet-stream
Size: 1701 bytes
Desc: not available
URL: <http://lists.cuis.st/mailman/archives/cuis-dev/attachments/20201209/ba65e305/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 4386-CuisCore-MauroJulianRizzi-2020Dec09-21h23m-MJR.001.cs.st
Type: application/octet-stream
Size: 659 bytes
Desc: not available
URL: <http://lists.cuis.st/mailman/archives/cuis-dev/attachments/20201209/ba65e305/attachment-0001.obj>