Issue 7295 in pharo: [ENH]: Critic Browser - Improve Resetting

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
8 messages Options
Reply | Threaded
Open this post in threaded view
|

Issue 7295 in pharo: [ENH]: Critic Browser - Improve Resetting

pharo
Status: Accepted
Owner: [hidden email]
CC: [hidden email],  [hidden email]
Labels: Type-Enh

New issue 7295 by [hidden email]: [ENH]: Critic Browser - Improve  
Resetting
http://code.google.com/p/pharo/issues/detail?id=7295

Pharo2.0a
Latest update: #20480

First (and slightly OT) I was surprised that the second time I ran the  
browser (from the World Menu), it started running the settings from the  
first run without any prompt. That doesn't seem right, potentially running  
a very time-consuming operation that I may not want.

Back to business... It took me a while to figure out how to reset the  
browser. I finally found that if I write clicked in the rules list, there  
was a "Reset Working Package Set" option, which is still not clear that it  
brings you back to the initial wizard, and seems to do nothing until you  
close and relaunch; it seems like it should automatically bring up the  
wizard.

So, my suggestion:
1.  either
     a. prompt for reset when opening if already set up
     b. or don't automatically run if already set up
2. Add a "Reset" button prominently
3. Reword "Reset Working Package Set" to something like "New Query"
4. Automatically bring up wizard on Reset


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Reply | Threaded
Open this post in threaded view
|

Re: Issue 7295 in pharo: [ENH]: Critic Browser - Improve Resetting

pharo

Comment #1 on issue 7295 by [hidden email]: [ENH]: Critic Browser -  
Improve Resetting
http://code.google.com/p/pharo/issues/detail?id=7295

I passed the message to simon.


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Reply | Threaded
Open this post in threaded view
|

Re: Issue 7295 in pharo: [ENH]: Critic Browser - Improve Resetting

pharo
Updates:
        Status: Duplicate
        Mergedinto: 7269

Comment #2 on issue 7295 by [hidden email]: [ENH]: Critic Browser -  
Improve Resetting
http://code.google.com/p/pharo/issues/detail?id=7295

merged with 7269


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Reply | Threaded
Open this post in threaded view
|

Re: Issue 7295 in pharo: [ENH]: Critic Browser - Improve Resetting

pharo
Updates:
        Status: Accepted
        Mergedinto:

Comment #3 on issue 7295 by [hidden email]: [ENH]: Critic Browser -  
Improve Resetting
http://code.google.com/p/pharo/issues/detail?id=7295

This is unrelated to 7269


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Reply | Threaded
Open this post in threaded view
|

Re: Issue 7295 in pharo: [ENH]: Critic Browser - Improve Resetting

pharo

Comment #4 on issue 7295 by [hidden email]: [ENH]: Critic Browser -  
Improve Resetting
http://code.google.com/p/pharo/issues/detail?id=7295

Yeah it is unrelated but Simon Allier (the guy that made code critics)  
fixed both bugs in the new version of code critics. This new version of  
code critics is in inbox at error 7269.


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Reply | Threaded
Open this post in threaded view
|

Re: Issue 7295 in pharo: [ENH]: Critic Browser - Improve Resetting

pharo

Comment #5 on issue 7295 by [hidden email]: [ENH]: Critic Browser -  
Improve Resetting
http://code.google.com/p/pharo/issues/detail?id=7295

I should check with latest commit of simon because he fixed it.
But he does not have an account on the bug tracker :(



_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Reply | Threaded
Open this post in threaded view
|

Re: Issue 7295 in pharo: [ENH]: Critic Browser - Improve Resetting

pharo
Updates:
        Status: Integrated

Comment #6 on issue 7295 by [hidden email]: [ENH]: Critic Browser -  
Improve Resetting
http://code.google.com/p/pharo/issues/detail?id=7295

Fix was included and integrated with Issue 7269: Code Critic automatically  
adds class without permission


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Reply | Threaded
Open this post in threaded view
|

Re: Issue 7295 in pharo: [ENH]: Critic Browser - Improve Resetting

pharo

Comment #7 on issue 7295 by [hidden email]: [ENH]: Critic Browser -  
Improve Resetting
http://code.google.com/p/pharo/issues/detail?id=7295

in 2.0 529


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker