Not attempting to show a widget zone preview image if the currently active theme doesn't have one.

--HG--
branch : dev

src/Orchard.Web/Modules/Orchard.Widgets/Controllers/AdminController.cs

@@ -6,6 +6,7 @@ using Orchard.ContentManagement;
 using Orchard.Core.Contents.Controllers;
 using Orchard.DisplayManagement;
 using Orchard.Environment.Extensions.Models;
+using Orchard.FileSystems.VirtualPath;
 using Orchard.Localization;
 using Orchard.Logging;
 using Orchard.Mvc.Extensions;
@@ -25,16 +26,19 @@ namespace Orchard.Widgets.Controllers {
 
         private readonly IWidgetsService _widgetsService;
         private readonly ISiteThemeService _siteThemeService;
+        private readonly IVirtualPathProvider _virtualPathProvider;
 
         public AdminController(
             IOrchardServices services,
             IWidgetsService widgetsService,
             IShapeFactory shapeFactory,
-            ISiteThemeService siteThemeService) {
+            ISiteThemeService siteThemeService,
+            IVirtualPathProvider virtualPathProvider) {
 
             Services = services;
             _widgetsService = widgetsService;
             _siteThemeService = siteThemeService;
+            _virtualPathProvider = virtualPathProvider;
 
             T = NullLocalizer.Instance;
             Logger = NullLogger.Instance;
@@ -67,13 +71,17 @@ namespace Orchard.Widgets.Controllers {
             IEnumerable<string> allZones = _widgetsService.GetZones();
             IEnumerable<string> currentThemesZones = _widgetsService.GetZones(currentTheme);
 
+            string zonePreviewImagePath = string.Format("{0}/{1}/ThemeZonePreview.png", currentTheme.Location, currentTheme.Id);
+            string zonePreviewImage = _virtualPathProvider.FileExists(zonePreviewImagePath) ? zonePreviewImagePath : null;
+
             dynamic viewModel = Shape.ViewModel()
                 .CurrentTheme(currentTheme)
                 .CurrentLayer(currentLayer)
                 .Layers(layers)
                 .Widgets(_widgetsService.GetWidgets())
                 .Zones(currentThemesZones)
-                .OrphanZones(allZones.Except(currentThemesZones));
+                .OrphanZones(allZones.Except(currentThemesZones))
+                .ZonePreviewImage(zonePreviewImage);
 
             // Casting to avoid invalid (under medium trust) reflection over the protected View method and force a static invocation.
             return View((object)viewModel);

src/Orchard.Web/Modules/Orchard.Widgets/Styles/orchard-widgets-admin.css

@@ -1,9 +1,12 @@
 #layout-widgets-placement {
+clear:left;
 float:left;
 width:65%;
 }
 #layout-widgets-assistance {
 float:right;
+position:relative;
+top:-30px;
 width:35%;
 }
 .widgets-container {
@@ -17,6 +20,7 @@ padding:2px 2px;
 
 
 #widgets-layers-control {
+float:left;
 margin-bottom:-20px;
 }
 #widgets-layers-control label {
@@ -163,8 +167,12 @@ color:#333;
 margin-top:5px;
 }
 
-#widgets-layer-visibility {
+#widgets-theme-zone-preview {
-margin-top:20px;
+margin:30px 0 20px;
+}
+
+#main #widgets-layer-visibility h3 {
+margin-top:0;
 }
 #widgets-layer-visibility li {
 cursor:default;

src/Orchard.Web/Modules/Orchard.Widgets/Views/Admin/Index.cshtml

@@ -10,7 +10,7 @@
     <div id="layout-widgets-assistance">
         <div id="widgets-assistance">
             <div class="widgets-container">
-                @Display.WidgetSimplePreview(CurrentTheme: Model.CurrentTheme)
+                @Display.WidgetSimplePreview(ZonePreviewImage: Model.ZonePreviewImage, CurrentTheme: Model.CurrentTheme)
                 @Display.WidgetLayerVisibility(Layers: Model.Layers, CurrentLayer: Model.CurrentLayer)
             </div>
         </div>

src/Orchard.Web/Modules/Orchard.Widgets/Views/WidgetSimplePreview.cshtml

@@ -1,2 +1,4 @@
 @using Orchard.Environment.Extensions.Models
-<p id="widgets-theme-zone-preview">@Html.Image(Href(Html.ThemePath((ExtensionDescriptor) Model.CurrentTheme, "/ThemeZonePreview.png")), T("{0} zone preview", (string)Model.CurrentTheme.Name).Text, null)</p>
+@if (HasText(Model.ZonePreviewImage)) {
+<p id="widgets-theme-zone-preview">@Html.Image(Href((string)Model.ZonePreviewImage), T("{0} zone preview", (string)Model.CurrentTheme.Name).Text, null)</p>
+}