Issue 905 in moose-technology: The rendering of ListPresentation should not raise error when setting the selection to a value outside of the visible elements
Status: New Owner: ---- CC: usman.bh...@gmail.com Labels: Type-Defect Priority-Medium Component-Glamour
New issue 905 by tu...@tudorgirba.com: The rendering of ListPresentation should not raise error when setting the selection to a value outside of the visible elements http://code.google.com/p/moose-technology/issues/detail?id=905
testSettingSelectionOutsideOfVisibleElementsShouldNotRaiseError | browser items | browser := GLMTabulator withStatusbar. browser column: #one; column: #two. browser transmit to: #one; andShow: [ :a | a tree showOnly: 30 ]. window := browser openOn: (1 to: 100). self shouldnt: [((browser paneNamed: #one) port:#selection) value: 40] raise: Error.
Comment #1 on issue 905 by usman.bh...@gmail.com: The rendering of ListPresentation should not raise error when setting the selection to a value outside of the visible elements http://code.google.com/p/moose-technology/issues/detail?id=905
Doru, so should we open a bug entry about this for pharo? Because it seems to be coming from Morphs and not from Glamour directly. I would like to work with Benjamin during this week to see what can be done about it.
Comment #2 on issue 905 by tu...@tudorgirba.com: The rendering of ListPresentation should not raise error when setting the selection to a value outside of the visible elements http://code.google.com/p/moose-technology/issues/detail?id=905
Indeed, the error comes from MorphTreeMorph. Feel free to open it in Pharo and fix it :)
Comment #3 on issue 905 by usman.bh...@gmail.com: The rendering of ListPresentation should not raise error when setting the selection to a value outside of the visible elements http://code.google.com/p/moose-technology/issues/detail?id=905
Comment #4 on issue 905 by usman.bh...@gmail.com: The rendering of ListPresentation should not raise error when setting the selection to a value outside of the visible elements http://code.google.com/p/moose-technology/issues/detail?id=905
The bug is fixed by Thierry, a guy from CEA who was visiting us for Pharo sprint. Please have a look.
http://code.google.com/p/pharo/issues/detail?id=7345
Now, he has suggested a fix for Glamour as well, I am attaching the fix for glamour here, can you have a look and if it looks ok, you can integrate in glamour. And then we can close this bug :-).
Attachments: Glamour-Morphic-Renderer-ThierryGoubier.182.mcz 48.0 KB
Updates: Status: Fixed
Comment #5 on issue 905 by tu...@tudorgirba.com: The rendering of ListPresentation should not raise error when setting the selection to a value outside of the visible elements http://code.google.com/p/moose-technology/issues/detail?id=905
The fix was integrated in Pharo 2.0
Updates: Labels: Milestone-4.8
Comment #6 on issue 905 by tu...@tudorgirba.com: The rendering of ListPresentation should not raise error when setting the selection to a value outside of the visible elements http://code.google.com/p/moose-technology/issues/detail?id=905
(No comment was entered for this change.)
-
moose-technology＠googlecode.com