Author: mareshkau Date: 2008-11-06 15:37:35 -0500 (Thu, 06 Nov 2008) New Revision: 11596 Modified: trunk/jst/plugins/org.jboss.tools.jst.jsp/src/org/jboss/tools/jst/jsp/outline/cssdialog/tabs/TabBackgroundControl.java Log: JBIDE-3084 Modified: trunk/jst/plugins/org.jboss.tools.jst.jsp/src/org/jboss/tools/jst/jsp/outline/cssdialog/tabs/TabBackgroundControl.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.jsp/src/org/jboss/tools/jst/jsp/outline/cssdialog/tabs/TabBackgroundControl.java 2008-11-06 19:09:04 UTC (rev 11595) +++ trunk/jst/plugins/org.jboss.tools.jst.jsp/src/org/jboss/tools/jst/jsp/outline/cssdialog/tabs/TabBackgroundControl.java 2008-11-06 20:37:35 UTC (rev 11596) @@ -157,8 +157,11 @@ public void modifyText(ModifyEvent event) { String tmp = backgroundImageCombo.getText(); if (tmp != null) { - if (tmp.trim().length() > 0) - styleAttributes.addAttribute(CSSConstants.BACKGROUND_IMAGE, tmp); + if (tmp.trim().length() > 0){ + + tmp = adjustBackgroundURL(tmp); + styleAttributes.addAttribute(CSSConstants.BACKGROUND_IMAGE, tmp); + } else styleAttributes.removeAttribute(CSSConstants.BACKGROUND_IMAGE); } @@ -199,6 +202,7 @@ IFile file = (IFile) dialog.getFirstResult(); String value = file.getFullPath().toString(); backgroundImageCombo.add(value); + value = adjustBackgroundURL(value); backgroundImageCombo.setText(value); } } @@ -266,4 +270,12 @@ else backgroundRepeatCombo.setText(Constants.EMPTY_STRING); } + //Fix for JBIDE-3084 + //in css background image should always be wraped into url(*); + private static String adjustBackgroundURL(String backgroundURL) { + if(backgroundURL!=null && backgroundURL.matches("(url)\\(.*\\)")==false) { //$NON-NLS-1$ + return "url("+backgroundURL+")"; //$NON-NLS-1$//$NON-NLS-2$ + } + return backgroundURL; + } } \ No newline at end of file