[Pharo-project] Preference to settings refactoring: alternativeBrowseIt case
current comment: "if true, then the "browse it" (cmd-b) feature in the text editor will offer you a list of class names matching the selection, if appropriate" Action: I would remove this preference since this feature is always desired. Consequence: Only used from ParagraphEditor>>browseIt wich directly send #browseClassFromIt to self if #alternativeBrowseIt is enabled. Since ParagraphEditor>>browseIt is the only sender of #browseClassFromIt, I would replace ParagraphEditor>>browseIt code with ParagraphEditor>>browseClassFromIt code and remove #browseClassFromIt. a .cs is attached Alain On May 12, 2009, at 1:48 PM, Alain Plantec wrote:
http://code.google.com/p/pharo/issues/detail?id=808
Alain
_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Excellent! Stef On May 12, 2009, at 2:46 PM, Stéphane Ducasse wrote:
current comment: "if true, then the "browse it" (cmd-b) feature in the text editor will offer you a list of class names matching the selection, if appropriate" Action: I would remove this preference since this feature is always desired. Consequence: Only used from ParagraphEditor>>browseIt wich directly send #browseClassFromIt to self if #alternativeBrowseIt is enabled. Since ParagraphEditor>>browseIt is the only sender of #browseClassFromIt, I would replace ParagraphEditor>>browseIt code with ParagraphEditor>>browseClassFromIt code and remove #browseClassFromIt. a .cs is attached Alain
On May 12, 2009, at 1:48 PM, Alain Plantec wrote:
http://code.google.com/p/pharo/issues/detail?id=808
Alain
_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
participants (2)
-
Alain Plantec -
Stéphane Ducasse