projects
/
lgpl
/
argeo-commons.git
/ blobdiff
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Improve forms framework.
[lgpl/argeo-commons.git]
/
org.argeo.cms.ui
/
src
/
org
/
argeo
/
cms
/
ui
/
forms
/
FormPageViewer.java
diff --git
a/org.argeo.cms.ui/src/org/argeo/cms/ui/forms/FormPageViewer.java
b/org.argeo.cms.ui/src/org/argeo/cms/ui/forms/FormPageViewer.java
index 8f0685280232ddb54cdfc3692f06ba588c42e6fa..72b09e28a6da0879a6e11dea79cb44c3a6407916 100644
(file)
--- a/
org.argeo.cms.ui/src/org/argeo/cms/ui/forms/FormPageViewer.java
+++ b/
org.argeo.cms.ui/src/org/argeo/cms/ui/forms/FormPageViewer.java
@@
-16,9
+16,9
@@
import javax.jcr.ValueFormatException;
import org.apache.commons.logging.Log;
import org.apache.commons.logging.LogFactory;
import org.apache.commons.logging.Log;
import org.apache.commons.logging.LogFactory;
-import org.argeo.cms.CmsException;
import org.argeo.cms.ui.CmsEditable;
import org.argeo.cms.ui.CmsImageManager;
import org.argeo.cms.ui.CmsEditable;
import org.argeo.cms.ui.CmsImageManager;
+import org.argeo.cms.ui.CmsView;
import org.argeo.cms.ui.util.CmsUiUtils;
import org.argeo.cms.ui.viewers.AbstractPageViewer;
import org.argeo.cms.ui.viewers.EditablePart;
import org.argeo.cms.ui.util.CmsUiUtils;
import org.argeo.cms.ui.viewers.AbstractPageViewer;
import org.argeo.cms.ui.viewers.EditablePart;
@@
-28,6
+28,7
@@
import org.argeo.cms.ui.widgets.EditableImage;
import org.argeo.cms.ui.widgets.Img;
import org.argeo.cms.ui.widgets.StyledControl;
import org.argeo.eclipse.ui.EclipseUiUtils;
import org.argeo.cms.ui.widgets.Img;
import org.argeo.cms.ui.widgets.StyledControl;
import org.argeo.eclipse.ui.EclipseUiUtils;
+import org.argeo.jcr.JcrException;
import org.argeo.jcr.JcrUtils;
import org.eclipse.jface.dialogs.MessageDialog;
import org.eclipse.rap.fileupload.FileDetails;
import org.argeo.jcr.JcrUtils;
import org.eclipse.jface.dialogs.MessageDialog;
import org.eclipse.rap.fileupload.FileDetails;
@@
-67,7
+68,7
@@
public class FormPageViewer extends AbstractPageViewer {
private final Section mainSection;
// TODO manage within the CSS
private final Section mainSection;
// TODO manage within the CSS
- private
int labelColWidth = 150
;
+ private
Integer labelColWidth = null
;
private int rowLayoutHSpacing = 8;
// Context cached in the viewer
private int rowLayoutHSpacing = 8;
// Context cached in the viewer
@@
-204,7
+205,7
@@
public class FormPageViewer extends AbstractPageViewer {
}
public void uploadFailed(FileUploadEvent event) {
}
public void uploadFailed(FileUploadEvent event) {
- throw new
Cms
Exception("Upload failed " + event, event.getException());
+ throw new
IllegalState
Exception("Upload failed " + event, event.getException());
}
public void uploadFinished(FileUploadEvent event) {
}
public void uploadFinished(FileUploadEvent event) {
@@
-305,7
+306,7
@@
public class FormPageViewer extends AbstractPageViewer {
protected CmsImageManager imageManager() {
if (imageManager == null)
protected CmsImageManager imageManager() {
if (imageManager == null)
- imageManager = Cms
UiUtils.getCmsView(
).getImageManager();
+ imageManager = Cms
View.getCmsView(mainSection
).getImageManager();
return imageManager;
}
return imageManager;
}
@@
-357,16
+358,17
@@
public class FormPageViewer extends AbstractPageViewer {
}
protected Label createPropertyLbl(Composite parent, String value) {
}
protected Label createPropertyLbl(Composite parent, String value) {
- return createPropertyLbl(parent, value, SWT.
TOP
);
+ return createPropertyLbl(parent, value, SWT.
NONE
);
}
protected Label createPropertyLbl(Composite parent, String value, int vAlign) {
}
protected Label createPropertyLbl(Composite parent, String value, int vAlign) {
-
boolean isSmall = CmsUiUtils.getCmsView(
).getUxContext().isSmall();
- Label label = new Label(parent,
isSmall ? SWT.LEFT : SWT.RIGHT
| SWT.WRAP);
+
// boolean isSmall = CmsView.getCmsView(parent
).getUxContext().isSmall();
+ Label label = new Label(parent,
SWT.LEAD
| SWT.WRAP);
label.setText(value + " ");
CmsUiUtils.style(label, FormStyle.propertyLabel.style());
label.setText(value + " ");
CmsUiUtils.style(label, FormStyle.propertyLabel.style());
- GridData gd = new GridData(isSmall ? SWT.LEFT : SWT.RIGHT, vAlign, false, false);
- gd.widthHint = labelColWidth;
+ GridData gd = new GridData(SWT.LEAD, vAlign, false, false);
+ if (labelColWidth != null)
+ gd.widthHint = labelColWidth;
label.setLayoutData(gd);
return label;
}
label.setLayoutData(gd);
return label;
}
@@
-457,12
+459,12
@@
public class FormPageViewer extends AbstractPageViewer {
section.layout();
section.getParent().layout();
} catch (RepositoryException re) {
section.layout();
section.getParent().layout();
} catch (RepositoryException re) {
- throw new
CmsException("unable to refresh " + "image section for " + context
);
+ throw new
JcrException("Unable to refresh " + "image section for " + context, re
);
}
}
});
} catch (RepositoryException re) {
}
}
});
} catch (RepositoryException re) {
- throw new
CmsException("unable to upload image " + name + " at " + context
);
+ throw new
JcrException("unable to upload image " + name + " at " + context, re
);
}
}
}
}
}
}
@@
-529,7
+531,7
@@
public class FormPageViewer extends AbstractPageViewer {
refresh(parSection);
layout(parSection);
} catch (RepositoryException re) {
refresh(parSection);
layout(parSection);
} catch (RepositoryException re) {
- throw new
Cms
Exception("Unable to delete " + sessionNode, re);
+ throw new
Jcr
Exception("Unable to delete " + sessionNode, re);
}
}
}
}
@@
-577,7
+579,7
@@
public class FormPageViewer extends AbstractPageViewer {
edit(emsp, 0);
cancelEdit();
} catch (RepositoryException e1) {
edit(emsp, 0);
cancelEdit();
} catch (RepositoryException e1) {
- throw new
Cms
Exception("Unable to remove value " + obj, e1);
+ throw new
Jcr
Exception("Unable to remove value " + obj, e1);
}
layout(emsp);
}
}
layout(emsp);
}