|Anonymous | Login||09-28-2020 10:23 UTC|
|Main | My View | View Issues | Change Log | Docs|
|Viewing Issue Simple Details [ Jump to Notes ]||[ View Advanced ] [ Issue History ] [ Print ]|
|ID||Category||Severity||Reproducibility||Date Submitted||Last Update|
|0006928||[Squeak Packages] OmniBrowser||minor||always||02-15-08 01:23||02-20-08 09:19|
|Reporter||Nicolai Hess||View Status||public|
|Summary||0006928: No pretty-printing in OBMultipleSelectionBrowser|
OBCmdPrettyPrint>>execute search the browserpanels for one of kind
OBDefinitionPanel to getDefinition and pretty-printing,
but OBMultipleSelectionBrowser uses OBMultipleDefinitionPanels
instead, and so OBCmdPrettyPrint leaves the execute method
I tried to fix this, by searching panels of kind OBDefinitionPanel or
But I don't know how to get the definition from here, as it is hold
by one of his OBCloseableDefinitionPanel.
Can I get the current active definitionpanel from the OBMultipleDefinitionPanel?
Maybe it is simpler to execute this command not by OBCommand>>execute, but
Here we can take the additional orSendTo-Argument to get the current Textmorph
than the model and hope it is a kind of DefinitionPanel:
| morph definition source |
(anObject isKindOf: OBPluggableTextMorph)
definition:= anObject model getDefinition.
source := morph text asString.
(definition prettyPrint: source) = source
ifTrue: [ ^ self ].
requestor browser announce: definition.
morph hasUnacceptedEdits: true
(0011828 - 687 - 738 - 738 - 738 - 738 - 738)
There are actually two pretty print commands available: One from OB-Refactory (which is broken in the multiple selection browser) and one in the cmd OBCmdMultiView which still works also in the multiple selection browser. You can access it in the fixed tab panel (the panel between the columns and the definition panel) on the right side.
The fix for the other command needs to be done in the OmniBrowser and in the OB-Refactory. I'm working on that.
One possibility to get the current active def panel from an OBMultipleDefintionPanel would be this code (requires the newest version of OB-Enhancements):
^self panels detect: [:ea | ea node = self currentNode]
(0011851 - 358 - 376 - 482 - 482 - 482 - 482)
I fixed this issue now.
You need the newest version of the following packages: OmniBrowser, OB-Refactory and OB-Enhancements. You can load them from http://source.wiresong.ca/ob. [^]
Please note that I recently changed a lot of code in OB-Enhancements which is not yet thoroughly tested. Hence you might find new issues when you update to the latest version.
|02-15-08 01:23||Nicolai Hess||New Issue|
|02-15-08 01:23||Nicolai Hess||Status||new => assigned|
|02-15-08 01:23||Nicolai Hess||Assigned To||=> Damien Cassou|
|02-15-08 08:51||Damien Cassou||Assigned To||Damien Cassou => dr|
|02-18-08 14:56||dr||Note Added: 0011828|
|02-20-08 09:19||dr||Status||assigned => resolved|
|02-20-08 09:19||dr||Resolution||open => fixed|
|02-20-08 09:19||dr||Note Added: 0011851|
| Mantis 1.0.8[^]
Copyright © 2000 - 2007 Mantis Group
42 total queries executed.|
33 unique queries executed.