Community
Participate
Working Groups
- the name is obsolete (eWeb) and it should explain what it does (BorderContainer with toggling splitter). - it is fairly specific in how it works. It toggles the left pane and gives the space to the center pane. So it's not really a reusable generic BorderContainer. - maybe dojo has a toggling splitter by now? - in bug 343912, I had to use a method getSplitter which is marked as deprecated and will be removed in dojo 2.0 This almost feels like it could be implemented as an external helper method that hooks into a BorderContainer in a specific way rather than extending it.
Please keep me in the loop if it gets tossed or rewritten. Because of the derivate work of dojo in that file, there are various legal files that would need cleaning up if this goes away.
(In reply to comment #1) > Please keep me in the loop if it gets tossed or rewritten. Because of the > derivate work of dojo in that file, there are various legal files that would > need cleaning up if this goes away. will do. I think I'd like to see the derivative code go away because it's that kind of copied implementation code that breaks us from dojo release to release.
see also bug 347308, where we are also hacking our copied version of the dojox.layout.ToggleSplitter.
We are no longer using this class, though I have not removed it yet. In addition to the legal files John mentioned, I think we could remove our use toggle splitter from the built dojo and any other build infrastructure (page optimization) associated with this file.
fixed in http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?id=c950298538f189fc97e5576679c09110195dd92d removed eWebBorderContainer. removed the special licensing discussion in about.html of client.core removed the dojo license file and about-files folder updated bundles/org.eclipse.orion.dojo/layer/layer.js and bundles/org.eclipse.orion.dojo/layer/layer.js to remove the reference to dojox.ToggleSplitter Some free cleanup while mucking around in this: - removed TitlePane from the layer.js files since we don't use it - removed a rogue "LabeledCheckbox.js" that was in the client.core root
