I found myself annoyed that trying to select the pasted in results of a 'copy reference' in a Browser in order to make it a 'link to method' doesn't work. Seems like something that *should*.
i.e. Magnitude + can be selected and made into a link that will open a Browser on that method, but Magnitude>>#+ cannot.
The issue is that the parsing used in TextAction>>#analyze: and TextLink>>#validate: is a bit naive and out of date.
The first problem is that the #analyze: code relies on using <> as a variety of delimiter - and that completely fails if you have the temerity to want to make a link to a method that include either character! It was probably a bad idea ab initio to use a delimiter character that is a valid part of a method name but I guess we're stuck with it.
The current comment explains that the code is intending to parse out cases such as -
3+4 <3+4> Click Here<3+4> <3+4>Click Here
It can't work for Magnitude > <Magnitude >> for example.
The second issue is that the #validate: method needs to see if part of the input string can be splitBy: '>>#'. *That* part I solved, if rather uglyly (yes, I invented new word for the occasion).
It's hardly a major thing but improving it would be nice. Actually specifying properly all the cases we want to handle would be a good start. And one couldn't honestly claim that TextEditor>>#changeEmphasis: and SmalltalkEditor>>#handleEmphasisExtra:with: are paragons of virtuous coding.
tim -- tim Rowledge; tim@rowledge.org; http://www.rowledge.org/tim Nostalgia: The good old days multiplied by a bad memory...
squeak-dev@lists.squeakfoundation.org