diff --git a/openbis/source/java/ch/systemsx/cisd/openbis/generic/client/web/client/application/ui/columns/framework/renderers/SimpleImageHtmlRenderer.java b/openbis/source/java/ch/systemsx/cisd/openbis/generic/client/web/client/application/ui/columns/framework/renderers/SimpleImageHtmlRenderer.java index 02aec0623e6b154e406f842ba22e8e5d04c4aeed..378740d457df2cb51e44ed3c2c3cc923612795d8 100644 --- a/openbis/source/java/ch/systemsx/cisd/openbis/generic/client/web/client/application/ui/columns/framework/renderers/SimpleImageHtmlRenderer.java +++ b/openbis/source/java/ch/systemsx/cisd/openbis/generic/client/web/client/application/ui/columns/framework/renderers/SimpleImageHtmlRenderer.java @@ -34,18 +34,7 @@ public class SimpleImageHtmlRenderer methodWithParameters.addParameter("mode", "thumbnail" + width + "x" + height); String imageURL = methodWithParameters.toString(); - return createEmbededImageHtml(imageURL, linkURL); + return URLMethodWithParameters.createEmbededImageHtml(imageURL, linkURL); } - public static String createEmbededImageHtml(String imageURL, String linkURLOrNull) - { - String linkStart = "", linkEnd = ""; - if (linkURLOrNull != null) - { - linkStart = "<a class='link-style' href='" + linkURLOrNull + "' target='_blank'>"; - linkEnd = "</a>"; - } - return "<div align='center'>" + linkStart + "<img src='" + imageURL + "' alt='" + " image " - + "'/>" + linkEnd + "</div>"; - } } diff --git a/openbis/source/java/ch/systemsx/cisd/openbis/generic/shared/basic/URLMethodWithParameters.java b/openbis/source/java/ch/systemsx/cisd/openbis/generic/shared/basic/URLMethodWithParameters.java index dae06cdb600091fcba9797918ccf11cab9bbbaa8..e597865162d549ace686e41593f810080a86c6b6 100644 --- a/openbis/source/java/ch/systemsx/cisd/openbis/generic/shared/basic/URLMethodWithParameters.java +++ b/openbis/source/java/ch/systemsx/cisd/openbis/generic/shared/basic/URLMethodWithParameters.java @@ -87,4 +87,16 @@ public class URLMethodWithParameters implements IsSerializable return builder.toString(); } + /** Creates HTML which displays an image linking to the given URL (if it is specified). */ + public static String createEmbededImageHtml(String imageURL, String linkURLOrNull) + { + String linkStart = "", linkEnd = ""; + if (linkURLOrNull != null) + { + linkStart = "<a class='link-style' href='" + linkURLOrNull + "' target='_blank'>"; + linkEnd = "</a>"; + } + return "<div align='center'>" + linkStart + "<img src='" + imageURL + "' alt='" + " image " + + "'/>" + linkEnd + "</div>"; + } } diff --git a/openbis/source/java/ch/systemsx/cisd/openbis/generic/shared/basic/dto/GeneratedImageTableCell.java b/openbis/source/java/ch/systemsx/cisd/openbis/generic/shared/basic/dto/GeneratedImageTableCell.java index 92b3088fd6d0f65d5c15d812cc7e11a8ddd9c854..9a55a13d19330b8077315424ad0eceedaf8ba1fa 100644 --- a/openbis/source/java/ch/systemsx/cisd/openbis/generic/shared/basic/dto/GeneratedImageTableCell.java +++ b/openbis/source/java/ch/systemsx/cisd/openbis/generic/shared/basic/dto/GeneratedImageTableCell.java @@ -150,12 +150,8 @@ public class GeneratedImageTableCell implements ISerializableComparable urlMethodThumb.addParameter(IMAGE_WIDTH_PARAM, thumbnailWidth); urlMethodThumb.addParameter(IMAGE_HEIGHT_PARAM, thumbnailHeight); - return getEmbededImageHtml(urlMethodThumb.toString(), urlMethodImage.toString()); + return URLMethodWithParameters.createEmbededImageHtml(urlMethodThumb.toString(), + urlMethodImage.toString()); } - private String getEmbededImageHtml(String thumbURL, String imageURL) - { - return "<div align='center'><a class='link-style' href='" + imageURL - + "' target='_blank'><img src='" + thumbURL + "' alt='" + "image" + "'/></a></div>"; - } } diff --git a/openbis/sourceTest/java/ch/systemsx/cisd/openbis/generic/shared/basic/dto/GeneratedImageTableCellTest.java b/openbis/sourceTest/java/ch/systemsx/cisd/openbis/generic/shared/basic/dto/GeneratedImageTableCellTest.java index d38c5ff1bb265de64250f37e8a3dd2d42d4e867a..0791435d384932ccc504783076dedd0fb9adc17f 100644 --- a/openbis/sourceTest/java/ch/systemsx/cisd/openbis/generic/shared/basic/dto/GeneratedImageTableCellTest.java +++ b/openbis/sourceTest/java/ch/systemsx/cisd/openbis/generic/shared/basic/dto/GeneratedImageTableCellTest.java @@ -3,7 +3,7 @@ package ch.systemsx.cisd.openbis.generic.shared.basic.dto; import org.testng.AssertJUnit; import org.testng.annotations.Test; -import ch.systemsx.cisd.openbis.generic.client.web.client.application.ui.columns.framework.renderers.SimpleImageHtmlRenderer; +import ch.systemsx.cisd.openbis.generic.shared.basic.URLMethodWithParameters; /** * @author Chandrasekhar Ramakrishnan @@ -18,7 +18,7 @@ public class GeneratedImageTableCellTest extends AssertJUnit cell.addParameter("code", "CODE_8472"); String cellHTML = cell.getHTMLString("http://my.server.ch", "sessionToken"); String rendererHTML = - SimpleImageHtmlRenderer + URLMethodWithParameters .createEmbededImageHtml( "http://my.server.ch/servletName?sessionID=sessionToken&code=CODE_8472&w=60&h=60", "http://my.server.ch/servletName?sessionID=sessionToken&code=CODE_8472&w=600&h=300"); diff --git a/screening/source/java/ch/systemsx/cisd/openbis/plugin/screening/client/web/client/application/detailviewers/ImageUrlUtils.java b/screening/source/java/ch/systemsx/cisd/openbis/plugin/screening/client/web/client/application/detailviewers/ImageUrlUtils.java index a20947e74983f2f47e91573b231b95eacaa4dfcb..1633796bd7d3fff69d85623e097fb07f5e479b2f 100644 --- a/screening/source/java/ch/systemsx/cisd/openbis/plugin/screening/client/web/client/application/detailviewers/ImageUrlUtils.java +++ b/screening/source/java/ch/systemsx/cisd/openbis/plugin/screening/client/web/client/application/detailviewers/ImageUrlUtils.java @@ -20,10 +20,9 @@ import com.extjs.gxt.ui.client.widget.Component; import com.extjs.gxt.ui.client.widget.Html; import com.extjs.gxt.ui.client.widget.LayoutContainer; -import ch.systemsx.cisd.openbis.generic.client.web.client.application.ui.columns.framework.renderers.SimpleImageHtmlRenderer; import ch.systemsx.cisd.openbis.generic.shared.basic.URLMethodWithParameters; -import ch.systemsx.cisd.openbis.plugin.screening.shared.basic.dto.WellImageChannelStack; import ch.systemsx.cisd.openbis.plugin.screening.shared.basic.dto.ScreeningConstants; +import ch.systemsx.cisd.openbis.plugin.screening.shared.basic.dto.WellImageChannelStack; /** * Generates URLs pointing to the images on Data Store server. @@ -75,7 +74,7 @@ public class ImageUrlUtils methodWithParameters.addParameter("mode", "thumbnail" + width + "x" + height); String imageURL = methodWithParameters.toString(); - return SimpleImageHtmlRenderer.createEmbededImageHtml(imageURL, linkURL); + return URLMethodWithParameters.createEmbededImageHtml(imageURL, linkURL); } /** creates a widget which displays the specified URL and adds it to the container */ @@ -107,7 +106,7 @@ public class ImageUrlUtils methodWithParameters.addParameter("mode", "thumbnail" + width + "x" + height); String imageURL = methodWithParameters.toString(); - return SimpleImageHtmlRenderer.createEmbededImageHtml(imageURL, linkURL); + return URLMethodWithParameters.createEmbededImageHtml(imageURL, linkURL); } private static URLMethodWithParameters createBasicImageURL(String sessionID, WellImages images, diff --git a/screening/source/java/ch/systemsx/cisd/openbis/plugin/screening/client/web/client/application/detailviewers/WellContentDialog.java b/screening/source/java/ch/systemsx/cisd/openbis/plugin/screening/client/web/client/application/detailviewers/WellContentDialog.java index cd5645e507cfcc116b5175baea52cba4b8cb31a7..263c9a1721b58d534e1e935e69ccec54790c74f7 100644 --- a/screening/source/java/ch/systemsx/cisd/openbis/plugin/screening/client/web/client/application/detailviewers/WellContentDialog.java +++ b/screening/source/java/ch/systemsx/cisd/openbis/plugin/screening/client/web/client/application/detailviewers/WellContentDialog.java @@ -274,14 +274,14 @@ public class WellContentDialog extends Dialog private static int getDialogWidth(WellImages images) { float imageSizeMultiplyFactor = getImageSizeMultiplyFactor(images); - return (int) (ONE_IMAGE_WIDTH_PX * imageSizeMultiplyFactor) * images.getTileColsNum() + 100; + return (int) (ONE_IMAGE_WIDTH_PX * imageSizeMultiplyFactor) * images.getTileColsNum() + 30; } private static int getDialogHeight(WellImages images) { float imageSizeMultiplyFactor = getImageSizeMultiplyFactor(images); return Math.max((int) (ONE_IMAGE_HEIGHT_PX * imageSizeMultiplyFactor) - * images.getTileRowsNum() + 100, 300); + * images.getTileRowsNum() + 300, 300); } private static int getImageHeight(WellImages images)