Le 21/05/15 10:55, Emmanuel Lécharny a écrit : > Hi, > > there are two class named BrowserWidget in ldapbrowser.common : > > org.apache.directory.studio.ldapbrowser.common.widgets.BrowserWidget > org.apache.directory.studio.ldapbrowser.common.widgets.browser.BrowserWidget > > The second widget is the one that display teh DIT, as expected. The > first one is, IMO, badly named. > > The first class is only ised as an abstract base class which contain the > list of listeners. It's extendeds by 27 widgets. I will rename it to > AbstractWidget, for clarity sake. I also wonder why this class is in the ldapbrowser.common plugin, instead of the common.ui plugin. I suspect this class has to be moved there, with its associated interfaces and classes, WidgetModifyListener and WidgetModifyEvent.
wdyt ?
