issue #2414 remove config and update css

admin/configuration.php

@@ -76,7 +76,6 @@ $display_checkboxes = array(
     'menubar_filter_icon',
     'index_search_in_set_button',
     'index_search_in_set_action',
-    'index_related_tags_display',
     'index_sort_order_input',
     'index_flat_icon',
     'index_posted_date_icon',

admin/themes/default/template/configuration_display.tpl

@@ -56,14 +56,6 @@
         </label>
       </li>
 
-      <li>
-        <label class="font-checkbox">
-          <span class="icon-check"></span>
-          <input type="checkbox" name="index_related_tags_display" {if ($display.index_related_tags_display)}checked="checked"{/if}>
-          {'Display related tags by default'|translate}
-        </label>
-      </li>
-
       <li>
         <label class="font-checkbox">
           <span class="icon-check"></span>

index.php

@@ -226,8 +226,7 @@ if ( empty($page['is_external']) )
         'SEARCH_IN_SET_BUTTON' => $conf['index_search_in_set_button'],
         'SEARCH_IN_SET_ACTION' => $conf['index_search_in_set_action'],
         'SEARCH_IN_SET_URL' => get_root_url().'search.php?tag_id='.implode(',', $page['body_data']['tag_ids']),
-        'RELATED_TAGS_DISPLAY' => $conf['index_related_tags_display'],
+        'COMBINABLE_TAGS' => $related_tags,
-        'RELATED_TAGS' => $related_tags,
       )
     );
   }

install/config.sql

@@ -80,5 +80,4 @@ INSERT INTO piwigo_config (param,value) VALUES ('index_search_in_set_action','tr
 INSERT INTO piwigo_config (param,value) VALUES ('upload_detect_duplicate','true');
 INSERT INTO piwigo_config (param,value) VALUES ('webmaster_id','1');
 INSERT INTO piwigo_config (param,value) VALUES ('use_standard_pages','true');
-INSERT INTO piwigo_config (param,value,comment) VALUES ('filters_views','a:14:{s:5:"words";a:2:{s:6:"access";s:9:"everybody";s:7:"default";b:1;}s:4:"tags";a:2:{s:6:"access";s:9:"everybody";s:7:"default";b:0;}s:9:"post_date";a:2:{s:6:"access";s:9:"everybody";s:7:"default";b:0;}s:13:"creation_date";a:2:{s:6:"access";s:9:"everybody";s:7:"default";b:1;}s:5:"album";a:2:{s:6:"access";s:9:"everybody";s:7:"default";b:1;}s:6:"author";a:2:{s:6:"access";s:9:"everybody";s:7:"default";b:0;}s:8:"added_by";a:2:{s:6:"access";s:9:"everybody";s:7:"default";b:0;}s:9:"file_type";a:2:{s:6:"access";s:9:"everybody";s:7:"default";b:0;}s:5:"ratio";a:2:{s:6:"access";s:9:"everybody";s:7:"default";b:0;}s:6:"rating";a:2:{s:6:"access";s:9:"everybody";s:7:"default";b:0;}s:9:"file_size";a:2:{s:6:"access";s:9:"everybody";s:7:"default";b:0;}s:6:"height";a:2:{s:6:"access";s:9:"everybody";s:7:"default";b:0;}s:5:"width";a:2:{s:6:"access";s:9:"everybody";s:7:"default";b:0;}s:17:"last_filters_conf";b:1;}','Filters displays configuration');
+INSERT INTO piwigo_config (param,value,comment) VALUES ('filters_views','a:14:{s:5:"words";a:2:{s:6:"access";s:9:"everybody";s:7:"default";b:1;}s:4:"tags";a:2:{s:6:"access";s:9:"everybody";s:7:"default";b:0;}s:9:"post_date";a:2:{s:6:"access";s:9:"everybody";s:7:"default";b:0;}s:13:"creation_date";a:2:{s:6:"access";s:9:"everybody";s:7:"default";b:1;}s:5:"album";a:2:{s:6:"access";s:9:"everybody";s:7:"default";b:1;}s:6:"author";a:2:{s:6:"access";s:9:"everybody";s:7:"default";b:0;}s:8:"added_by";a:2:{s:6:"access";s:9:"everybody";s:7:"default";b:0;}s:9:"file_type";a:2:{s:6:"access";s:9:"everybody";s:7:"default";b:0;}s:5:"ratio";a:2:{s:6:"access";s:9:"everybody";s:7:"default";b:0;}s:6:"rating";a:2:{s:6:"access";s:9:"everybody";s:7:"default";b:0;}s:9:"file_size";a:2:{s:6:"access";s:9:"everybody";s:7:"default";b:0;}s:6:"height";a:2:{s:6:"access";s:9:"everybody";s:7:"default";b:0;}s:5:"width";a:2:{s:6:"access";s:9:"everybody";s:7:"default";b:0;}s:17:"last_filters_conf";b:1;}','Filters displays configuration');
-INSERT INTO piwigo_config (param,value) VALUES ('index_related_tags_display','true');

install/db/180-database.php

@@ -11,10 +11,12 @@ if (!defined('PHPWG_ROOT_PATH'))
   die('Hacking attempt!');
 }
 
