Status: Started Owner: damien.c...@gmail.com CC: tudor.gi...@gmail.com Labels: Type-Defect Priority-Medium Component-Glamour
New issue 694 by damien.c...@gmail.com: Should be able to 'do it' in text presentations http://code.google.com/p/moose-technology/issues/detail?id=694
In Glamour, it is not possible to select a piece of text and "do it".
Steps to reproduce:
1- GLMBasicExamples open 2- Select one example 3- Select 'Source code' 4- Type 'self' somewhere and print it
It should print 'GLMBasicExamples' and not 'nil' as it currently does.
Updates: Status: Verified
Comment #1 on issue 694 by damien.c...@gmail.com: Should be able to 'do it' in text presentations http://code.google.com/p/moose-technology/issues/detail?id=694
Fixed in Glamour-Examples-DamienCassou.201.mcz, Glamour-Morphic-Widgets-DamienCassou.25.mcz, and Glamour-Presentations-DamienCassou.101.mcz
Updates: Labels: Milestone-4.6
Comment #2 on issue 694 by tudor.gi...@gmail.com: Should be able to 'do it' in text presentations http://code.google.com/p/moose-technology/issues/detail?id=694
This is an issue that should be coded in the browser. It should not be offered by default.
The TextPresentation offers variableBindings. Please take a look at it and see if that solves the issue. I actually never tried to bind self.
Comment #3 on issue 694 by damien.c...@gmail.com: Should be able to 'do it' in text presentations http://code.google.com/p/moose-technology/issues/detail?id=694
Ok, I changed the default to be 'nil' so that no binding is done on 'self'. I also changed the GTInspector so that 'self' is now bound to the currently inspected object instead of 'nil'.
The 'variableBindings' proposed by TextPresentation allows for binding anything but 'self' (and probably all the Smalltalk special variable but I haven't checked).
Comment #4 on issue 694 by tudor.gi...@gmail.com: Should be able to 'do it' in text presentations http://code.google.com/p/moose-technology/issues/detail?id=694
Damien added doItReceiver: to the TextPresentation. It solves the problem for the moment. In the future, we will separate SmalltalkCodePresentation from TextPresentation.
Updates: Status: Fixed
Comment #5 on issue 694 by tudor.gi...@gmail.com: Should be able to 'do it' in text presentations http://code.google.com/p/moose-technology/issues/detail?id=694
(No comment was entered for this change.)