Hi, I begun to fix a bug, I have a working fix (I think). But I am not satisfy by a method I made. I think the method can be much better but I don't have the knowledge and time to improve it now. If someone wish to take a look: https://pharo.fogbugz.com/f/cases/11091/Ensure-the-current-extension-is-alwa... -- Cyril Ferlicot http://www.synectique.eu 165 Avenue Bretagne Lille 59000 France
Le 24/10/2015 15:27, Ferlicot D. Cyril a écrit :
Hi,
I begun to fix a bug, I have a working fix (I think). But I am not satisfy by a method I made. I think the method can be much better but I don't have the knowledge and time to improve it now.
If someone wish to take a look: https://pharo.fogbugz.com/f/cases/11091/Ensure-the-current-extension-is-alwa...
Thanks to Nicolai and Esteban for the help. As Esteban said this change can be confusing so I want to have your opinion guys. Do you think that the protocols of the current extension should be before the normal protocols ? There is an example on the issue at: https://pharo.fogbugz.com/f/cases/11091/Ensure-the-current-extension-is-alwa... Should Pharo integrate this slice or not ? -- Cyril Ferlicot http://www.synectique.eu 165 Avenue Bretagne Lille 59000 France
2015-10-25 0:58 GMT+02:00 Ferlicot D. Cyril <cyril.ferlicot@gmail.com>:
Le 24/10/2015 15:27, Ferlicot D. Cyril a écrit :
Hi,
I begun to fix a bug, I have a working fix (I think). But I am not satisfy by a method I made. I think the method can be much better but I don't have the knowledge and time to improve it now.
If someone wish to take a look:
https://pharo.fogbugz.com/f/cases/11091/Ensure-the-current-extension-is-alwa...
Thanks to Nicolai and Esteban for the help. As Esteban said this change can be confusing so I want to have your opinion guys.
Do you think that the protocols of the current extension should be before the normal protocols ? There is an example on the issue at:
https://pharo.fogbugz.com/f/cases/11091/Ensure-the-current-extension-is-alwa...
Should Pharo integrate this slice or not ?
I like it. But maybe we can add an option in Nautilus settings for this.
-- Cyril Ferlicot
165 Avenue Bretagne Lille 59000 France
This is really nice that you guys spent time improving the user experience. I deeply appreciate it. Stef Le 24/10/15 09:27, Ferlicot D. Cyril a écrit :
Hi,
I begun to fix a bug, I have a working fix (I think). But I am not satisfy by a method I made. I think the method can be much better but I don't have the knowledge and time to improve it now.
If someone wish to take a look: https://pharo.fogbugz.com/f/cases/11091/Ensure-the-current-extension-is-alwa...
I like it, but wonder if it should even appear above "-- all --". Possibly it should also remain in its original sort position in the list for consistency. I tried this out but the second entry cannot be selected, which seems strange. Maybe list selection is by-content rather than by-index - and I wonder if this is peculiar to Nautilus or standard for list selection in the system? cheers -ben On Mon, Oct 26, 2015 at 6:33 AM, stepharo <stepharo@free.fr> wrote:
This is really nice that you guys spent time improving the user experience. I deeply appreciate it.
Stef
Le 24/10/15 09:27, Ferlicot D. Cyril a écrit :
Hi,
I begun to fix a bug, I have a working fix (I think). But I am not satisfy by a method I made. I think the method can be much better but I don't have the knowledge and time to improve it now.
If someone wish to take a look:
https://pharo.fogbugz.com/f/cases/11091/Ensure-the-current-extension-is-alwa...
participants (5)
-
Ben Coman -
Ferlicot D. Cyril -
Nicolai Hess -
Stephan Eggermont -
stepharo