Some improvements to styles in result list. Now the result controller doesn't appear when there are no results.

Author: shannah

Dataface/ResultList.php

@@ -339,11 +339,10 @@ function toHtml($mode = 'all'){
  		// where Direction is "asc" or "desc"
 
 
- 		
  		if (true or $this->_resultSet->found() > 0 ) {
 
 
- 			if  ($desktop and @$app->prefs['use_old_resultlist_controller'] ){
+ 			if  ($desktop and @$app->prefs['use_old_resultlist_controller'] and $this->_resultSet->found() > 0){
 				ob_start();
 				df_display(array(), 'Dataface_ResultListController.html');
 				$controller = ob_get_contents();
@@ -373,7 +372,7 @@ function toHtml($mode = 'all'){
 			}
 			unset($query);
 
-			if ( @$app->prefs['use_old_resultlist_controller'] ){
+			if ( @$app->prefs['use_old_resultlist_controller'] and $this->_resultSet->found() > 0){
 				echo '<div class="resultlist-controller" id="resultlist-controller-top">';
 
 				echo $controller;
@@ -735,7 +734,7 @@ function toHtml($mode = 'all'){
                 }
                 echo "</tbody>
                     </table>";
-                if ( $canSelect ){
+                if ( $canSelect and $this->_resultSet->found() > 0){
                     echo  '<form id="result_list_selected_items_form" method="post" action="'.df_absolute_url(DATAFACE_SITE_HREF).'">';
                     $app =& Dataface_Application::getInstance();
                     $q =& $app->getQuery();
@@ -777,7 +776,7 @@ function toHtml($mode = 'all'){
                     }
                 }
 
-                if ( @$app->prefs['use_old_resultlist_controller'] ){
+                if ( @$app->prefs['use_old_resultlist_controller'] and $this->_resultSet->found() > 0){
                     echo '<div class="resultlist-controller" id="resultlist-controller-bottom">';
 
                     echo $controller;
@@ -794,7 +793,7 @@ function toHtml($mode = 'all'){
         if ($desktop) {
             // Just to prevent from running twice.
             // In actuality this is used for both mobile and desktop
-			if ( @$app->prefs['use_old_resultlist_controller'] ){
+			if ( @$app->prefs['use_old_resultlist_controller'] and $this->_resultSet->found() > 0){
 				ob_start();
 				df_display(array(), 'Dataface_ResultListController.html');
 				$out .= ob_get_contents();

Dataface/templates/Dataface_List_View.html

@@ -31,11 +31,13 @@
 		{/if}
 		<div id="list-window">
 		{if !$ENV.prefs.use_old_resultlist_controller}
+            {if $ENV.resultSet->found() > 0}
 			{if !$ENV.prefs.hide_resultlist_controller}
 			{if $ENV.resultSet->found() > $ENV.resultSet->limit() or !$ENV.APPLICATION_OBJECT->prefs.show_resultlist_controller_only_when_needed}
 				{result_controller}
 			{/if}
 			{/if}
+            {/if}
 		{/if}
 		<div style="clear:both">
 		{block name="before_result_list_content"}
@@ -45,11 +47,13 @@
 		{block name="after_result_list_content"}
 		</div>
 		{if !$ENV.prefs.use_old_resultlist_controller}
+            {if $ENV.resultSet->found() > 0}
 			{if !$ENV.prefs.hide_resultlist_controller}
 			{if ($ENV.resultSet->found() > $ENV.resultSet->limit()) or !$ENV.APPLICATION_OBJECT->prefs.show_resultlist_controller_only_when_needed}
 				{result_controller}
 			{/if}
 			{/if}
+            {/if}
 		{/if}
 		</div>
 		{/define_slot}

actions.ini.php

@@ -81,18 +81,7 @@ class="featured-action"
 [post]
 	permission = post
 
-;; Show all records in the current table
-[show_all]
-	label = Show All
-	description = Show all records in table
-	url = "{$site_href}?-action=list&-table={$table}"
-	icon = "{$dataface_url}/images/zoom-out.gif"
-	accessKey = a
-	category = table_actions
-	mode = list
-	permission = show all
-	order=4
-	tags="#large#"
+
 
 [copy_replace] 
     label="Copy Set"
@@ -597,18 +586,7 @@ class="featured-action"
 [post]
 	permission = post
 
-;; Show all records in the current table
-[show_all]
-	label = Show All
-	description = Show all records in table
-	url = "{$site_href}?-action=list&-table={$table}"
-	icon = "{$dataface_url}/images/zoom-out.gif"
-	accessKey = a
-	category = table_actions
-	mode = list
-	permission = show all
-	order=4
-	tags="#large#"
+
 
 [copy_replace] 
     label="Copy Set"

plone.css

@@ -2887,7 +2887,6 @@ nav.personal-tools-menu a {
       border: 1px solid rgb(218, 220, 224);
       border-radius: 4px;
       display: block;
-      float: right;
       padding: 0.25em;
         padding-left: 10px;
         padding-right: 10px;
@@ -2896,6 +2895,22 @@ nav.personal-tools-menu a {
     transition-duration: 0.5s;
         margin-bottom: 0;
   }
+  @media only screen and (min-width:768px) {
+      div.xf-button-bar-actions li.mobile,
+      nav.tables li.mobile,
+      nav.personal-tools-menu  li.mobile {
+          display:none;
+      }
+  }
+  @media only screen and (max-width:768px) {
+      div.xf-button-bar-actions li.desktop,
+      nav.tables li.desktop,
+      nav.personal-tools-menu  li.desktop {
+          display:none;
+      }
+  }
+  body.large div.xf-button-bar-actions li,
+  body.large nav.tables li,
   nav.tables li {
       float:left;
   }
@@ -3004,6 +3019,9 @@ div.xf-button-bar-actions > ul > li.featured-action,
     opacity: 1;
     display: block
   }
+  
+  
+  
   div.xf-button-bar-actions ul li ul li,
   nav.tables ul li ul li,
   nav.personal-tools-menu ul li ul li {
@@ -3016,6 +3034,12 @@ div.xf-button-bar-actions > ul > li.featured-action,
     padding:0.5em;;
   }
 
+  nav.personal-tools-menu {
+      float:right;
+  }
+  nav.personal-tools-menu > ul {
+      display:flex;
+  }
 
 
 /*table tabs*/