[Checkins] SVN: zopeorg.theme/trunk/zopeorg/theme/ made FeatureViewlet to be visible for IViewView only - we don't need it for Edit or any other screen. Used dirty way of achieving this. Simple view="plone.app.layout.globals.interfaces.IViewView" for a viewlet didn't work out for un-known (yet ; )) reason

[Denis Mishunov]

Log message for revision 83475:
  made FeatureViewlet to be visible for IViewView only - we don't need it for Edit or any other screen. Used dirty way of achieving this. Simple view="plone.app.layout.globals.interfaces.IViewView" for a viewlet didn't work out for un-known (yet ;)) reason

Changed:
  U   zopeorg.theme/trunk/zopeorg/theme/browser/configure.zcml
  U   zopeorg.theme/trunk/zopeorg/theme/browser/templates/feature.pt
  U   zopeorg.theme/trunk/zopeorg/theme/browser/viewlets.py
  U   zopeorg.theme/trunk/zopeorg/theme/skins/zopeorg_theme_styles/main.css.dtml

-=-
Modified: zopeorg.theme/trunk/zopeorg/theme/browser/configure.zcml
===================================================================
--- zopeorg.theme/trunk/zopeorg/theme/browser/configure.zcml	2008-02-03 22:56:59 UTC (rev 83474)
+++ zopeorg.theme/trunk/zopeorg/theme/browser/configure.zcml	2008-02-03 23:16:44 UTC (rev 83475)
@@ -151,15 +151,6 @@
         />
 
     <!-- Feature's blurb viewlet -->
-    <!-- <browser:viewlet  
-        name="zopeorg.feature"
-        for=".interfaces.IFeature"
-        manager=".interfaces.IFeatureView"
-        view="plone.app.layout.globals.interfaces.IViewView"        
-        class=".viewlets.FeatureViewlet"
-        layer=".interfaces.IThemeSpecific"
-        permission="zope2.View" 
-        />  -->          
     <browser:viewlet  
         name="zopeorg.feature"
         for=".interfaces.IFeature"

Modified: zopeorg.theme/trunk/zopeorg/theme/browser/templates/feature.pt
===================================================================
--- zopeorg.theme/trunk/zopeorg/theme/browser/templates/feature.pt	2008-02-03 22:56:59 UTC (rev 83474)
+++ zopeorg.theme/trunk/zopeorg/theme/browser/templates/feature.pt	2008-02-03 23:16:44 UTC (rev 83475)
@@ -1,5 +1,6 @@
 <div id="portal-feature"
-     i18n:domain="zopeorg.theme">
+     i18n:domain="zopeorg.theme"
+     tal:condition="view/visible">
      
      <img tal:replace="structure python:view.tag(scale='mini', css_class='featureImage')" src="" alt="" />
      

Modified: zopeorg.theme/trunk/zopeorg/theme/browser/viewlets.py
===================================================================
--- zopeorg.theme/trunk/zopeorg/theme/browser/viewlets.py	2008-02-03 22:56:59 UTC (rev 83474)
+++ zopeorg.theme/trunk/zopeorg/theme/browser/viewlets.py	2008-02-03 23:16:44 UTC (rev 83475)
@@ -2,12 +2,19 @@
 
 from plone.app.layout.viewlets.common import ViewletBase, SearchBoxViewlet, PersonalBarViewlet
 
+from plone.app.layout.globals.interfaces import IViewView 
+
 class FeatureViewlet(ViewletBase):
     render = ViewPageTemplateFile('templates/feature.pt')
     
-    # def update(self):
-    #     if IViewView.providedBy(self.__parent__):
-    #         alsoProvides(self, IViewView)
+    def visible(self):
+        # we should *climb up* to object itself from
+        # 1. viewlet to viewletManager
+        # 2. viewletManager to object
+        if IViewView.providedBy(self.__parent__.__parent__):
+            return True
+        else:
+            return False
         
     def tag(self, **kwargs):
         return self.context.getField('image').tag(self.context, **kwargs)

Modified: zopeorg.theme/trunk/zopeorg/theme/skins/zopeorg_theme_styles/main.css.dtml
===================================================================
--- zopeorg.theme/trunk/zopeorg/theme/skins/zopeorg_theme_styles/main.css.dtml	2008-02-03 22:56:59 UTC (rev 83474)
+++ zopeorg.theme/trunk/zopeorg/theme/skins/zopeorg_theme_styles/main.css.dtml	2008-02-03 23:16:44 UTC (rev 83475)
@@ -135,6 +135,7 @@
 
 li.formTab a.selected {
     color: &dtml-linkActiveColor;;
+    background-color: #E3E3E3;
 }
 
 /* footer styles */