[squeak-dev] Merge Request: removeClass.cs

Marcel Taeumel marcel.taeumel at hpi.de
Wed Mar 3 09:18:52 UTC 2021


Hmm... tricky. :-)

Here is the current one:

- Could you keep the title and add the prefixes "yes" or "no" before the options?
- What is your preferred way of having line breaks in the text?

Here is the current remove-selector dialog:

... Well, this does not scale at all. What about having a single extra button for "more options" in such confirmation dialogs? Maybe like a small drop-down menu.

[Yes] [No]  <--- SPACE---> [More...]

Best,
Marcel

Am 02.03.2021 18:26:16 schrieb Thiede, Christoph <christoph.thiede at student.hpi.uni-potsdam.de>:
This changeset refactors the #removeClass logic (which has been upgraded by Marcel recently) and extracts it to SystemNavigation, analogously to System-ct.1221. It also refines the dialog to search for senders and subclasses of the class to remove as we know it from #removeMessage.



Please merge or give me feedback for improvements. :-)

Best,
Christoph
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20210303/5b09bf07/attachment-0001.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: image.png
Type: image/png
Size: 28990 bytes
Desc: not available
URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20210303/5b09bf07/attachment-0003.png>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: image.png
Type: image/png
Size: 20780 bytes
Desc: not available
URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20210303/5b09bf07/attachment-0004.png>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: pastedImage.png
Type: image/png
Size: 23381 bytes
Desc: not available
URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20210303/5b09bf07/attachment-0005.png>


More information about the Squeak-dev mailing list