Ok, I changed the ActionBar for a ButtonBar. Also I added a refresh items button, in case you saved a new rule you can see it without closing the current window. All is on the master branch.
I will start to work on this in a time.

Thanks,
Sebastian
De: Pharo-users <pharo-users-bounces@lists.pharo.org> en nombre de St�phane Ducasse <stephane.ducasse@inria.fr>
Enviado: jueves, 10 de septiembre de 2020 15:35
Para: Any question about pharo is welcome <pharo-users@lists.pharo.org>
Asunto: Re: [Pharo-users] Rewrite Tool set in Spec2 changes
 


On 9 Sep 2020, at 19:28, Sebastian Jordan <sebastianjmt@hotmail.com> wrote:

Hi to all,

Thanks to the received feedback, I made some changes on RewriteTool-Spec2.
  • First, when loading a rule a more detailed description of the rules is displayed on the RewriteRulesLoaderPresenter.
Yes this is much better.
- could you use a menuButtonbar for the buttons? Like that we can style it later. 
  • Also, I added several examples of Rewrite Rules. These rules are examples of common refactors that are made in Pharo.
Yes this is good. 
  • Finally, when you open Match Tool from the Basic Rewrite Editor, the searchFor part of the rule is set to MatchTool pattern code.
Yes you see having a default value is much better. 

Now what would be great is to see if we can use the same logic than the playground
so that the tool keeps a list of previous expression. 

S

Screenshot of the new way to load a rule: https://imgur.com/gallery/Rp6Jfw2
The repo of the tool: https://github.com/jordanmontt/RewriteTool-Spec2

Thanks!
Sebastian


--------------------------------------------
St�phane Ducasse
03 59 35 87 52
Assistant: Aurore Dalle 
FAX 03 59 57 78 50
TEL 03 59 35 86 16
S. Ducasse - Inria
40, avenue Halley, 
Parc Scientifique de la Haute Borne, B�t.A, Park Plaza
Villeneuve d'Ascq 59650
France