<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
  <head>
    <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
    <title></title>
  </head>
  <body bgcolor="#ffffff" text="#000000">
    Hi Mauro,<br>
    <br>
    Let me go back a little bit. If you want to keep your code as an
    optional add on, I think it is quite good. The only thing I'd
    perhaps change is to avoid redefinition of existing methods as much
    as possible, because these are fragile, and will break if the code
    in the base image changes.<br>
    <br>
    But if you want your contribution to be part of the base image (this
    is reasonable, we want Cuis to be comfortable for users!), then keep
    in mind that Cuis tries to be minimalist. We include all that is
    worth its own weight, in terms of class and method count, lines of
    code, etc. This keeps Cuis being a system that is easy to adapt and
    evolve. In this light, having several menu options for essentially
    the same thing is what I called "a bit too much". I'd rather prefer
    a single option on each menu, that suggests the destination folder,
    and allows modifying it, and remembers it for next time.<br>
    <br>
    Additionally, for code to become part of Cuis base image, we need to
    save it in .cs.st changeset files, instead of a .pck.st package
    file.<br>
    <br>
    Thanks,<br>
    <br>
    On 11/23/2020 8:17 PM, Mauro Rizzi via Cuis-dev wrote:
    <blockquote
cite="mid:CAMLYRA3g3_7=J+C0Paof4mce2n-dG+TTVoRawOUBbpCCSL4naw@mail.gmail.com"
      type="cite">
      <div dir="ltr">
        <div>Hello Juan,</div>
        <div><br>
        </div>
        <div>
          The fileOutTo submenu option does exactly what you suggest.<br>
        </div>
        <div><br>
        </div>
        <div>If you never use the fileOutTo option fileOut has the same
          behaviour as the base image fileOut since the default fileOut
          location for my setup is smalltalkImageDirectory (The same
          used for the original fileOut).</div>
        <div><br>
        </div>
        <div>My idea was that as a user you would do fileOutTo to the
          folder you're currently working on, use fileOut normally in
          that folder until you need to work on a different folder and
          then you'd do fileOutTo to that new folder and continue on
          from there.<br>
        </div>
        <div><br>
        </div>
        <div>However, if you think it's better to have the option to
          file out to the image directory with a single button on the
          context menus I could change the package so it keeps the
          original fileOut option and adds fileOut to work directory and
          fileOut to new work directory as new options (So there would
          be 3 fileOut options, the current smalltalk implementation,
          fileOutToWorkDirectory and fileOutToNewDestination).</div>
        <div><br>
        </div>
        <div>Of course I would also add the corresponding methods across
          the system so the fileOut method has the old implementation
          and my current fileOut reimplementation would be renamed to
          fileOutToWorkDirectory.</div>
        <div><br>
        </div>
        <div>However i'm not sure if you wanted the current fileOutTo
          implementation and missed the option or if you want to have
          the regular fileOut option stay on the menu. I'd love to get
          further feedback on why you feel the current implementation
          sounds a bit too much for you (So I can improve it!).<br>
        </div>
        <div><br>
        </div>
        <div>Kind regards.</div>
        <div><b>Mauro Rizzi</b><br>
        </div>
      </div>
      <br>
      <div class="gmail_quote">
        <div dir="ltr" class="gmail_attr">El lun, 23 nov 2020 a las
          17:33, Juan Vuletich (<<a moz-do-not-send="true"
            href="mailto:juan@jvuletich.org">juan@jvuletich.org</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 bgcolor="#ffffff"> On 11/18/2020 12:44 PM, Mauro Rizzi
            via Cuis-dev wrote:
            <blockquote type="cite">
              <div dir="ltr">
                <div>Hello everybody!</div>
                <div><br>
                </div>
                <div>My name is Mauro Rizzi and i'm currently studying
                  Computer Engineering in the Engineering Faculty of the
                  University of Buenos Aires.</div>
                <div><br>
                </div>
                <div>We're using cuisUniversity as an environment for
                  one of our classes and I found current fileOut
                  implementations to be counter productive to a good git
                  workflow. The current idea of having fileOut always
                  fileOut to image path means you either have to have
                  your repository on your image folder or you have to
                  manually move the file to your local repository
                  folder.</div>
                <div><br>
                </div>
                <div>This is not conductive to making frequent commits
                  but rather it results in us just doing a single giant
                  commit at the end of the session.<br>
                </div>
                <div><br>
                </div>
                <div>Because of this I set out to make a package that
                  would streamline the user experience allowing for cuis
                  to remember the last destination you filed out to and
                  adding the option for the user to directly specify a
                  new destination.</div>
                <div><br>
                </div>
                <div>You can find the package in my <a
                    moz-do-not-send="true"
                    href="https://github.com/Mauro-Rizzi/FileOutRework"
                    target="_blank">FileOutRework repository.</a></div>
                <div><br>
                </div>
                <div>I've just started with this rework but I was
                  encouraged by my teacher to share it with you, so far
                  I've only implemented this behaviour for the Category
                  list context submenus for the System browser.</div>
                <div><br>
                </div>
                <div>I'm open to any comments, criticisms or requests
                  for collaboration.</div>
                <div><br>
                </div>
                <div>Hope you all have a nice week!</div>
                <div>Kind regards.</div>
                <div><b>Mauro Rizzi</b><br>
                </div>
                <br>
              </div>
            </blockquote>
            <br>
            Hi Mauro,<br>
            <br>
            I agree that doing file out always an arbitrary tacit
            directory might be uncomfortable. Still, duplicating the
            file out menu options sounds s bit too much to me. Would a
            reasonable alternative be to include dialog with the full
            path+name in an edit box? The edited path could be
            remembered for suggesting it next time.<br>
            <br>
            Thanks,<br>
            <pre cols="72">-- 
Juan Vuletich
<a moz-do-not-send="true" href="http://www.cuis-smalltalk.org" target="_blank">www.cuis-smalltalk.org</a>
<a moz-do-not-send="true" href="https://github.com/Cuis-Smalltalk/Cuis-Smalltalk-Dev" target="_blank">https://github.com/Cuis-Smalltalk/Cuis-Smalltalk-Dev</a>
<a moz-do-not-send="true" href="https://github.com/jvuletich" target="_blank">https://github.com/jvuletich</a>
<a moz-do-not-send="true" href="https://www.linkedin.com/in/juan-vuletich-75611b3" target="_blank">https://www.linkedin.com/in/juan-vuletich-75611b3</a>
@JuanVuletich</pre>
          </div>
        </blockquote>
      </div>
    </blockquote>
    <br>
    <br>
    <pre class="moz-signature" cols="72">-- 
Juan Vuletich
<a class="moz-txt-link-abbreviated" href="http://www.cuis-smalltalk.org">www.cuis-smalltalk.org</a>
<a class="moz-txt-link-freetext" href="https://github.com/Cuis-Smalltalk/Cuis-Smalltalk-Dev">https://github.com/Cuis-Smalltalk/Cuis-Smalltalk-Dev</a>
<a class="moz-txt-link-freetext" href="https://github.com/jvuletich">https://github.com/jvuletich</a>
<a class="moz-txt-link-freetext" href="https://www.linkedin.com/in/juan-vuletich-75611b3">https://www.linkedin.com/in/juan-vuletich-75611b3</a>
@JuanVuletich</pre>
  </body>
</html>