-$upgrade_description = 'add config parameters to display by default or not "related tags" ';
+$upgrade_description = 'add config parameters to display by default or not "related tags" [Aborted] ';
-
-conf_update_param('index_related_tags_display', true);
-
-echo "\n".$upgrade_description."\n";
 
+//The contents of the database upgrade have been emptied 
+//due to the fact that we decicded to not use this configuration
+//in the end for the related tags display
+//We need to keep this database ugrapde to avoid errors 
+//for those that have already passed the "180"th upgrade.
+//This database upgrade now does nothing.
 ?>

themes/default/template/include/related_tags.inc.tpl

@@ -1,10 +1,6 @@
-<div class="mcs-side-results search-in-set-button" id="related-tags-toggle">
+<h3 class="related-tags-title">{'Related tags'|translate}</h3>
-  <div>
-    <p>{'Related tags'|translate}<i class="gallery-icon-up-open rotated"></i></p>
-  </div>
-</div>
 
-{foreach from=$RELATED_TAGS item=tag}
+{foreach from=$COMBINABLE_TAGS item=tag}
 <span class="related-tags {if isset($RELATED_TAGS_DISPLAY) and $RELATED_TAGS_DISPLAY == false} hide{/if}">{strip}
   <a class="tagLevel {if isset($tag.level)}{$tag.level}{/if}" href=
   {if isset($tag.U_ADD)}
@@ -12,16 +8,5 @@
   {else}
     "{$tag.URL}" title="{'display photos linked to this tag'|@translate}">
   {/if}
-    + {$tag.name}</a><div class="tag-counter">{$tag.counter}</div></span>{/strip}
+    + {$tag.name}<div class="tag-counter">{$tag.counter}</div></a></span>{/strip}
-{/foreach}
+{/foreach}
-
-{footer_script require='jquery'}
-
-$(document).ready(function () { 
-  $('#related-tags-toggle').on("click", function (e) { 
-    $('.related-tags').toggle();
-    $('#related-tags-toggle .gallery-icon-up-open').toggleClass('rotated'); 
-  }); 
-});
-
-{/footer_script}

themes/default/template/index.tpl

@@ -192,8 +192,9 @@
   </div>
 {/if}
 
+{if isset($COMBINABLE_TAGS)}
 {include file='include/related_tags.inc.tpl'}
-
+{/if}
 </div>
 
 {if !empty($CONTENT_DESCRIPTION)}

themes/default/theme.css

@@ -879,7 +879,7 @@ because of this it needs to specifically be added to each theme */
   display: flex;
   align-items: center;
   cursor: pointer;
-  margin-right: 10px;
+  margin-right: 15px;
   color: #ccc;
   width: fit-content;
 }
@@ -908,7 +908,6 @@ because of this it needs to specifically be added to each theme */
   font-size: 10px;
   background: #777;
 }
-
 .action-buttons{
   display:flex;
   align-items:start;
@@ -917,38 +916,34 @@ because of this it needs to specifically be added to each theme */
   padding:0 15px;
 }
 
-#related-tags-toggle i{
+h3.related-tags-title{
-  margin-left:5px;
+  margin:4px 0;
-}
+  margin-right:10px;
-
+  font-weight:400;
-#related-tags-toggle i::before {
-  transition: transform 0.3s ease;
-}
-
-#related-tags-toggle i.rotated::before {
-  transform: rotate(90deg);
-} 
-
-.related-tags.hide{
-  display:none;
 }
 
 span.related-tags{
-  padding:3px 5px;
+  padding:4px 12px;
-  border:1px solid #D6D6D6;
   margin:3px;
   border-radius:15px;
+  background-color:#f5f5f5
 }
 
 span.related-tags a{
   font-weight:700;
   color:#787878;
+  text-decoration:none;
+}
+
+span.related-tags:hover{
+  background-color:#e5e5e5;
 }
 
 span.related-tags .tag-counter{
   display: inline-block;
-  margin-left: 7px;
+  margin-left:9px;
-  color:#D6D6D6;
+  color:#ACACAC;
+  font-weight:700;
 }
 
 #relatedTagsBox.switchBox A {