Author: vjrj_ Date: 2011-05-05 04:24:50 +0200 (Thu, 05 May 2011) New Revision: 1343 Modified: trunk/src/main/java/cc/kune/common/client/actions/gwtui/AbstractGwtButtonGui.java trunk/src/main/java/cc/kune/common/client/actions/gwtui/AbstractGwtMenuItemGui.java trunk/src/main/java/cc/kune/common/client/actions/gwtui/GwtIconLabelGui.java trunk/src/main/java/cc/kune/common/client/actions/gwtui/GwtMenuGui.java trunk/src/main/java/cc/kune/common/client/actions/gwtui/GwtMenuSeparatorGui.java trunk/src/main/java/cc/kune/common/client/actions/gwtui/GwtSubMenuGui.java trunk/src/main/java/cc/kune/common/client/actions/gwtui/GwtToolbarGui.java trunk/src/main/java/cc/kune/common/client/actions/gwtui/GwtToolbarSeparatorGui.java trunk/src/main/java/cc/kune/common/client/actions/gxtui/AbstractGxtButtonGui.java trunk/src/main/java/cc/kune/common/client/actions/gxtui/AbstractGxtMenuGui.java trunk/src/main/java/cc/kune/common/client/actions/gxtui/AbstractGxtMenuItemGui.java trunk/src/main/java/cc/kune/common/client/actions/gxtui/GxtMenuGui.java trunk/src/main/java/cc/kune/common/client/actions/gxtui/GxtMenuSeparatorGui.java trunk/src/main/java/cc/kune/common/client/actions/gxtui/GxtMenuTitleItemGui.java trunk/src/main/java/cc/kune/common/client/actions/gxtui/GxtToolbarGui.java trunk/src/main/java/cc/kune/common/client/actions/gxtui/GxtToolbarSeparatorGui.java trunk/src/main/java/cc/kune/common/client/actions/ui/AbstractGuiItem.java trunk/src/main/java/cc/kune/common/client/ui/IconLabel.java trunk/src/main/java/cc/kune/common/public/kune-common.css trunk/src/main/java/cc/kune/core/client/services/FileDownloadUtils.java trunk/src/main/java/cc/kune/core/client/sitebar/SiteUserOptionsPresenter.java trunk/src/main/java/cc/kune/core/client/sn/AbstractSNPresenter.java trunk/src/main/java/cc/kune/core/client/sn/UserSNPresenter.java trunk/src/main/java/cc/kune/core/client/sn/actions/registry/GroupSNConfActions.java trunk/src/main/java/cc/kune/core/client/sn/actions/registry/UserSNConfActions.java trunk/src/main/java/cc/kune/wave/client/WebClient.java Log: Some style fixes Modified: trunk/src/main/java/cc/kune/common/client/actions/gwtui/AbstractGwtButtonGui.java =================================================================== --- trunk/src/main/java/cc/kune/common/client/actions/gwtui/AbstractGwtButtonGui.java 2011-05-05 01:45:26 UTC (rev 1342) +++ trunk/src/main/java/cc/kune/common/client/actions/gwtui/AbstractGwtButtonGui.java 2011-05-05 02:24:50 UTC (rev 1343) @@ -123,6 +123,12 @@ layout(); } + @Override + public void setIconUrl(final String url) { + iconLabel.setLeftIconUrl(url); + layout(); + } + public void setPressed(final boolean pressed) { final ToggleButton toggleButton = (ToggleButton) button; Modified: trunk/src/main/java/cc/kune/common/client/actions/gwtui/AbstractGwtMenuItemGui.java =================================================================== --- trunk/src/main/java/cc/kune/common/client/actions/gwtui/AbstractGwtMenuItemGui.java 2011-05-05 01:45:26 UTC (rev 1342) +++ trunk/src/main/java/cc/kune/common/client/actions/gwtui/AbstractGwtMenuItemGui.java 2011-05-05 02:24:50 UTC (rev 1343) @@ -192,6 +192,12 @@ layout(); } + @Override + public void setIconUrl(final String url) { + iconLabel.setRightIconUrl(url); + layout(); + } + private void setRadioChecked(final Boolean checked) { iconLabel.setLeftIconResource(checked ? res.radioChecked() : res.radioUnChecked()); } Modified: trunk/src/main/java/cc/kune/common/client/actions/gwtui/GwtIconLabelGui.java =================================================================== --- trunk/src/main/java/cc/kune/common/client/actions/gwtui/GwtIconLabelGui.java 2011-05-05 01:45:26 UTC (rev 1342) +++ trunk/src/main/java/cc/kune/common/client/actions/gwtui/GwtIconLabelGui.java 2011-05-05 02:24:50 UTC (rev 1343) @@ -81,6 +81,11 @@ } @Override + public void setIconUrl(final String url) { + iconLabel.setLeftIconUrl(url); + } + + @Override public void setText(final String text) { iconLabel.setText(text); } Modified: trunk/src/main/java/cc/kune/common/client/actions/gwtui/GwtMenuGui.java =================================================================== --- trunk/src/main/java/cc/kune/common/client/actions/gwtui/GwtMenuGui.java 2011-05-05 01:45:26 UTC (rev 1342) +++ trunk/src/main/java/cc/kune/common/client/actions/gwtui/GwtMenuGui.java 2011-05-05 02:24:50 UTC (rev 1343) @@ -114,6 +114,14 @@ } @Override + public void setIconUrl(final String url) { + if (notStandAlone) { + iconLabel.setLeftIconUrl(url); + layout(); + } + } + + @Override public void setText(final String text) { if (notStandAlone) { iconLabel.setText(text); Modified: trunk/src/main/java/cc/kune/common/client/actions/gwtui/GwtMenuSeparatorGui.java =================================================================== --- trunk/src/main/java/cc/kune/common/client/actions/gwtui/GwtMenuSeparatorGui.java 2011-05-05 01:45:26 UTC (rev 1342) +++ trunk/src/main/java/cc/kune/common/client/actions/gwtui/GwtMenuSeparatorGui.java 2011-05-05 02:24:50 UTC (rev 1343) @@ -26,34 +26,40 @@ public class GwtMenuSeparatorGui extends AbstractGuiItem { - @Override - public AbstractGuiItem create(final GuiActionDescrip descriptor) { - final AbstractGwtMenuGui menu = ((AbstractGwtMenuGui) descriptor.getParent().getValue(ParentWidget.PARENT_UI)); - if (menu == null) { - throw new UIException("To add a menu separator you need to add the menu before. Item: " + descriptor); - } - menu.addSeparator(); - return menu; + @Override + public AbstractGuiItem create(final GuiActionDescrip descriptor) { + final AbstractGwtMenuGui menu = ((AbstractGwtMenuGui) descriptor.getParent().getValue( + ParentWidget.PARENT_UI)); + if (menu == null) { + throw new UIException("To add a menu separator you need to add the menu before. Item: " + + descriptor); } + menu.addSeparator(); + return menu; + } - @Override - protected void setEnabled(final boolean enabled) { - } + @Override + protected void setEnabled(final boolean enabled) { + } - @Override - protected void setIconStyle(final String style) { - } + @Override + protected void setIconStyle(final String style) { + } - @Override - protected void setText(final String text) { - } + @Override + public void setIconUrl(final String url) { + } - @Override - protected void setToolTipText(final String text) { - } + @Override + protected void setText(final String text) { + } - @Override - public boolean shouldBeAdded() { - return false; - } + @Override + protected void setToolTipText(final String text) { + } + + @Override + public boolean shouldBeAdded() { + return false; + } } Modified: trunk/src/main/java/cc/kune/common/client/actions/gwtui/GwtSubMenuGui.java =================================================================== --- trunk/src/main/java/cc/kune/common/client/actions/gwtui/GwtSubMenuGui.java 2011-05-05 01:45:26 UTC (rev 1342) +++ trunk/src/main/java/cc/kune/common/client/actions/gwtui/GwtSubMenuGui.java 2011-05-05 02:24:50 UTC (rev 1343) @@ -28,57 +28,64 @@ public class GwtSubMenuGui extends AbstractGwtMenuGui { - private IconLabel iconLabel; - private MenuItem item; + private IconLabel iconLabel; + private MenuItem item; - @Override - public AbstractGuiItem create(final GuiActionDescrip descriptor) { - super.descriptor = descriptor; - super.create(descriptor); - item = new MenuItem("", menu); - iconLabel = new IconLabel(""); - configureItemFromProperties(); - final AbstractGwtMenuGui parentMenu = ((AbstractGwtMenuGui) descriptor.getParent().getValue(PARENT_UI)); - final int position = descriptor.getPosition(); - if (position == GuiActionDescrip.NO_POSITION) { - parentMenu.add(item); - } else { - parentMenu.insert(position, item); - } - descriptor.putValue(ParentWidget.PARENT_UI, this); - return this; + @Override + public AbstractGuiItem create(final GuiActionDescrip descriptor) { + super.descriptor = descriptor; + super.create(descriptor); + item = new MenuItem("", menu); + iconLabel = new IconLabel(""); + configureItemFromProperties(); + final AbstractGwtMenuGui parentMenu = ((AbstractGwtMenuGui) descriptor.getParent().getValue( + PARENT_UI)); + final int position = descriptor.getPosition(); + if (position == GuiActionDescrip.NO_POSITION) { + parentMenu.add(item); + } else { + parentMenu.insert(position, item); } + descriptor.putValue(ParentWidget.PARENT_UI, this); + return this; + } - private void layout() { - item.setHTML(iconLabel.toString()); - } + private void layout() { + item.setHTML(iconLabel.toString()); + } - @Override - public void setEnabled(final boolean enabled) { - item.setVisible(enabled); - } + @Override + public void setEnabled(final boolean enabled) { + item.setVisible(enabled); + } - @Override - public void setIconStyle(final String style) { - iconLabel.setRightIcon(style); - layout(); - } + @Override + public void setIconStyle(final String style) { + iconLabel.setRightIcon(style); + layout(); + } - @Override - public void setText(final String text) { - iconLabel.setText(text); - layout(); - } + @Override + public void setIconUrl(final String url) { + iconLabel.setRightIconUrl(url); + layout(); + } - @Override - public void setToolTipText(final String tooltip) { - item.setTitle(tooltip); - } + @Override + public void setText(final String text) { + iconLabel.setText(text); + layout(); + } - @Override - public void setVisible(final boolean visible) { - item.setVisible(visible); - iconLabel.setVisible(visible); - layout(); - } + @Override + public void setToolTipText(final String tooltip) { + item.setTitle(tooltip); + } + + @Override + public void setVisible(final boolean visible) { + item.setVisible(visible); + iconLabel.setVisible(visible); + layout(); + } } Modified: trunk/src/main/java/cc/kune/common/client/actions/gwtui/GwtToolbarGui.java =================================================================== --- trunk/src/main/java/cc/kune/common/client/actions/gwtui/GwtToolbarGui.java 2011-05-05 01:45:26 UTC (rev 1342) +++ trunk/src/main/java/cc/kune/common/client/actions/gwtui/GwtToolbarGui.java 2011-05-05 02:24:50 UTC (rev 1343) @@ -28,60 +28,64 @@ public class GwtToolbarGui extends AbstractGuiItem implements ParentWidget { - private GwtComplexToolbar toolbar; + private GwtComplexToolbar toolbar; - @Override - public void add(final UIObject uiObject) { - toolbar.add(uiObject); - } + @Override + public void add(final UIObject uiObject) { + toolbar.add(uiObject); + } - public Widget addFill() { - return toolbar.addFill(); - } + public Widget addFill() { + return toolbar.addFill(); + } - public Widget addSeparator() { - return toolbar.addSeparator(); - } + public Widget addSeparator() { + return toolbar.addSeparator(); + } - public Widget addSpacer() { - return toolbar.addSpacer(); - } + public Widget addSpacer() { + return toolbar.addSpacer(); + } - @Override - public AbstractGuiItem create(final GuiActionDescrip descriptor) { - super.descriptor = descriptor; - toolbar = new GwtComplexToolbar(); - initWidget(toolbar); - configureItemFromProperties(); - descriptor.putValue(ParentWidget.PARENT_UI, this); - return this; - } + @Override + public AbstractGuiItem create(final GuiActionDescrip descriptor) { + super.descriptor = descriptor; + toolbar = new GwtComplexToolbar(); + initWidget(toolbar); + configureItemFromProperties(); + descriptor.putValue(ParentWidget.PARENT_UI, this); + return this; + } - @Override - public void insert(final int position, final UIObject widget) { - toolbar.insert(widget, position); - } + @Override + public void insert(final int position, final UIObject widget) { + toolbar.insert(widget, position); + } - @Override - protected void setEnabled(final boolean enabled) { - } + @Override + protected void setEnabled(final boolean enabled) { + } - @Override - protected void setIconStyle(final String style) { - } + @Override + protected void setIconStyle(final String style) { + } - @Override - protected void setText(final String text) { - } + @Override + public void setIconUrl(final String url) { + } - @Override - protected void setToolTipText(final String text) { - toolbar.setTitle(text); - } + @Override + protected void setText(final String text) { + } - @Override - public boolean shouldBeAdded() { - return true; - } + @Override + protected void setToolTipText(final String text) { + toolbar.setTitle(text); + } + @Override + public boolean shouldBeAdded() { + return true; + } + } Modified: trunk/src/main/java/cc/kune/common/client/actions/gwtui/GwtToolbarSeparatorGui.java =================================================================== --- trunk/src/main/java/cc/kune/common/client/actions/gwtui/GwtToolbarSeparatorGui.java 2011-05-05 01:45:26 UTC (rev 1342) +++ trunk/src/main/java/cc/kune/common/client/actions/gwtui/GwtToolbarSeparatorGui.java 2011-05-05 02:24:50 UTC (rev 1343) @@ -30,56 +30,62 @@ public class GwtToolbarSeparatorGui extends AbstractGuiItem { - private Widget widget; + private Widget widget; - @Override - public AbstractGuiItem create(final GuiActionDescrip descriptor) { - super.descriptor = descriptor; - final GwtToolbarGui toolbar = ((GwtToolbarGui) descriptor.getParent().getValue(ParentWidget.PARENT_UI)); - if (toolbar == null) { - throw new UIException("To add a toolbar separator you need to add the toolbar before. Item: " + descriptor); - } - final Type type = ((ToolbarSeparatorDescriptor) descriptor).getSeparatorType(); - switch (type) { - case fill: - widget = toolbar.addFill(); - break; - case spacer: - widget = toolbar.addSpacer(); - break; - case separator: - widget = toolbar.addSeparator(); - break; - default: - break; - } - configureItemFromProperties(); - return toolbar; + @Override + public AbstractGuiItem create(final GuiActionDescrip descriptor) { + super.descriptor = descriptor; + final GwtToolbarGui toolbar = ((GwtToolbarGui) descriptor.getParent().getValue( + ParentWidget.PARENT_UI)); + if (toolbar == null) { + throw new UIException("To add a toolbar separator you need to add the toolbar before. Item: " + + descriptor); } - - @Override - protected void setEnabled(final boolean enabled) { + final Type type = ((ToolbarSeparatorDescriptor) descriptor).getSeparatorType(); + switch (type) { + case fill: + widget = toolbar.addFill(); + break; + case spacer: + widget = toolbar.addSpacer(); + break; + case separator: + widget = toolbar.addSeparator(); + break; + default: + break; } + configureItemFromProperties(); + return toolbar; + } - @Override - protected void setIconStyle(final String style) { - } + @Override + protected void setEnabled(final boolean enabled) { + } - @Override - protected void setText(final String text) { - } + @Override + protected void setIconStyle(final String style) { + } - @Override - protected void setToolTipText(final String text) { - } + @Override + public void setIconUrl(final String url) { + } - @Override - public void setVisible(final boolean visible) { - widget.setVisible(visible); - } + @Override + protected void setText(final String text) { + } - @Override - public boolean shouldBeAdded() { - return false; - } + @Override + protected void setToolTipText(final String text) { + } + + @Override + public void setVisible(final boolean visible) { + widget.setVisible(visible); + } + + @Override + public boolean shouldBeAdded() { + return false; + } } Modified: trunk/src/main/java/cc/kune/common/client/actions/gxtui/AbstractGxtButtonGui.java =================================================================== --- trunk/src/main/java/cc/kune/common/client/actions/gxtui/AbstractGxtButtonGui.java 2011-05-05 01:45:26 UTC (rev 1342) +++ trunk/src/main/java/cc/kune/common/client/actions/gxtui/AbstractGxtButtonGui.java 2011-05-05 02:24:50 UTC (rev 1343) @@ -28,6 +28,7 @@ import cc.kune.common.client.actions.ui.descrip.ButtonDescriptor; import cc.kune.common.client.actions.ui.descrip.GuiActionDescrip; import cc.kune.common.client.actions.ui.descrip.PushButtonDescriptor; +import cc.kune.common.client.errors.NotImplementedException; import com.extjs.gxt.ui.client.Style.ButtonScale; import com.extjs.gxt.ui.client.event.ButtonEvent; @@ -40,112 +41,118 @@ public abstract class AbstractGxtButtonGui extends AbstractChildGuiItem { - private Button button; + private Button button; - public AbstractGxtButtonGui() { - super(); - } + public AbstractGxtButtonGui() { + super(); + } - public AbstractGxtButtonGui(final ButtonDescriptor buttonDescriptor) { - super(buttonDescriptor); - } + public AbstractGxtButtonGui(final ButtonDescriptor buttonDescriptor) { + super(buttonDescriptor); + } - @Override - protected void addStyle(final String style) { - button.addStyleName(style); - } + @Override + protected void addStyle(final String style) { + button.addStyleName(style); + } - @Override - public AbstractGuiItem create(final GuiActionDescrip descriptor) { - super.descriptor = descriptor; - descriptor.putValue(ParentWidget.PARENT_UI, this); - if (descriptor instanceof PushButtonDescriptor) { - button = new ToggleButton(); - } else { - button = new Button(); - } - button.setAutoWidth(false); - button.setAutoHeight(false); - button.setBorders(false); - final String id = descriptor.getId(); - if (id != null) { - button.ensureDebugId(id); - } - button.addSelectionListener(new SelectionListener<ButtonEvent>() { - @Override - public void componentSelected(final ButtonEvent event) { - descriptor.fire(new ActionEvent(button, getTargetObjectOfAction(descriptor), Event.as(event.getEvent()))); - } - }); - if (!descriptor.isChild()) { - // If button is inside a toolbar don't init... - initWidget(button); - } else { - if (descriptor.isChild()) { - child = button; - } - } - super.create(descriptor); - configureItemFromProperties(); - return this; + @Override + public AbstractGuiItem create(final GuiActionDescrip descriptor) { + super.descriptor = descriptor; + descriptor.putValue(ParentWidget.PARENT_UI, this); + if (descriptor instanceof PushButtonDescriptor) { + button = new ToggleButton(); + } else { + button = new Button(); } - - @Override - protected void setEnabled(final boolean enabled) { - button.setEnabled(enabled); + button.setAutoWidth(false); + button.setAutoHeight(false); + button.setBorders(false); + final String id = descriptor.getId(); + if (id != null) { + button.ensureDebugId(id); } - - @Override - public void setIconResource(final ImageResource icon) { - button.setIcon(AbstractImagePrototype.create(icon)); - button.setScale(ButtonScale.SMALL); + button.addSelectionListener(new SelectionListener<ButtonEvent>() { + @Override + public void componentSelected(final ButtonEvent event) { + descriptor.fire(new ActionEvent(button, getTargetObjectOfAction(descriptor), + Event.as(event.getEvent()))); + } + }); + if (!descriptor.isChild()) { + // If button is inside a toolbar don't init... + initWidget(button); + } else { + if (descriptor.isChild()) { + child = button; + } } + super.create(descriptor); + configureItemFromProperties(); + return this; + } - @Override - protected void setIconStyle(final String style) { - button.setIconStyle(style); - button.setScale(ButtonScale.SMALL); - } + @Override + protected void setEnabled(final boolean enabled) { + button.setEnabled(enabled); + } - public void setPressed(final boolean pressed) { - final ToggleButton toggleButton = (ToggleButton) button; + @Override + public void setIconResource(final ImageResource icon) { + button.setIcon(AbstractImagePrototype.create(icon)); + button.setScale(ButtonScale.SMALL); + } - if (toggleButton.isPressed() != pressed) { - toggleButton.toggle(pressed); - } - } + @Override + protected void setIconStyle(final String style) { + button.setIconStyle(style); + button.setScale(ButtonScale.SMALL); + } - @Override - protected void setText(final String text) { - button.setText(text); - } + @Override + public void setIconUrl(final String url) { + throw new NotImplementedException(); + } - @Override - protected void setToolTipText(final String tooltip) { - if (tooltip != null && !tooltip.isEmpty()) { - final KeyStroke key = (KeyStroke) descriptor.getValue(Action.ACCELERATOR_KEY); - if (key == null) { - // button.setToolTip(new GxtDefTooltip(tooltip)); - button.setTitle(tooltip); - } else { - // button.setToolTip(new GxtDefTooltip(tooltip + - // key.toString())); - button.setTitle(tooltip + key.toString()); - } - } + public void setPressed(final boolean pressed) { + final ToggleButton toggleButton = (ToggleButton) button; + + if (toggleButton.isPressed() != pressed) { + toggleButton.toggle(pressed); } + } - @Override - public void setVisible(final boolean visible) { - if (button.isRendered()) { - // ?? - } - button.setVisible(visible); + @Override + protected void setText(final String text) { + button.setText(text); + } + @Override + protected void setToolTipText(final String tooltip) { + if (tooltip != null && !tooltip.isEmpty()) { + final KeyStroke key = (KeyStroke) descriptor.getValue(Action.ACCELERATOR_KEY); + if (key == null) { + // button.setToolTip(new GxtDefTooltip(tooltip)); + button.setTitle(tooltip); + } else { + // button.setToolTip(new GxtDefTooltip(tooltip + + // key.toString())); + button.setTitle(tooltip + key.toString()); + } } + } - @Override - public boolean shouldBeAdded() { - return !descriptor.isChild(); + @Override + public void setVisible(final boolean visible) { + if (button.isRendered()) { + // ?? } + button.setVisible(visible); + + } + + @Override + public boolean shouldBeAdded() { + return !descriptor.isChild(); + } } Modified: trunk/src/main/java/cc/kune/common/client/actions/gxtui/AbstractGxtMenuGui.java =================================================================== --- trunk/src/main/java/cc/kune/common/client/actions/gxtui/AbstractGxtMenuGui.java 2011-05-05 01:45:26 UTC (rev 1342) +++ trunk/src/main/java/cc/kune/common/client/actions/gxtui/AbstractGxtMenuGui.java 2011-05-05 02:24:50 UTC (rev 1343) @@ -27,6 +27,7 @@ import cc.kune.common.client.actions.ui.descrip.GuiActionDescrip; import cc.kune.common.client.actions.ui.descrip.MenuDescriptor; import cc.kune.common.client.actions.ui.descrip.Position; +import cc.kune.common.client.errors.NotImplementedException; import com.extjs.gxt.ui.client.widget.menu.Menu; import com.extjs.gxt.ui.client.widget.menu.MenuItem; @@ -36,98 +37,103 @@ public abstract class AbstractGxtMenuGui extends AbstractChildGuiItem implements ParentWidget { - public enum MenuPosition { - b, // The top left corner (default) - bl, // The center of the top edge - br, // The bottom right corner ,// The top right corner - c, // The center of the left edge - l, // In the center of the element - r, // The center of the right edge - t, // The bottom left corner - tl, // The center of the bottom edge - tr - } - public static final String DEF_MENU_POSITION = "bl"; - public static final String MENU_POSITION = "menu-position"; - protected Menu menu; + public enum MenuPosition { + b, // The top left corner (default) + bl, // The center of the top edge + br, // The bottom right corner ,// The top right corner + c, // The center of the left edge + l, // In the center of the element + r, // The center of the right edge + t, // The bottom left corner + tl, // The center of the bottom edge + tr + } + public static final String DEF_MENU_POSITION = "bl"; + public static final String MENU_POSITION = "menu-position"; + protected Menu menu; - public AbstractGxtMenuGui() { - } + public AbstractGxtMenuGui() { + } - public AbstractGxtMenuGui(final GuiActionDescrip descriptor) { - super(descriptor); - } + public AbstractGxtMenuGui(final GuiActionDescrip descriptor) { + super(descriptor); + } - @Override - public void add(final UIObject item) { - menu.add((MenuItem) item); - } + @Override + public void add(final UIObject item) { + menu.add((MenuItem) item); + } - public void addSeparator() { - menu.add(new SeparatorMenuItem()); - } + public void addSeparator() { + menu.add(new SeparatorMenuItem()); + } - @Override - public void configureItemFromProperties() { - super.configureItemFromProperties(); - descriptor.addPropertyChangeListener(new PropertyChangeListener() { - @Override - public void propertyChange(final PropertyChangeEvent event) { - if (event.getPropertyName().equals(MenuDescriptor.MENU_CLEAR)) { - menu.removeAll(); - } - } - }); - } + @Override + public void configureItemFromProperties() { + super.configureItemFromProperties(); + descriptor.addPropertyChangeListener(new PropertyChangeListener() { + @Override + public void propertyChange(final PropertyChangeEvent event) { + if (event.getPropertyName().equals(MenuDescriptor.MENU_CLEAR)) { + menu.removeAll(); + } + } + }); + } - @Override - public AbstractGuiItem create(final GuiActionDescrip descriptor) { - super.descriptor = descriptor; - super.create(descriptor); - menu = new Menu(); - descriptor.addPropertyChangeListener(new PropertyChangeListener() { - @Override - public void propertyChange(final PropertyChangeEvent event) { - if (event.getPropertyName().equals(MenuDescriptor.MENU_HIDE)) { - menu.hide(); - } - } - }); - descriptor.addPropertyChangeListener(new PropertyChangeListener() { - @Override - public void propertyChange(final PropertyChangeEvent event) { - if (event.getPropertyName().equals(MenuDescriptor.MENU_SHOW)) { - show(descriptor.getValue(MenuDescriptor.MENU_SHOW_NEAR_TO)); - } - } - }); - return this; - } + @Override + public AbstractGuiItem create(final GuiActionDescrip descriptor) { + super.descriptor = descriptor; + super.create(descriptor); + menu = new Menu(); + descriptor.addPropertyChangeListener(new PropertyChangeListener() { + @Override + public void propertyChange(final PropertyChangeEvent event) { + if (event.getPropertyName().equals(MenuDescriptor.MENU_HIDE)) { + menu.hide(); + } + } + }); + descriptor.addPropertyChangeListener(new PropertyChangeListener() { + @Override + public void propertyChange(final PropertyChangeEvent event) { + if (event.getPropertyName().equals(MenuDescriptor.MENU_SHOW)) { + show(descriptor.getValue(MenuDescriptor.MENU_SHOW_NEAR_TO)); + } + } + }); + return this; + } - protected String getMenuPosition() { - final MenuPosition position = (MenuPosition) descriptor.getValue(MENU_POSITION); - return position == null ? DEF_MENU_POSITION : position.name(); - } + protected String getMenuPosition() { + final MenuPosition position = (MenuPosition) descriptor.getValue(MENU_POSITION); + return position == null ? DEF_MENU_POSITION : position.name(); + } - @Override - public void insert(final int position, final UIObject item) { - menu.insert((MenuItem) item, position); - } + @Override + public void insert(final int position, final UIObject item) { + menu.insert((MenuItem) item, position); + } - @Override - public boolean shouldBeAdded() { - return !descriptor.isChild(); - } + @Override + public void setIconUrl(final String url) { + throw new NotImplementedException(); + } - public void show(final Object relative) { - if (relative instanceof String) { - menu.show(RootPanel.get((String) relative).getElement(), getMenuPosition()); - } else if (relative instanceof UIObject) { - menu.show(((UIObject) relative).getElement(), getMenuPosition()); - } else if (relative instanceof Position) { - final Position position = (Position) relative; - menu.showAt(position.getX(), position.getY()); - } + @Override + public boolean shouldBeAdded() { + return !descriptor.isChild(); + } + + public void show(final Object relative) { + if (relative instanceof String) { + menu.show(RootPanel.get((String) relative).getElement(), getMenuPosition()); + } else if (relative instanceof UIObject) { + menu.show(((UIObject) relative).getElement(), getMenuPosition()); + } else if (relative instanceof Position) { + final Position position = (Position) relative; + menu.showAt(position.getX(), position.getY()); } + } } \ No newline at end of file Modified: trunk/src/main/java/cc/kune/common/client/actions/gxtui/AbstractGxtMenuItemGui.java =================================================================== --- trunk/src/main/java/cc/kune/common/client/actions/gxtui/AbstractGxtMenuItemGui.java 2011-05-05 01:45:26 UTC (rev 1342) +++ trunk/src/main/java/cc/kune/common/client/actions/gxtui/AbstractGxtMenuItemGui.java 2011-05-05 02:24:50 UTC (rev 1343) @@ -31,6 +31,7 @@ import cc.kune.common.client.actions.ui.descrip.MenuCheckItemDescriptor; import cc.kune.common.client.actions.ui.descrip.MenuItemDescriptor; import cc.kune.common.client.actions.ui.descrip.MenuRadioItemDescriptor; +import cc.kune.common.client.errors.NotImplementedException; import cc.kune.common.client.utils.TextUtils; import com.extjs.gxt.ui.client.event.MenuEvent; @@ -45,123 +46,129 @@ public abstract class AbstractGxtMenuItemGui extends AbstractChildGuiItem { - private MenuItem item; + private MenuItem item; - public AbstractGxtMenuItemGui() { - super(); - } + public AbstractGxtMenuItemGui() { + super(); + } - public AbstractGxtMenuItemGui(final MenuItemDescriptor descriptor) { - super(descriptor); + public AbstractGxtMenuItemGui(final MenuItemDescriptor descriptor) { + super(descriptor); - } + } - private void confCheckListener(final MenuItemDescriptor descriptor, final CheckMenuItem checkItem) { - descriptor.addPropertyChangeListener(new PropertyChangeListener() { - @Override - public void propertyChange(final PropertyChangeEvent event) { - if (event.getPropertyName().equals(MenuCheckItemDescriptor.CHECKED)) { - checkItem.setChecked((Boolean) event.getNewValue()); - } - } - }); - } - - @Override - public AbstractGuiItem create(final GuiActionDescrip descriptor) { - super.descriptor = descriptor; - if (descriptor instanceof MenuRadioItemDescriptor) { - final CheckMenuItem checkItem = createCheckItem((MenuItemDescriptor) descriptor); - checkItem.setGroup(((MenuRadioItemDescriptor) descriptor).getGroup()); - confCheckListener((MenuItemDescriptor) descriptor, checkItem); - item = checkItem; - } else if (descriptor instanceof MenuCheckItemDescriptor) { - final CheckMenuItem checkItem = createCheckItem((MenuItemDescriptor) descriptor); - confCheckListener((MenuItemDescriptor) descriptor, checkItem); - item = checkItem; - } else { - item = new MenuItem(""); + private void confCheckListener(final MenuItemDescriptor descriptor, final CheckMenuItem checkItem) { + descriptor.addPropertyChangeListener(new PropertyChangeListener() { + @Override + public void propertyChange(final PropertyChangeEvent event) { + if (event.getPropertyName().equals(MenuCheckItemDescriptor.CHECKED)) { + checkItem.setChecked((Boolean) event.getNewValue()); } + } + }); + } - final String id = descriptor.getId(); - if (id != null) { - item.ensureDebugId(id); - } - item.addSelectionListener(new SelectionListener<MenuEvent>() { - @Override - public void componentSelected(final MenuEvent ce) { - final AbstractAction action = descriptor.getAction(); - if (action != null) { - action.actionPerformed(new ActionEvent(item, getTargetObjectOfAction(descriptor), - Event.getCurrentEvent())); - } - } - }); - child = item; - super.create(descriptor); - configureItemFromProperties(); - return this; + @Override + public AbstractGuiItem create(final GuiActionDescrip descriptor) { + super.descriptor = descriptor; + if (descriptor instanceof MenuRadioItemDescriptor) { + final CheckMenuItem checkItem = createCheckItem((MenuItemDescriptor) descriptor); + checkItem.setGroup(((MenuRadioItemDescriptor) descriptor).getGroup()); + confCheckListener((MenuItemDescriptor) descriptor, checkItem); + item = checkItem; + } else if (descriptor instanceof MenuCheckItemDescriptor) { + final CheckMenuItem checkItem = createCheckItem((MenuItemDescriptor) descriptor); + confCheckListener((MenuItemDescriptor) descriptor, checkItem); + item = checkItem; + } else { + item = new MenuItem(""); } - private CheckMenuItem createCheckItem(final MenuItemDescriptor descriptor) { - final CheckMenuItem checkItem = new CheckMenuItem(); - checkItem.setChecked(((MenuCheckItemDescriptor) descriptor).isChecked()); - return checkItem; + final String id = descriptor.getId(); + if (id != null) { + item.ensureDebugId(id); } + item.addSelectionListener(new SelectionListener<MenuEvent>() { + @Override + public void componentSelected(final MenuEvent ce) { + final AbstractAction action = descriptor.getAction(); + if (action != null) { + action.actionPerformed(new ActionEvent(item, getTargetObjectOfAction(descriptor), + Event.getCurrentEvent())); + } + } + }); + child = item; + super.create(descriptor); + configureItemFromProperties(); + return this; + } - private String createShortCut(final KeyStroke key, final String style) { - // See: https://yui-ext.com/forum/showthread.php?t=5762 - final Element keyLabel = DOM.createSpan(); - keyLabel.setId(style); - keyLabel.setInnerText(key.toString()); - return keyLabel.getString(); - } + private CheckMenuItem createCheckItem(final MenuItemDescriptor descriptor) { + final CheckMenuItem checkItem = new CheckMenuItem(); + checkItem.setChecked(((MenuCheckItemDescriptor) descriptor).isChecked()); + return checkItem; + } - public MenuItem getItem() { - return item; - } + private String createShortCut(final KeyStroke key, final String style) { + // See: https://yui-ext.com/forum/showthread.php?t=5762 + final Element keyLabel = DOM.createSpan(); + keyLabel.setId(style); + keyLabel.setInnerText(key.toString()); + return keyLabel.getString(); + } - @Override - protected void setEnabled(final boolean enabled) { - item.setVisible(enabled); - } + public MenuItem getItem() { + return item; + } - @Override - public void setIconResource(final ImageResource icon) { - item.setIcon(AbstractImagePrototype.create(icon)); - } + @Override + protected void setEnabled(final boolean enabled) { + item.setVisible(enabled); + } - @Override - protected void setIconStyle(final String style) { - item.setIconStyle(style); - } + @Override + public void setIconResource(final ImageResource icon) { + item.setIcon(AbstractImagePrototype.create(icon)); + } - @Override - protected void setText(final String text) { - if (text != null) { - final KeyStroke key = (KeyStroke) descriptor.getValue(Action.ACCELERATOR_KEY); - if (key == null) { - item.setText(text); - } else { - item.setText(text + createShortCut(key, "oc-mshortcut-hidden") + createShortCut(key, "oc-mshortcut")); - } - } - } + @Override + protected void setIconStyle(final String style) { + item.setIconStyle(style); + } - @Override - protected void setToolTipText(final String tooltip) { - if (TextUtils.notEmpty(tooltip)) { - item.setToolTip(new GxtDefTooltip(tooltip)); - } + @Override + public void setIconUrl(final String url) { + throw new NotImplementedException(); + } + + @Override + protected void setText(final String text) { + if (text != null) { + final KeyStroke key = (KeyStroke) descriptor.getValue(Action.ACCELERATOR_KEY); + if (key == null) { + item.setText(text); + } else { + item.setText(text + createShortCut(key, "oc-mshortcut-hidden") + + createShortCut(key, "oc-mshortcut")); + } } + } - @Override - public void setVisible(final boolean visible) { - item.setVisible(visible); + @Override + protected void setToolTipText(final String tooltip) { + if (TextUtils.notEmpty(tooltip)) { + item.setToolTip(new GxtDefTooltip(tooltip)); } + } - @Override - public boolean shouldBeAdded() { // NOPMD by vjrj on 18/01/11 0:48 - return false; - } + @Override + public void setVisible(final boolean visible) { + item.setVisible(visible); + } + + @Override + public boolean shouldBeAdded() { // NOPMD by vjrj on 18/01/11 0:48 + return false; + } } Modified: trunk/src/main/java/cc/kune/common/client/actions/gxtui/GxtMenuGui.java =================================================================== --- trunk/src/main/java/cc/kune/common/client/actions/gxtui/GxtMenuGui.java 2011-05-05 01:45:26 UTC (rev 1342) +++ trunk/src/main/java/cc/kune/common/client/actions/gxtui/GxtMenuGui.java 2011-05-05 02:24:50 UTC (rev 1343) @@ -25,6 +25,7 @@ import cc.kune.common.client.actions.ui.ParentWidget; import cc.kune.common.client.actions.ui.descrip.GuiActionDescrip; import cc.kune.common.client.actions.ui.descrip.MenuDescriptor; +import cc.kune.common.client.errors.NotImplementedException; import cc.kune.common.client.utils.TextUtils; import com.extjs.gxt.ui.client.Style.ButtonScale; @@ -39,125 +40,130 @@ public class GxtMenuGui extends AbstractGxtMenuGui implements ParentWidget { - private SplitButton button; - private boolean notStandAlone; + private SplitButton button; + private boolean notStandAlone; - public GxtMenuGui() { - super(); - } + public GxtMenuGui() { + super(); + } - @Override - public void add(final UIObject item) { - menu.add((MenuItem) item); - } + @Override + public void add(final UIObject item) { + menu.add((MenuItem) item); + } - @Override - public void addSeparator() { - menu.add(new SeparatorMenuItem()); - } + @Override + public void addSeparator() { + menu.add(new SeparatorMenuItem()); + } - @Override - protected void addStyle(final String style) { - button.addStyleName(style); - } + @Override + protected void addStyle(final String style) { + button.addStyleName(style); + } - @Override - public void configureItemFromProperties() { - super.configureItemFromProperties(); - descriptor.addPropertyChangeListener(new PropertyChangeListener() { - @Override - public void propertyChange(final PropertyChangeEvent event) { - if (event.getPropertyName().equals(MenuDescriptor.MENU_CLEAR)) { - menu.removeAll(); - } - } - }); - } + @Override + public void configureItemFromProperties() { + super.configureItemFromProperties(); + descriptor.addPropertyChangeListener(new PropertyChangeListener() { + @Override + public void propertyChange(final PropertyChangeEvent event) { + if (event.getPropertyName().equals(MenuDescriptor.MENU_CLEAR)) { + menu.removeAll(); + } + } + }); + } - @Override - public AbstractGuiItem create(final GuiActionDescrip descriptor) { - super.descriptor = descriptor; - descriptor.putValue(ParentWidget.PARENT_UI, this); - // Standalone menus are menus without and associated button in a - // toolbar (sometimes, a menu showed in a grid, or other special - // widgets) - notStandAlone = !((MenuDescriptor) descriptor).isStandalone(); - if (notStandAlone) { - button = new SplitButton(""); - button.setStylePrimaryName("oc-button"); - button.addSelectionListener(new SelectionListener<ButtonEvent>() { - @Override - public void componentSelected(final ButtonEvent ce) { - show(button); - } - }); - final String id = descriptor.getId(); - if (id != null) { - button.ensureDebugId(id); - } - if (!descriptor.isChild()) { - initWidget(button); - } else { - child = button; - } + @Override + public AbstractGuiItem create(final GuiActionDescrip descriptor) { + super.descriptor = descriptor; + descriptor.putValue(ParentWidget.PARENT_UI, this); + // Standalone menus are menus without and associated button in a + // toolbar (sometimes, a menu showed in a grid, or other special + // widgets) + notStandAlone = !((MenuDescriptor) descriptor).isStandalone(); + if (notStandAlone) { + button = new SplitButton(""); + button.setStylePrimaryName("oc-button"); + button.addSelectionListener(new SelectionListener<ButtonEvent>() { + @Override + public void componentSelected(final ButtonEvent ce) { + show(button); } - super.create(descriptor); - configureItemFromProperties(); - return this; + }); + final String id = descriptor.getId(); + if (id != null) { + button.ensureDebugId(id); + } + if (!descriptor.isChild()) { + initWidget(button); + } else { + child = button; + } } + super.create(descriptor); + configureItemFromProperties(); + return this; + } - @Override - public void insert(final int position, final UIObject item) { - menu.insert((MenuItem) item, position); - } + @Override + public void insert(final int position, final UIObject item) { + menu.insert((MenuItem) item, position); + } - @Override - public void setEnabled(final boolean enabled) { - if (button != null) { - button.setEnabled(enabled); - } + @Override + public void setEnabled(final boolean enabled) { + if (button != null) { + button.setEnabled(enabled); } + } - @Override - public void setIconResource(final ImageResource icon) { - if (button != null) { - button.setIcon(AbstractImagePrototype.create(icon)); - button.setScale(ButtonScale.SMALL); - } + @Override + public void setIconResource(final ImageResource icon) { + if (button != null) { + button.setIcon(AbstractImagePrototype.create(icon)); + button.setScale(ButtonScale.SMALL); } + } - @Override - public void setIconStyle(final String style) { - if (button != null) { - button.setIconStyle(style); - } + @Override + public void setIconStyle(final String style) { + if (button != null) { + button.setIconStyle(style); } + } - @Override - public void setText(final String text) { - if (button != null) { - button.setText(text); - } + @Override + public void setIconUrl(final String url) { + throw new NotImplementedException(); + } + + @Override + public void setText(final String text) { + if (button != null) { + button.setText(text); } + } - @Override - public void setToolTipText(final String tooltip) { - if (button != null) { - if (TextUtils.notEmpty(tooltip)) { - button.setToolTip(new GxtDefTooltip(tooltip)); - } - } + @Override + public void setToolTipText(final String tooltip) { + if (button != null) { + if (TextUtils.notEmpty(tooltip)) { + button.setToolTip(new GxtDefTooltip(tooltip)); + } } + } - @Override - public void setVisible(final boolean visible) { - if (button != null) { - button.setVisible(visible); - } + @Override + public void setVisible(final boolean visible) { + if (button != null) { + button.setVisible(visible); } + } - @Override - public boolean shouldBeAdded() { - return notStandAlone; - } + @Override + public boolean shouldBeAdded() { + return notStandAlone; + } } Modified: trunk/src/main/java/cc/kune/common/client/actions/gxtui/GxtMenuSeparatorGui.java =================================================================== --- trunk/src/main/java/cc/kune/common/client/actions/gxtui/GxtMenuSeparatorGui.java 2011-05-05 01:45:26 UTC (rev 1342) +++ trunk/src/main/java/cc/kune/common/client/actions/gxtui/GxtMenuSeparatorGui.java 2011-05-05 02:24:50 UTC (rev 1343) @@ -28,40 +28,44 @@ public class GxtMenuSeparatorGui extends AbstractChildGuiItem implements ParentWidget { - @Override - public void add(final UIObject uiObject) { - } + @Override + public void add(final UIObject uiObject) { + } - @Override - public AbstractGuiItem create(final GuiActionDescrip descriptor) { - super.create(descriptor); - final GxtMenuGui menu = (GxtMenuGui) parent; - menu.addSeparator(); - return menu; - } + @Override + public AbstractGuiItem create(final GuiActionDescrip descriptor) { + super.create(descriptor); + final GxtMenuGui menu = (GxtMenuGui) parent; + menu.addSeparator(); + return menu; + } - @Override - public void insert(final int position, final UIObject widget) { - } + @Override + public void insert(final int position, final UIObject widget) { + } - @Override - protected void setEnabled(final boolean enabled) { - } + @Override + protected void setEnabled(final boolean enabled) { + } - @Override - protected void setIconStyle(final String style) { - } + @Override + protected void setIconStyle(final String style) { + } - @Override - protected void setText(final String text) { - } + @Override + public void setIconUrl(final String url) { + } - @Override - protected void setToolTipText(final String text) { - } + @Override + protected void setText(final String text) { + } - @Override - public boolean shouldBeAdded() { - return false; - } + @Override + protected void setToolTipText(final String text) { + } + + @Override + public boolean shouldBeAdded() { + return false; + } } Modified: trunk/src/main/java/cc/kune/common/client/actions/gxtui/GxtMenuTitleItemGui.java =================================================================== --- trunk/src/main/java/cc/kune/common/client/actions/gxtui/GxtMenuTitleItemGui.java 2011-05-05 01:45:26 UTC (rev 1342) +++ trunk/src/main/java/cc/kune/common/client/actions/gxtui/GxtMenuTitleItemGui.java 2011-05-05 02:24:50 UTC (rev 1343) @@ -23,73 +23,79 @@ import cc.kune.common.client.actions.ui.AbstractGuiItem; import cc.kune.common.client.actions.ui.descrip.GuiActionDescrip; import cc.kune.common.client.actions.ui.descrip.MenuItemDescriptor; +import cc.kune.common.client.errors.NotImplementedException; import cc.kune.common.client.utils.TextUtils; import com.extjs.gxt.ui.client.widget.menu.HeaderMenuItem; public class GxtMenuTitleItemGui extends AbstractChildGuiItem { - private HeaderMenuItem item; + private HeaderMenuItem item; - public GxtMenuTitleItemGui() { - super(); - } + public GxtMenuTitleItemGui() { + super(); + } - public GxtMenuTitleItemGui(final MenuItemDescriptor descriptor) { - super(descriptor); + public GxtMenuTitleItemGui(final MenuItemDescriptor descriptor) { + super(descriptor); - } + } - @Override - public AbstractGuiItem create(final GuiActionDescrip descriptor) { - super.descriptor = descriptor; - item = new HeaderMenuItem(""); + @Override + public AbstractGuiItem create(final GuiActionDescrip descriptor) { + super.descriptor = descriptor; + item = new HeaderMenuItem(""); - final String id = descriptor.getId(); - if (id != null) { - item.ensureDebugId(id); - } - child = item; - super.create(descriptor); - configureItemFromProperties(); - return this; + final String id = descriptor.getId(); + if (id != null) { + item.ensureDebugId(id); } + child = item; + super.create(descriptor); + configureItemFromProperties(); + return this; + } - public HeaderMenuItem getItem() { - return item; - } + public HeaderMenuItem getItem() { + return item; + } - @Override - protected void setEnabled(final boolean enabled) { - item.setVisible(enabled); - } + @Override + protected void setEnabled(final boolean enabled) { + item.setVisible(enabled); + } - @Override - protected void setIconStyle(final String style) { - item.addStyleName(style); - } + @Override + protected void setIconStyle(final String style) { + item.addStyleName(style); + } - @Override - protected void setText(final String text) { - if (text != null) { - item.setText(text); - } - } + @Override + public void setIconUrl(final String url) { + throw new NotImplementedException(); + } - @Override - protected void setToolTipText(final String tooltip) { - if (TextUtils.notEmpty(tooltip)) { - item.setToolTip(new GxtDefTooltip(tooltip)); - } + @Override + protected void setText(final String text) { + if (text != null) { + item.setText(text); } + } - @Override - public void setVisible(final boolean visible) { - item.setVisible(visible); + @Override + protected void setToolTipText(final String tooltip) { + if (TextUtils.notEmpty(tooltip)) { + item.setToolTip(new GxtDefTooltip(tooltip)); } + } - @Override - public boolean shouldBeAdded() { // NOPMD by vjrj on 18/01/11 0:48 - return false; - } + @Override + public void setVisible(final boolean visible) { + item.setVisible(visible); + } + + @Override + public boolean shouldBeAdded() { // NOPMD by vjrj on 18/01/11 0:48 + return false; + } } Modified: trunk/src/main/java/cc/kune/common/client/actions/gxtui/GxtToolbarGui.java =================================================================== --- trunk/src/main/java/cc/kune/common/client/actions/gxtui/GxtToolbarGui.java 2011-05-05 01:45:26 UTC (rev 1342) +++ trunk/src/main/java/cc/kune/common/client/actions/gxtui/GxtToolbarGui.java 2011-05-05 02:24:50 UTC (rev 1343) @@ -29,69 +29,73 @@ public class GxtToolbarGui extends AbstractGuiItem implements ParentWidget { - private ToolBar toolbar; + private ToolBar toolbar; - public GxtToolbarGui() { - super(); - } + public GxtToolbarGui() { + super(); + } - public GxtToolbarGui(final GuiActionDescrip descriptor) { - super(descriptor); - } + public GxtToolbarGui(final GuiActionDescrip descriptor) { + super(descriptor); + } - @Override - public void add(final UIObject uiObject) { - toolbar.add((Component) uiObject); - } + @Override + public void add(final UIObject uiObject) { + toolbar.add((Component) uiObject); + } - // public FillToolItem addFill() { - // final FillToolItem item = new FillToolItem(); - // toolbar.add(item); - // return item; - // } - // - // public SeparatorToolItem addSeparator() { - // final SeparatorToolItem item = new SeparatorToolItem(); - // toolbar.add(item); - // return item; - // } - // - // public LabelToolItem addSpacer() { - // final LabelToolItem item = new LabelToolItem(); - // toolbar.add(item); - // return item; - // } + // public FillToolItem addFill() { + // final FillToolItem item = new FillToolItem(); + // toolbar.add(item); + // return item; + // } + // + // public SeparatorToolItem addSeparator() { + // final SeparatorToolItem item = new SeparatorToolItem(); + // toolbar.add(item); + // return item; + // } + // + // public LabelToolItem addSpacer() { + // final LabelToolItem item = new LabelToolItem(); + // toolbar.add(item); + // return item; + // } - @Override - public AbstractGuiItem create(final GuiActionDescrip descriptor) { - super.descriptor = descriptor; - toolbar = new ToolBar(); - initWidget(toolbar); - configureItemFromProperties(); - descriptor.putValue(ParentWidget.PARENT_UI, this); - return this; - } + @Override + public AbstractGuiItem create(final GuiActionDescrip descriptor) { + super.descriptor = descriptor; + toolbar = new ToolBar(); + initWidget(toolbar); + configureItemFromProperties(); + descriptor.putValue(ParentWidget.PARENT_UI, this); + return this; + } - @Override - public void insert(final int position, final UIObject uiObject) { - toolbar.insert((Component) uiObject, position); - } + @Override + public void insert(final int position, final UIObject uiObject) { + toolbar.insert((Component) uiObject, position); + } - @Override - protected void setEnabled(final boolean enabled) { - } + @Override + protected void setEnabled(final boolean enabled) { + } - @Override - protected void setIconStyle(final String style) { - } + @Override + protected void setIconStyle(final String style) { + } - @Override - protected void setText(final String text) { - } + @Override + public void setIconUrl(final String url) { + } - @Override - protected void setToolTipText(final String text) { - toolbar.setTitle(text); - } + @Override + protected void setText(final String text) { + } + @Override + protected void setToolTipText(final String text) { + toolbar.setTitle(text); + } + } Modified: trunk/src/main/java/cc/kune/common/client/actions/gxtui/GxtToolbarSeparatorGui.java =================================================================== --- trunk/src/main/java/cc/kune/common/client/actions/gxtui/GxtToolbarSeparatorGui.java 2011-05-05 01:45:26 UTC (rev 1342) +++ trunk/src/main/java/cc/kune/common/client/actions/gxtui/GxtToolbarSeparatorGui.java 2011-05-05 02:24:50 UTC (rev 1343) @@ -31,50 +31,54 @@ public class GxtToolbarSeparatorGui extends AbstractChildGuiItem { - @Override - public AbstractGuiItem create(final GuiActionDescrip descriptor) { - final GxtToolbarGui toolbar = (GxtToolbarGui) parent; - final Type type = ((ToolbarSeparatorDescriptor) descriptor).getSeparatorType(); - switch (type) { - case fill: - child = new FillToolItem(); - break; - case spacer: - child = new LabelToolItem(); - break; - case separator: - child = new SeparatorToolItem(); - break; - default: - break; - } - super.create(descriptor); - configureItemFromProperties(); - return toolbar; + @Override + public AbstractGuiItem create(final GuiActionDescrip descriptor) { + final GxtToolbarGui toolbar = (GxtToolbarGui) parent; + final Type type = ((ToolbarSeparatorDescriptor) descriptor).getSeparatorType(); + switch (type) { + case fill: + child = new FillToolItem(); + break; + case spacer: + child = new LabelToolItem(); + break; + case separator: + child = new SeparatorToolItem(); + break; + default: + break; } + super.create(descriptor); + configureItemFromProperties(); + return toolbar; + } - @Override - protected void setEnabled(final boolean enabled) { - // do nothing - } + @Override + protected void setEnabled(final boolean enabled) { + // do nothing + } - @Override - protected void setIconStyle(final String style) { - // do nothing - } + @Override + protected void setIconStyle(final String style) { + // do nothing + } - @Override - protected void setText(final String text) { - // do nothing - } + @Override + public void setIconUrl(final String url) { + } - @Override - protected void setToolTipText(final String text) { - // do nothing - } + @Override + protected void setText(final String text) { + // do nothing + } - @Override - public boolean shouldBeAdded() { - return false; - } + @Override + protected void setToolTipText(final String text) { + // do nothing + } + + @Override + public boolean shouldBeAdded() { + return false; + } } Modified: trunk/src/main/java/cc/kune/common/client/actions/ui/AbstractGuiItem.java =================================================================== --- trunk/src/main/java/cc/kune/common/client/actions/ui/AbstractGuiItem.java 2011-05-05 01:45:26 UTC (rev 1342) +++ trunk/src/main/java/cc/kune/common/client/actions/ui/AbstractGuiItem.java 2011-05-05 02:24:50 UTC (rev 1343) @@ -35,128 +35,135 @@ public abstract class AbstractGuiItem extends Composite implements GuiBinding { - protected GuiActionDescrip descriptor; + protected GuiActionDescrip descriptor; - public AbstractGuiItem() { - super(); - } + public AbstractGuiItem() { + super(); + } - public AbstractGuiItem(final GuiActionDescrip descriptor) { - super(); - this.descriptor = descriptor; - } + public AbstractGuiItem(final GuiActionDescrip descriptor) { + super(); + this.descriptor = descriptor; + } - protected void addStyle(final String style) { - if (super.isOrWasAttached()) { - super.addStyleName(style); - } + protected void addStyle(final String style) { + if (super.isOrWasAttached()) { + super.addStyleName(style); } + } - protected void clearStyles() { - if (super.isOrWasAttached()) { - super.setStyleName("k-none"); - } + protected void clearStyles() { + if (super.isOrWasAttached()) { + super.setStyleName("k-none"); } + } - private void configure() { - configureProperties(); - final PropertyChangeListener changeListener = createActionPropertyChangeListener(); - descriptor.getAction().addPropertyChangeListener(changeListener); - descriptor.addPropertyChangeListener(changeListener); - } + private void configure() { + configureProperties(); + final PropertyChangeListener changeListener = createActionPropertyChangeListener(); + descriptor.getAction().addPropertyChangeListener(changeListener); + descriptor.addPropertyChangeListener(changeListener); + } - /** - * Sets the item properties from the stored values - */ - public void configureItemFromProperties() { - configure(); - } + /** + * Sets the item properties from the stored values + */ + public void configureItemFromProperties() { + configure(); + } - private void configureProperties() { - setText((String) (descriptor.getValue(Action.NAME))); - setToolTipText((String) (descriptor.getValue(Action.SHORT_DESCRIPTION))); - setIcon(descriptor.getValue(Action.SMALL_ICON)); - setEnabled((Boolean) descriptor.getValue(AbstractAction.ENABLED)); - setVisible((Boolean) descriptor.getValue(GuiActionDescrip.VISIBLE)); - setStyles((String) descriptor.getValue(Action.STYLES)); - } + private void configureProperties() { + setText((String) (descriptor.getValue(Action.NAME))); + setToolTipText((String) (descriptor.getValue(Action.SHORT_DESCRIPTION))); + setIcon(descriptor.getValue(Action.SMALL_ICON)); + setEnabled((Boolean) descriptor.getValue(AbstractAction.ENABLED)); + setVisible((Boolean) descriptor.getValue(GuiActionDescrip.VISIBLE)); + setStyles((String) descriptor.getValue(Action.STYLES)); + } - @Override - public abstract AbstractGuiItem create(final GuiActionDescrip descriptor); + @Override + public abstract AbstractGuiItem create(final GuiActionDescrip descriptor); - private PropertyChangeListener createActionPropertyChangeListener() { - return new PropertyChangeListener() { - @Override - public void propertyChange(final PropertyChangeEvent event) { - final Object newValue = event.getNewValue(); - if (event.getPropertyName().equals(Action.ENABLED)) { - setEnabled((Boolean) newValue); - } else if (event.getPropertyName().equals(Action.NAME)) { - setText((String) newValue); - } else if (event.getPropertyName().equals(Action.SMALL_ICON)) { - setIcon(newValue); - } else if (event.getPropertyName().equals(Action.SHORT_DESCRIPTION)) { - setToolTipText((String) newValue); - } else if (event.getPropertyName().equals(GuiActionDescrip.VISIBLE)) { - setVisible((Boolean) newValue); - } else if (event.getPropertyName().equals(Action.STYLES)) { - setStyles((String) newValue); - } - } - }; - } + private PropertyChangeListener createActionPropertyChangeListener() { + return new PropertyChangeListener() { + @Override + public void propertyChange(final PropertyChangeEvent event) { + final Object newValue = event.getNewValue(); + if (event.getPropertyName().equals(Action.ENABLED)) { + setEnabled((Boolean) newValue); + } else if (event.getPropertyName().equals(Action.NAME)) { + setText((String) newValue); + } else if (event.getPropertyName().equals(Action.SMALL_ICON)) { + setIcon(newValue); + } else if (event.getPropertyName().equals(Action.SHORT_DESCRIPTION)) { + setToolTipText((String) newValue); + } else if (event.getPropertyName().equals(GuiActionDescrip.VISIBLE)) { + setVisible((Boolean) newValue); + } else if (event.getPropertyName().equals(Action.STYLES)) { + setStyles((String) newValue); + } + } + }; + } - protected Object getTargetObjectOfAction(final GuiActionDescrip descriptor) { - // If the action is associated with a item (like a Group, a - // group shortname, a username, etc) we pass this item to - // the action, if not we only pass the menuitem - return descriptor.hasTarget() ? descriptor.getTarget() - : descriptor.isChild() ? descriptor.getParent().getTarget() : ActionEvent.NO_TARGET; - } + protected Object getTargetObjectOfAction(final GuiActionDescrip descriptor) { + // If the action is associated with a item (like a Group, a + // group shortname, a username, etc) we pass this item to + // the action, if not we only pass the menuitem + return descriptor.hasTarget() ? descriptor.getTarget() + : descriptor.isChild() ? descriptor.getParent().getTarget() : ActionEvent.NO_TARGET; + } - protected abstract void setEnabled(boolean enabled); + protected abstract void setEnabled(boolean enabled); - private void setEnabled(final Boolean enabled) { - setEnabled(enabled == null ? true : enabled); - } + private void setEnabled(final Boolean enabled) { + setEnabled(enabled == null ? true : enabled); + } - private void setIcon(final Object icon) { - if (icon instanceof ImageResource) { - setIconResource((ImageResource) icon); - } else if (icon instanceof String) { - setIconStyle((String) icon); - } else if (icon != null) { - throw new NotImplementedException(); - } + private void setIcon(final Object icon) { + if (icon instanceof ImageResource) { + setIconResource((ImageResource) icon); + } else if (icon instanceof String) { + final String iconS = (String) icon; + if (iconS.startsWith("http")) { + setIconUrl(iconS); + } else { + setIconStyle(iconS); + } + } else if (icon != null) { + throw new NotImplementedException(); } + } - public void setIconResource(final ImageResource icon) { - setIconStyle((IconConstants.CSS_SUFFIX + icon.getName())); - } + public void setIconResource(final ImageResource icon) { + setIconStyle((IconConstants.CSS_SUFFIX + icon.getName())); + } - protected abstract void setIconStyle(String style); + protected abstract void setIconStyle(String style); - private void setStyles(final String styles) { - if (styles != null) { - clearStyles(); - for (final String style : TextUtils.splitTags(styles)) { - addStyle(style); - } - } + public abstract void setIconUrl(String url); + + private void setStyles(final String styles) { + if (styles != null) { + clearStyles(); + for (final String style : TextUtils.splitTags(styles)) { + addStyle(style); + } } + } - protected abstract void setText(String text); + protected abstract void setText(String text); - protected abstract void setToolTipText(String text); + protected abstract void setToolTipText(String text); - private void setVisible(final Boolean visible) { - // if you have problems with setVisible check if the GuiItem calls - // configureItemFromProperties on creation - setVisible(visible == null ? true : visible); - } + private void setVisible(final Boolean visible) { + // if you have problems with setVisible check if the GuiItem calls + // configureItemFromProperties on creation + setVisible(visible == null ? true : visible); + } - @Override - public boolean shouldBeAdded() { // NOPMD by vjrj on 18/01/11 0:48 - return true; - } + @Override + public boolean shouldBeAdded() { // NOPMD by vjrj on 18/01/11 0:48 + return true; + } } Modified: trunk/src/main/java/cc/kune/common/client/ui/IconLabel.java =================================================================== --- trunk/src/main/java/cc/kune/common/client/ui/IconLabel.java 2011-05-05 01:45:26 UTC (rev 1342) +++ trunk/src/main/java/cc/kune/common/client/ui/IconLabel.java 2011-05-05 02:24:50 UTC (rev 1343) @@ -43,6 +43,8 @@ interface IconTitleUiBinder extends UiBinder<Widget, IconLabel> { } + private static final String DEF = "16px"; + private static IconTitleUiBinder uiBinder = GWT.create(IconTitleUiBinder.class); @UiField @@ -137,6 +139,12 @@ iconLeft.addStyleName("k-iconlabel-left"); } + public void setLeftIconUrl(final String url) { + iconLeft.setUrl(url); + iconLeft.setSize(DEF, DEF); + iconLeft.addStyleName("k-iconlabel-left"); + } + public void setRightIcon(final String imgCss) { iconRight.setUrl("images/clear.gif"); iconRight.setStyleName(imgCss); @@ -149,6 +157,12 @@ iconRight.addStyleName("k-iconlabel-right"); } + public void setRightIconUrl(final String url) { + iconRight.setUrl(url); + iconRight.setSize(DEF, DEF); + iconRight.addStyleName("k-iconlabel-right"); + } + @Override public void setStyleName(final String style) { flow.setStyleName(style); Modified: trunk/src/main/java/cc/kune/common/public/kune-common.css =================================================================== --- trunk/src/main/java/cc/kune/common/public/kune-common.css 2011-05-05 01:45:26 UTC (rev 1342) +++ trunk/src/main/java/cc/kune/common/public/kune-common.css 2011-05-05 02:24:50 UTC (rev 1343) @@ -449,6 +449,7 @@ .k-iconlabel-text { vertical-align: top; padding-left: 3px; + padding-right: 3px; float: left; } Modified: trunk/src/main/java/cc/kune/core/client/services/FileDownloadUtils.java =================================================================== --- trunk/src/main/java/cc/kune/core/client/services/FileDownloadUtils.java 2011-05-05 01:45:26 UTC (rev 1342) +++ trunk/src/main/java/cc/kune/core/client/services/FileDownloadUtils.java 2011-05-05 02:24:50 UTC (rev 1343) @@ -23,6 +23,8 @@ import cc.kune.common.client.utils.UrlParam; import cc.kune.core.client.state.Session; import cc.kune.core.shared.domain.utils.StateToken; +import cc.kune.core.shared.dto.GroupDTO; +import cc.kune.core.shared.dto.UserSimpleDTO; import com.google.gwt.core.client.GWT; import com.google.gwt.user.client.DOM; @@ -31,63 +33,75 @@ public class FileDownloadUtils { - private static final String DOWNLOADSERVLET = "servlets/FileDownloadManager"; - private static final String LOGODOWNLOADSERVLET = "servlets/EntityLogoDownloadManager"; + private static final String DOWNLOADSERVLET = "servlets/FileDownloadManager"; + protected static final String GROUP_NO_AVATAR_IMAGE = "images/group-def-icon.png"; + private static final String LOGODOWNLOADSERVLET = "servlets/EntityLogoDownloadManager"; + protected static final String PERSON_NO_AVATAR_IMAGE = "images/unknown.jpg"; - private final ImageUtils imageUtils; - private final Session session; + private final ImageUtils imageUtils; + private final Session session; - @Inject - public FileDownloadUtils(final Session session, final ImageUtils imageUtils) { - this.session = session; - this.imageUtils = imageUtils; - } + @Inject + public FileDownloadUtils(final Session session, final ImageUtils imageUtils) { + this.session = session; + this.imageUtils = imageUtils; + } - private String calculateUrl(final StateToken token, final boolean download, final boolean useHash) { - final Url url = new Url(GWT.getModuleBaseURL() + DOWNLOADSERVLET, new UrlParam(FileConstants.TOKEN, - token.toString())); - if (download) { - url.add(new UrlParam(FileConstants.DOWNLOAD, download)); - } - if (useHash) { - final String hash = session.getUserHash(); - if (hash != null) { - url.add(new UrlParam(FileConstants.HASH, hash)); - } - } - return url.toString(); + private String calculateUrl(final StateToken token, final boolean download, final boolean useHash) { + final Url url = new Url(GWT.getModuleBaseURL() + DOWNLOADSERVLET, new UrlParam(FileConstants.TOKEN, + token.toString())); + if (download) { + url.add(new UrlParam(FileConstants.DOWNLOAD, download)); } - - public void downloadFile(final StateToken token) { - final String url = calculateUrl(token, true, true); - DOM.setElementAttribute(RootPanel.get("__download").getElement(), "src", url); + if (useHash) { + final String hash = session.getUserHash(); + if (hash != null) { + url.add(new UrlParam(FileConstants.HASH, hash)); + } } + return url.toString(); + } - public String getImageResizedUrl(final StateToken token, final ImageSize imageSize) { - return calculateUrl(token, false, true) + "&" + new UrlParam(FileConstants.IMGSIZE, imageSize.toString()); - } + public void downloadFile(final StateToken token) { + final String url = calculateUrl(token, true, true); + DOM.setElementAttribute(RootPanel.get("__download").getElement(), "src", url); + } - public String getImageUrl(final StateToken token) { - return calculateUrl(token, false, true); - } + public String getGroupLogo(final GroupDTO group) { + return group.hasLogo() ? getLogoImageUrl(group.getStateToken()) + : group.isPersonal() ? PERSON_NO_AVATAR_IMAGE : GROUP_NO_AVATAR_IMAGE; + } - public String getLogoAvatarHtml(final StateToken groupToken, final boolean groupHasLogo, final boolean isPersonal, - final int size, final int hvspace) { - if (groupHasLogo) { - return "<img hspace='" + hvspace + "' vspace='" + hvspace + "' align='left' style='width: " + size - + "px; height: " + size + "px;' src='" + getLogoImageUrl(groupToken) + "'>"; - } else { - return isPersonal ? imageUtils.getImageHtml(ImageDescriptor.personDef) - : imageUtils.getImageHtml(ImageDescriptor.groupDefIcon); - } - } + public String getImageResizedUrl(final StateToken token, final ImageSize imageSize) { + return calculateUrl(token, false, true) + "&" + + new UrlParam(FileConstants.IMGSIZE, imageSize.toString()); + } - public String getLogoImageUrl(final StateToken token) { - return new Url(GWT.getModuleBaseURL() + LOGODOWNLOADSERVLET, - new UrlParam(FileConstants.TOKEN, token.toString())).toString(); + public String getImageUrl(final StateToken token) { + return calculateUrl(token, false, true); + } + + public String getLogoAvatarHtml(final StateToken groupToken, final boolean groupHasLogo, + final boolean isPersonal, final int size, final int hvspace) { + if (groupHasLogo) { + return "<img hspace='" + hvspace + "' vspace='" + hvspace + "' align='left' style='width: " + size + + "px; height: " + size + "px;' src='" + getLogoImageUrl(groupToken) + "'>"; + } else { + return isPersonal ? imageUtils.getImageHtml(ImageDescriptor.personDef) + : imageUtils.getImageHtml(ImageDescriptor.groupDefIcon); } + } - public String getUrl(final StateToken token) { - return calculateUrl(token, false, false); - } + public String getLogoImageUrl(final StateToken token) { + return new Url(GWT.getModuleBaseURL() + LOGODOWNLOADSERVLET, new UrlParam(FileConstants.TOKEN, + token.toString())).toString(); + } + + public String getUrl(final StateToken token) { + return calculateUrl(token, false, false); + } + + public String getUserAvatar(final UserSimpleDTO user) { + return user.hasLogo() ? getLogoImageUrl(user.getStateToken()) : PERSON_NO_AVATAR_IMAGE; + } } Modified: trunk/src/main/java/cc/kune/core/client/sitebar/SiteUserOptionsPresenter.java =================================================================== --- trunk/src/main/java/cc/kune/core/client/sitebar/SiteUserOptionsPresenter.java 2011-05-05 01:45:26 UTC (rev 1342) +++ trunk/src/main/java/cc/kune/core/client/sitebar/SiteUserOptionsPresenter.java 2011-05-05 02:24:50 UTC (rev 1343) @@ -104,8 +104,7 @@ } private void addPartipation(final GroupDTO group) { - final String logoImageUrl = group.hasLogo() ? downloadProvider.get().getLogoImageUrl( - group.getStateToken()) : "images/group-def-icon.gif"; + final String logoImageUrl = downloadProvider.get().getGroupLogo(group); final MenuItemDescriptor participant = new MenuItemDescriptor(gotoGroupAction); participant.setTarget(group); participant.putValue(Action.NAME, group.getLongName()); Modified: trunk/src/main/java/cc/kune/core/client/sn/AbstractSNPresenter.java =================================================================== --- trunk/src/main/java/cc/kune/core/client/sn/AbstractSNPresenter.java 2011-05-05 01:45:26 UTC (rev 1342) +++ trunk/src/main/java/cc/kune/core/client/sn/AbstractSNPresenter.java 2011-05-05 02:24:50 UTC (rev 1343) @@ -13,32 +13,31 @@ import com.gwtplatform.mvp.client.View; import com.gwtplatform.mvp.client.proxy.Proxy; -public abstract class AbstractSNPresenter<V extends View, Proxy_ extends Proxy<?>> extends Presenter<View, Proxy<?>> { - protected static final String GROUP_NO_AVATAR_IMAGE = "images/group-def-icon.png"; - protected static final String PERSON_NO_AVATAR_IMAGE = "images/unknown.jpg"; - protected final Provider<FileDownloadUtils> downloadProvider; +public abstract class AbstractSNPresenter<V extends View, Proxy_ extends Proxy<?>> extends + Presenter<View, Proxy<?>> { - public AbstractSNPresenter(final EventBus eventBus, final View view, final Proxy<?> proxy, - final Provider<FileDownloadUtils> downloadProvider) { - super(eventBus, view, proxy); - this.downloadProvider = downloadProvider; - } + protected final Provider<FileDownloadUtils> downloadProvider; - protected GuiActionDescCollection createMenuItems(final Object target, - final AbstractSNMembersActionsRegistry registry, final String title) { - final GuiActionDescCollection items = new GuiActionDescCollection(); - items.add(new MenuTitleItemDescriptor(title)); - for (final Provider<MenuItemDescriptor> provider : registry) { - final MenuItemDescriptor menuItem = provider.get(); - menuItem.setTarget(target); - items.add(menuItem); - } - return items; - } + public AbstractSNPresenter(final EventBus eventBus, final View view, final Proxy<?> proxy, + final Provider<FileDownloadUtils> downloadProvider) { + super(eventBus, view, proxy); + this.downloadProvider = downloadProvider; + } - protected String getAvatar(final GroupDTO group) { - return group.hasLogo() ? downloadProvider.get().getLogoImageUrl(group.getStateToken()) - : group.isPersonal() ? PERSON_NO_AVATAR_IMAGE : GROUP_NO_AVATAR_IMAGE; + protected GuiActionDescCollection createMenuItems(final Object target, + final AbstractSNMembersActionsRegistry registry, final String title) { + final GuiActionDescCollection items = new GuiActionDescCollection(); + items.add(new MenuTitleItemDescriptor(title)); + for (final Provider<MenuItemDescriptor> provider : registry) { + final MenuItemDescriptor menuItem = provider.get(); + menuItem.setTarget(target); + items.add(menuItem); } + return items; + } + protected String getAvatar(final GroupDTO group) { + return downloadProvider.get().getGroupLogo(group); + } + } Modified: trunk/src/main/java/cc/kune/core/client/sn/UserSNPresenter.java =================================================================== --- trunk/src/main/java/cc/kune/core/client/sn/UserSNPresenter.java 2011-05-05 01:45:26 UTC (rev 1342) +++ trunk/src/main/java/cc/kune/core/client/sn/UserSNPresenter.java 2011-05-05 02:24:50 UTC (rev 1343) @@ -152,8 +152,7 @@ final UserBuddiesDataDTO userBuddiesData = state.getUserBuddies(); final List<UserSimpleDTO> buddies = userBuddiesData.getBuddies(); for (final UserSimpleDTO user : buddies) { - final String avatarUrl = user.hasLogo() ? downloadProvider.get().getLogoImageUrl( - user.getStateToken()) : PERSON_NO_AVATAR_IMAGE; + final String avatarUrl = downloadProvider.get().getUserAvatar(user); getView().addBuddie(user, avatarUrl, user.getName(), "", createMenuItems(user, userMenuItemsRegistry, user.getName())); } Modified: trunk/src/main/java/cc/kune/core/client/sn/actions/registry/GroupSNConfActions.java =================================================================== --- trunk/src/main/java/cc/kune/core/client/sn/actions/registry/GroupSNConfActions.java 2011-05-05 01:45:26 UTC (rev 1342) +++ trunk/src/main/java/cc/kune/core/client/sn/actions/registry/GroupSNConfActions.java 2011-05-05 02:24:50 UTC (rev 1343) @@ -29,80 +29,84 @@ @SuppressWarnings("serial") public class GroupSNConfActions extends AbstractSNActionsRegistry { - public static final SubMenuDescriptor MODERATION_SUBMENU = new SubMenuDescriptor(); - public static final MenuDescriptor OPTIONS_MENU = new MenuDescriptor(); - public static final SubMenuDescriptor VISIBILITY_SUBMENU = new SubMenuDescriptor(); + public static final SubMenuDescriptor MODERATION_SUBMENU = new SubMenuDescriptor(); + public static final MenuDescriptor OPTIONS_MENU = new MenuDescriptor(); + public static final String OPTIONS_STYLES = "k-sn-options-menu, k-noborder, k-nobackcolor, k-btn-min"; + public static final SubMenuDescriptor VISIBILITY_SUBMENU = new SubMenuDescriptor(); - @Inject - public GroupSNConfActions(final Session session, final StateManager stateManager, - final I18nTranslationService i18n, final Provider<MembersVisibilityMenuItem> membersVisibility, - final Provider<MembersModerationMenuItem> membersModeration, final CoreResources res, - final IsLoggedCondition isLoggedCondition, final JoinGroupAction joinGroupAction, - final IsGroupCondition isGroupCondition, final UnJoinGroupAction unJoinGroupAction) { - OPTIONS_MENU.withToolTip(i18n.t("Options")).withIcon(res.arrowdownsitebar()).withStyles( - "k-sn-options-menu, k-noborder, k-nobackcolor"); - final GuiActionDescrip optionsTitle = new MenuTitleItemDescriptor(i18n.t("Options")).withParent(OPTIONS_MENU); - final MenuRadioItemDescriptor anyoneItem = membersVisibility.get().withVisibility( - SocialNetworkVisibility.anyone); - final MenuRadioItemDescriptor onlyMembersItem = membersVisibility.get().withVisibility( - SocialNetworkVisibility.onlymembers); - final MenuRadioItemDescriptor onlyAdminsItem = membersVisibility.get().withVisibility( - SocialNetworkVisibility.onlyadmins); - final MenuRadioItemDescriptor closedItem = membersModeration.get().withModeration(AdmissionType.Closed); - final MenuRadioItemDescriptor moderatedItem = membersModeration.get().withModeration(AdmissionType.Moderated); - final MenuRadioItemDescriptor openItem = membersModeration.get().withModeration(AdmissionType.Open); - add(OPTIONS_MENU); - add(optionsTitle); - add(VISIBILITY_SUBMENU.withText(i18n.t("Those who can view this member list")).withParent(OPTIONS_MENU)); - add(MODERATION_SUBMENU.withText(i18n.t("New members policy")).withParent(OPTIONS_MENU)); - add(anyoneItem.withParent(VISIBILITY_SUBMENU).withText(i18n.t("anyone"))); - add(onlyMembersItem.withParent(VISIBILITY_SUBMENU).withText(i18n.t("only members"))); - add(onlyAdminsItem.withParent(VISIBILITY_SUBMENU).withText(i18n.t("only admins"))); - add(moderatedItem.withParent(MODERATION_SUBMENU).withText(i18n.t("moderate request to join"))); - add(openItem.withParent(MODERATION_SUBMENU).withText(i18n.t("auto accept request to join"))); - // add(closedItem.withParent(MODERATION_SUBMENU).withText( - // i18n.t("closed for new members"))); + @Inject + public GroupSNConfActions(final Session session, final StateManager stateManager, + final I18nTranslationService i18n, final Provider<MembersVisibilityMenuItem> membersVisibility, + final Provider<MembersModerationMenuItem> membersModeration, final CoreResources res, + final IsLoggedCondition isLoggedCondition, final JoinGroupAction joinGroupAction, + final IsGroupCondition isGroupCondition, final UnJoinGroupAction unJoinGroupAction) { + OPTIONS_MENU.withToolTip(i18n.t("Options")).withIcon(res.arrowdownsitebar()).withStyles( + OPTIONS_STYLES); + final GuiActionDescrip optionsTitle = new MenuTitleItemDescriptor(i18n.t("Options")).withParent(OPTIONS_MENU); + final MenuRadioItemDescriptor anyoneItem = membersVisibility.get().withVisibility( + SocialNetworkVisibility.anyone); + final MenuRadioItemDescriptor onlyMembersItem = membersVisibility.get().withVisibility( + SocialNetworkVisibility.onlymembers); + final MenuRadioItemDescriptor onlyAdminsItem = membersVisibility.get().withVisibility( + SocialNetworkVisibility.onlyadmins); + final MenuRadioItemDescriptor closedItem = membersModeration.get().withModeration( + AdmissionType.Closed); + final MenuRadioItemDescriptor moderatedItem = membersModeration.get().withModeration( + AdmissionType.Moderated); + final MenuRadioItemDescriptor openItem = membersModeration.get().withModeration(AdmissionType.Open); + add(OPTIONS_MENU); + add(optionsTitle); + add(VISIBILITY_SUBMENU.withText(i18n.t("Those who can view this member list")).withParent( + OPTIONS_MENU)); + add(MODERATION_SUBMENU.withText(i18n.t("New members policy")).withParent(OPTIONS_MENU)); + add(anyoneItem.withParent(VISIBILITY_SUBMENU).withText(i18n.t("anyone"))); + add(onlyMembersItem.withParent(VISIBILITY_SUBMENU).withText(i18n.t("only members"))); + add(onlyAdminsItem.withParent(VISIBILITY_SUBMENU).withText(i18n.t("only admins"))); + add(moderatedItem.withParent(MODERATION_SUBMENU).withText(i18n.t("moderate request to join"))); + add(openItem.withParent(MODERATION_SUBMENU).withText(i18n.t("auto accept request to join"))); + // add(closedItem.withParent(MODERATION_SUBMENU).withText( + // i18n.t("closed for new members"))); - final ButtonDescriptor joinBtn = new ButtonDescriptor(joinGroupAction); - final ButtonDescriptor unJoinBtn = new ButtonDescriptor(unJoinGroupAction); - // unJoinBtn.add(isLoggedCondition); - add(joinBtn); // .withStyles("k-no-backimage, k-noborder, k-nobackcolor")); - add(unJoinBtn); // .withStyles("k-no-backimage, k-noborder, k-nobackcolor")); + final ButtonDescriptor joinBtn = new ButtonDescriptor(joinGroupAction); + final ButtonDescriptor unJoinBtn = new ButtonDescriptor(unJoinGroupAction); + // unJoinBtn.add(isLoggedCondition); + add(joinBtn); // .withStyles("k-no-backimage, k-noborder, k-nobackcolor")); + add(unJoinBtn); // .withStyles("k-no-backimage, k-noborder, k-nobackcolor")); - stateManager.onStateChanged(true, new StateChangedHandler() { - @Override - public void onStateChanged(final StateChangedEvent event) { - final boolean administrable = event.getState().getGroupRights().isAdministrable(); - OPTIONS_MENU.setVisible(administrable); - OPTIONS_MENU.setEnabled(administrable); - final StateAbstractDTO state = event.getState(); - final GroupDTO currentGroup = state.getGroup(); - if (currentGroup.isNotPersonal()) { - switch (state.getSocialNetworkData().getSocialNetworkVisibility()) { - case anyone: - anyoneItem.setChecked(true); - break; - case onlyadmins: - onlyAdminsItem.setChecked(true); - break; - case onlymembers: - onlyMembersItem.setChecked(true); - break; - } - } - switch (currentGroup.getAdmissionType()) { - case Moderated: - moderatedItem.setChecked(true); - break; - case Open: - openItem.setChecked(true); - break; - case Closed: - closedItem.setChecked(true); - break; - } - unJoinBtn.setVisible(session.isLogged()); - } - }); - } + stateManager.onStateChanged(true, new StateChangedHandler() { + @Override + public void onStateChanged(final StateChangedEvent event) { + final boolean administrable = event.getState().getGroupRights().isAdministrable(); + OPTIONS_MENU.setVisible(administrable); + OPTIONS_MENU.setEnabled(administrable); + final StateAbstractDTO state = event.getState(); + final GroupDTO currentGroup = state.getGroup(); + if (currentGroup.isNotPersonal()) { + switch (state.getSocialNetworkData().getSocialNetworkVisibility()) { + case anyone: + anyoneItem.setChecked(true); + break; + case onlyadmins: + onlyAdminsItem.setChecked(true); + break; + case onlymembers: + onlyMembersItem.setChecked(true); + break; + } + } + switch (currentGroup.getAdmissionType()) { + case Moderated: + moderatedItem.setChecked(true); + break; + case Open: + openItem.setChecked(true); + break; + case Closed: + closedItem.setChecked(true); + break; + } + unJoinBtn.setVisible(session.isLogged()); + } + }); + } } Modified: trunk/src/main/java/cc/kune/core/client/sn/actions/registry/UserSNConfActions.java =================================================================== --- trunk/src/main/java/cc/kune/core/client/sn/actions/registry/UserSNConfActions.java 2011-05-05 01:45:26 UTC (rev 1342) +++ trunk/src/main/java/cc/kune/core/client/sn/actions/registry/UserSNConfActions.java 2011-05-05 02:24:50 UTC (rev 1343) @@ -22,57 +22,60 @@ @SuppressWarnings("serial") public class UserSNConfActions extends AbstractSNActionsRegistry { - public static final MenuDescriptor OPTIONS_MENU = new MenuDescriptor(); - public static final SubMenuDescriptor VISIBILITY_SUBMENU = new SubMenuDescriptor(); + public static final MenuDescriptor OPTIONS_MENU = new MenuDescriptor(); + public static final SubMenuDescriptor VISIBILITY_SUBMENU = new SubMenuDescriptor(); - @Inject - public UserSNConfActions(final Session session, final StateManager stateManager, final I18nTranslationService i18n, - final Provider<UserSNVisibilityMenuItem> userBuddiesVisibility, final CoreResources res, - final IsGroupCondition isGroupCondition, final AddNewBuddiesAction addNewBuddiesAction) { - OPTIONS_MENU.withToolTip(i18n.t("Options")).withIcon(res.arrowdownsitebar()).withStyles( - "k-sn-options-menu, k-noborder, k-nobackcolor"); - final MenuRadioItemDescriptor anyoneItem = userBuddiesVisibility.get().withVisibility(UserSNetVisibility.anyone); - final MenuRadioItemDescriptor onlyYourBuddiesItem = userBuddiesVisibility.get().withVisibility( - UserSNetVisibility.yourbuddies); - final MenuRadioItemDescriptor onlyYou = userBuddiesVisibility.get().withVisibility(UserSNetVisibility.onlyyou); - assert anyoneItem.getAction() != onlyYourBuddiesItem.getAction(); - assert anyoneItem.getAction() != onlyYou.getAction(); - add(OPTIONS_MENU); - add(VISIBILITY_SUBMENU.withText(i18n.t("Those who can view your network")).withParent(OPTIONS_MENU)); - add(anyoneItem.withParent(VISIBILITY_SUBMENU).withText(i18n.t("anyone"))); - add(onlyYourBuddiesItem.withParent(VISIBILITY_SUBMENU).withText(i18n.t("only your buddies"))); - add(onlyYou.withParent(VISIBILITY_SUBMENU).withText(i18n.t("only you"))); + @Inject + public UserSNConfActions(final Session session, final StateManager stateManager, + final I18nTranslationService i18n, final Provider<UserSNVisibilityMenuItem> userBuddiesVisibility, + final CoreResources res, final IsGroupCondition isGroupCondition, + final AddNewBuddiesAction addNewBuddiesAction) { + OPTIONS_MENU.withToolTip(i18n.t("Options")).withIcon(res.arrowdownsitebar()).withStyles( + GroupSNConfActions.OPTIONS_STYLES); + final MenuRadioItemDescriptor anyoneItem = userBuddiesVisibility.get().withVisibility( + UserSNetVisibility.anyone); + final MenuRadioItemDescriptor onlyYourBuddiesItem = userBuddiesVisibility.get().withVisibility( + UserSNetVisibility.yourbuddies); + final MenuRadioItemDescriptor onlyYou = userBuddiesVisibility.get().withVisibility( + UserSNetVisibility.onlyyou); + assert anyoneItem.getAction() != onlyYourBuddiesItem.getAction(); + assert anyoneItem.getAction() != onlyYou.getAction(); + add(OPTIONS_MENU); + add(VISIBILITY_SUBMENU.withText(i18n.t("Those who can view your network")).withParent(OPTIONS_MENU)); + add(anyoneItem.withParent(VISIBILITY_SUBMENU).withText(i18n.t("anyone"))); + add(onlyYourBuddiesItem.withParent(VISIBILITY_SUBMENU).withText(i18n.t("only your buddies"))); + add(onlyYou.withParent(VISIBILITY_SUBMENU).withText(i18n.t("only you"))); - final ButtonDescriptor addBuddieBtn = new ButtonDescriptor(addNewBuddiesAction); + final ButtonDescriptor addBuddieBtn = new ButtonDescriptor(addNewBuddiesAction); - add(addBuddieBtn.withStyles("k-no-backimage")); + add(addBuddieBtn.withStyles("k-no-backimage")); - stateManager.onStateChanged(true, new StateChangedHandler() { - @Override - public void onStateChanged(final StateChangedEvent event) { - final StateAbstractDTO state = event.getState(); - final boolean administrable = state.getGroupRights().isAdministrable(); - OPTIONS_MENU.setVisible(administrable); - OPTIONS_MENU.setEnabled(administrable); - if (state.getGroup().isPersonal()) { - final UserSNetVisibility visibility = state.getSocialNetworkData().getUserBuddiesVisibility(); - switch (visibility) { - case anyone: - anyoneItem.setChecked(true); - break; - case yourbuddies: - onlyYourBuddiesItem.setChecked(true); - break; - case onlyyou: - onlyYou.setChecked(true); - break; - } - // NotifyUser.info(i18n.t("Visibility of your network is " + - // visibility.toString())); - } - addBuddieBtn.setVisible(session.isLogged() && state.getGroup().isPersonal() - && session.getCurrentUser().getShortName().equals(state.getGroup().getShortName())); - } - }); - } + stateManager.onStateChanged(true, new StateChangedHandler() { + @Override + public void onStateChanged(final StateChangedEvent event) { + final StateAbstractDTO state = event.getState(); + final boolean administrable = state.getGroupRights().isAdministrable(); + OPTIONS_MENU.setVisible(administrable); + OPTIONS_MENU.setEnabled(administrable); + if (state.getGroup().isPersonal()) { + final UserSNetVisibility visibility = state.getSocialNetworkData().getUserBuddiesVisibility(); + switch (visibility) { + case anyone: + anyoneItem.setChecked(true); + break; + case yourbuddies: + onlyYourBuddiesItem.setChecked(true); + break; + case onlyyou: + onlyYou.setChecked(true); + break; + } + // NotifyUser.info(i18n.t("Visibility of your network is " + + // visibility.toString())); + } + addBuddieBtn.setVisible(session.isLogged() && state.getGroup().isPersonal() + && session.getCurrentUser().getShortName().equals(state.getGroup().getShortName())); + } + }); + } } Modified: trunk/src/main/java/cc/kune/wave/client/WebClient.java =================================================================== --- trunk/src/main/java/cc/kune/wave/client/WebClient.java 2011-05-05 01:45:26 UTC (rev 1342) +++ trunk/src/main/java/cc/kune/wave/client/WebClient.java 2011-05-05 02:24:50 UTC (rev 1343) @@ -358,7 +358,7 @@ getStackTraceAsync(e, new Accessor<SafeHtml>() { @Override public void use(SafeHtml stack) { - NotifyUser.error("Oops! Something has gone wrong. Please contact the site admins with <em>more details</em>", true); + NotifyUser.error("Oops! Something has gone wrong. Please contact the site admins with <em>more details</em>"); // error.addDetail(stack, null); String message = stack.asString().replace("<br>", "\n"); REMOTE_LOG.severe(message);