From ef2673d3cf058d94031ee54738f9e9950f86b729 Mon Sep 17 00:00:00 2001
From: Leo Germani <leogermani@gmail.com>
Date: Mon, 27 Aug 2018 17:40:34 -0300
Subject: [PATCH 01/12] do not attach automatic generated thumbnails to item
 (#115)

---
 src/classes/repositories/class-tainacan-items.php | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/classes/repositories/class-tainacan-items.php b/src/classes/repositories/class-tainacan-items.php
index 5b1b9268c..194580696 100644
--- a/src/classes/repositories/class-tainacan-items.php
+++ b/src/classes/repositories/class-tainacan-items.php
@@ -425,7 +425,7 @@ class Items extends Repository {
 				$TainacanMedia = \Tainacan\Media::get_instance();
 				$thumb_blob    = $TainacanMedia->get_pdf_cover( $filepath );
 				if ( $thumb_blob ) {
-					$thumb_id = $TainacanMedia->insert_attachment_from_blob( $thumb_blob, basename( $filepath ) . '-cover.jpg', $item->get_id() );
+					$thumb_id = $TainacanMedia->insert_attachment_from_blob( $thumb_blob, basename( $filepath ) . '-cover.jpg' );
 
 					return $thumb_id;
 				}
@@ -443,7 +443,7 @@ class Items extends Repository {
 					return $existing_thumb;
 				} else {
 					$TainacanMedia = \Tainacan\Media::get_instance();
-					$thumb_id      = $TainacanMedia->insert_attachment_from_url( $thumb_url, $item->get_id() );
+					$thumb_id      = $TainacanMedia->insert_attachment_from_url( $thumb_url );
 					update_post_meta( $item->get_id(), $meta_key, $thumb_id );
 
 					return $thumb_id;

From 09e57dd2076d4719bde23598bd54c7ebbecaeda6 Mon Sep 17 00:00:00 2001
From: Mateus Machado Luna <mateus.m.luna@gmail.com>
Date: Mon, 27 Aug 2018 17:21:06 -0300
Subject: [PATCH 02/12] Two small changes for making theme list compatible with
 color change on theme

---
 src/admin/pages/lists/items-page.vue      | 2 +-
 src/admin/pages/lists/term-items-page.vue | 2 +-
 src/admin/scss/_tooltips.scss             | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/admin/pages/lists/items-page.vue b/src/admin/pages/lists/items-page.vue
index f6ccaa385..8885aa5f8 100644
--- a/src/admin/pages/lists/items-page.vue
+++ b/src/admin/pages/lists/items-page.vue
@@ -52,7 +52,7 @@
             </div>
             <a
                     @click="openAdvancedSearch = !openAdvancedSearch"
-                    class="is-size-7 has-text-secondary is-pulled-right is-hidden-mobile">{{ $i18n.get('advanced_search') }}</a>
+                    class="is-size-7 is-pulled-right is-hidden-mobile">{{ $i18n.get('advanced_search') }}</a>
             
             <h3 class="has-text-weight-semibold">{{ $i18n.get('filters') }}</h3>
             <a
diff --git a/src/admin/pages/lists/term-items-page.vue b/src/admin/pages/lists/term-items-page.vue
index b5b092271..0d526e204 100644
--- a/src/admin/pages/lists/term-items-page.vue
+++ b/src/admin/pages/lists/term-items-page.vue
@@ -54,7 +54,7 @@
             </div>
             <a
                     @click="openAdvancedSearch = !openAdvancedSearch"
-                    class="is-size-7 has-text-secondary is-pulled-right is-hidden-mobile">{{ $i18n.get('advanced_search') }}</a>
+                    class="is-size-7 is-pulled-right is-hidden-mobile">{{ $i18n.get('advanced_search') }}</a>
             
             <h3 class="has-text-weight-semibold">{{ $i18n.get('filters') }}</h3>
             <a
diff --git a/src/admin/scss/_tooltips.scss b/src/admin/scss/_tooltips.scss
index 6ad92875b..3445a749c 100644
--- a/src/admin/scss/_tooltips.scss
+++ b/src/admin/scss/_tooltips.scss
@@ -10,7 +10,7 @@
 
     .tooltip-inner {
         background: $turquoise1;
-        color: $gray4;
+        color: $gray5;
         font-size: 11px;
         border-radius: 5px;
         padding: 20px;

From 8f417e5c691615993807c870ad26e04b0c14e675 Mon Sep 17 00:00:00 2001
From: Mateus Machado Luna <mateus.m.luna@gmail.com>
Date: Tue, 28 Aug 2018 09:47:35 -0300
Subject: [PATCH 03/12] Adds extra loading to prevent blinking while loading
 metadata in items page.

---
 src/js/event-bus-search.js | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/src/js/event-bus-search.js b/src/js/event-bus-search.js
index b6083cf0c..8d40ebe5e 100644
--- a/src/js/event-bus-search.js
+++ b/src/js/event-bus-search.js
@@ -47,7 +47,6 @@ export default {
             },
             watch: {
                 '$route' (to, from) {
-
                     // Should set Collection ID from URL only when in admin.
                     if (this.$route.name == 'CollectionItemsPage' || this.$route.name == 'ItemsPage')
                         this.collectionId = !this.$route.params.collectionId ? this.$route.params.collectionId : parseInt(this.$route.params.collectionId);
@@ -299,13 +298,12 @@ export default {
                 },
                 loadItems(to) {
                     
-                    // Forces fetch_only to be filled before any search happens
-                    if (this.$store.getters['search/getPostQuery']['fetch_only'] == undefined) {
-                        this.$emit( 'hasToPrepareMetadataAndFilters', to);
-                    } else {
+                    this.$emit( 'isLoadingItems', true);
 
-                        this.$emit( 'isLoadingItems', true);
-                    
+                    // Forces fetch_only to be filled before any search happens
+                    if (this.$store.getters['search/getPostQuery']['fetch_only'] == undefined) {     
+                        this.$emit( 'hasToPrepareMetadataAndFilters', to);
+                    } else {   
                         this.$store.dispatch('collection/fetchItems', {
                             'collectionId': this.collectionId,
                             'isOnTheme': (this.$route.name == null),

From 19e3b790c6c7e0322b3710d1b21d5779597ef978 Mon Sep 17 00:00:00 2001
From: Leo Germani <leogermani@gmail.com>
Date: Mon, 27 Aug 2018 18:24:04 -0300
Subject: [PATCH 04/12] change collections rewrite slug

---
 src/classes/repositories/class-tainacan-collections.php | 3 ++-
 src/migrations.php                                      | 5 +++++
 2 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/src/classes/repositories/class-tainacan-collections.php b/src/classes/repositories/class-tainacan-collections.php
index a62c0a4cd..5e8df8d22 100644
--- a/src/classes/repositories/class-tainacan-collections.php
+++ b/src/classes/repositories/class-tainacan-collections.php
@@ -245,7 +245,8 @@ class Collections extends Repository {
 			'has_archive'         => true,
 			'query_var'           => true,
 			'can_export'          => true,
-			'rewrite'             => true,
+			/* Translators: The Collections slug - will be the URL for the collections archive */
+			'rewrite'             => ['slug' => sanitize_title(_x('collections', 'Slug: the string that will be used to build the URL', 'tainacan'))],
 			'capability_type'     => Entities\Collection::get_capability_type(),
 			'map_meta_cap'        => true,
 			'supports'            => [
diff --git a/src/migrations.php b/src/migrations.php
index d3543406c..7467d4e69 100644
--- a/src/migrations.php
+++ b/src/migrations.php
@@ -273,6 +273,11 @@ class Migrations {
 
         }
 	}
+
+	static function refresh_rewrite_rules() {
+		// needed after we changed the Collections post type rewrite slug
+		flush_rewrite_rules(false);
+	}
 	
 }
 

From 9059eb383379458c804c8d5084a9b3694ca1ffb8 Mon Sep 17 00:00:00 2001
From: Mateus Machado Luna <mateus.m.luna@gmail.com>
Date: Tue, 28 Aug 2018 10:33:18 -0300
Subject: [PATCH 05/12] Uses replace instead of push routes on event search bus
 to prevent browser history messing up. Ref. #85.

---
 src/js/event-bus-search.js | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/src/js/event-bus-search.js b/src/js/event-bus-search.js
index 8d40ebe5e..5bd0d2705 100644
--- a/src/js/event-bus-search.js
+++ b/src/js/event-bus-search.js
@@ -46,7 +46,8 @@ export default {
                 });
             },
             watch: {
-                '$route' (to, from) {
+                '$route'  (to, from) {
+                    console.log("router watch");
                     // Should set Collection ID from URL only when in admin.
                     if (this.$route.name == 'CollectionItemsPage' || this.$route.name == 'ItemsPage')
                         this.collectionId = !this.$route.params.collectionId ? this.$route.params.collectionId : parseInt(this.$route.params.collectionId);
@@ -290,8 +291,9 @@ export default {
                     this.updateURLQueries();  
                 },
                 updateURLQueries() {
-                    this.$router.push({query: {}});
-                    this.$router.push({query: this.$store.getters['search/getPostQuery']});
+                    console.log("update");
+                    this.$router.replace({query: {}});
+                    this.$router.replace({query: this.$store.getters['search/getPostQuery']});
                 },
                 updateStoreFromURL() {
                     this.$store.dispatch('search/set_postquery', this.$route.query);
@@ -316,7 +318,7 @@ export default {
                             this.$emit( 'hasFiltered', res.hasFiltered);
 
                             if(res.advancedSearchResults){
-                                this.$router.push({query: this.$store.getters['search/getPostQuery'],});
+                                this.$router.replace({query: this.$store.getters['search/getPostQuery'],});
                                 this.$emit('advancedSearchResults', res.advancedSearchResults);
                             }
                         })

From 6f99eb596849f54e98293ffe6a1de746e0778a8f Mon Sep 17 00:00:00 2001
From: Mateus Machado Luna <mateus.m.luna@gmail.com>
Date: Tue, 28 Aug 2018 10:34:55 -0300
Subject: [PATCH 06/12] Removes console logs.

---
 src/js/event-bus-search.js | 2 --
 1 file changed, 2 deletions(-)

diff --git a/src/js/event-bus-search.js b/src/js/event-bus-search.js
index 5bd0d2705..32904f960 100644
--- a/src/js/event-bus-search.js
+++ b/src/js/event-bus-search.js
@@ -47,7 +47,6 @@ export default {
             },
             watch: {
                 '$route'  (to, from) {
-                    console.log("router watch");
                     // Should set Collection ID from URL only when in admin.
                     if (this.$route.name == 'CollectionItemsPage' || this.$route.name == 'ItemsPage')
                         this.collectionId = !this.$route.params.collectionId ? this.$route.params.collectionId : parseInt(this.$route.params.collectionId);
@@ -291,7 +290,6 @@ export default {
                     this.updateURLQueries();  
                 },
                 updateURLQueries() {
-                    console.log("update");
                     this.$router.replace({query: {}});
                     this.$router.replace({query: this.$store.getters['search/getPostQuery']});
                 },

From d80cf9d7ddec93b65e57a843cb9a8f34c3b51d42 Mon Sep 17 00:00:00 2001
From: Leo Germani <leogermani@gmail.com>
Date: Tue, 28 Aug 2018 11:01:11 -0300
Subject: [PATCH 07/12] adding debug tools to bg process

---
 .../class-tainacan-background-process.php     |  7 ++++-
 src/classes/libs/wp-background-process.php    | 27 +++++++++++++++++--
 2 files changed, 31 insertions(+), 3 deletions(-)

diff --git a/src/classes/class-tainacan-background-process.php b/src/classes/class-tainacan-background-process.php
index 81af266e5..d6805cec9 100644
--- a/src/classes/class-tainacan-background-process.php
+++ b/src/classes/class-tainacan-background-process.php
@@ -276,24 +276,29 @@ abstract class Background_Process extends \WP_Background_Process {
 			}
 		} while ( false !== $task && ! $this->time_exceeded() && ! $this->memory_exceeded() );
 
-		
+		$this->debug('process limit reached');
 
 		if ( false !== $task )  {
 			$this->update( $batch->key, $task );
+			$this->debug('Batch updated');
 		} else {
 			$this->close( $batch->key );
+			$this->debug('Batch closed');
 		}
 
 		$this->unlock_process();
 
 		// Start next batch or complete process.
 		if ( ! $this->is_queue_empty() ) {
+			$this->debug('Dispatch after limit reached');
 			$this->dispatch();
 		} else {
+			$this->debug('Complete');
 			$this->complete();
 			$this->write_log($batch->key, ['Process Finished']);
 		}
 
+		$this->debug('dying');
 		wp_die();
 	}
 
diff --git a/src/classes/libs/wp-background-process.php b/src/classes/libs/wp-background-process.php
index d234d7f42..6c3652e71 100644
--- a/src/classes/libs/wp-background-process.php
+++ b/src/classes/libs/wp-background-process.php
@@ -177,9 +177,9 @@ if ( ! class_exists( 'WP_Background_Process' ) ) {
 				// No data to process.
 				wp_die();
 			}
-
+			$this->debug('checking nonce');
 			check_ajax_referer( $this->identifier, 'nonce' );
-
+			$this->debug('ok!');
 			$this->handle();
 
 			wp_die();
@@ -221,6 +221,7 @@ if ( ! class_exists( 'WP_Background_Process' ) ) {
 		protected function is_process_running() {
 			if ( get_site_transient( $this->identifier . '_process_lock' ) ) {
 				// Process already running.
+				$this->debug('process already running');
 				return true;
 			}
 
@@ -235,6 +236,7 @@ if ( ! class_exists( 'WP_Background_Process' ) ) {
 		 * defined in the time_exceeded() method.
 		 */
 		protected function lock_process() {
+			$this->debug('locking process');
 			$this->start_time = time(); // Set start time of current process.
 
 			$lock_duration = ( property_exists( $this, 'queue_lock_time' ) ) ? $this->queue_lock_time : 60; // 1 minute
@@ -251,6 +253,7 @@ if ( ! class_exists( 'WP_Background_Process' ) ) {
 		 * @return $this
 		 */
 		protected function unlock_process() {
+			$this->debug('unlocking process');
 			delete_site_transient( $this->identifier . '_process_lock' );
 
 			return $this;
@@ -441,17 +444,21 @@ if ( ! class_exists( 'WP_Background_Process' ) ) {
 		 * and data exists in the queue.
 		 */
 		public function handle_cron_healthcheck() {
+			$this->debug('running handle_cron_healthcheck');
 			if ( $this->is_process_running() ) {
 				// Background process already running.
+				$this->debug('running handle_cron_healthcheck: process running');
 				exit;
 			}
 
 			if ( $this->is_queue_empty() ) {
 				// No data to process.
+				$this->debug('running handle_cron_healthcheck: queue empty');
 				$this->clear_scheduled_event();
 				exit;
 			}
 
+			$this->debug('running handle_cron_healthcheck: dispatching');
 			$this->dispatch();
 
 			exit;
@@ -475,6 +482,7 @@ if ( ! class_exists( 'WP_Background_Process' ) ) {
 				exit;
 			}
 
+			$this->debug('handle_cron_healthcheck_check scheduling event');
 			$this->schedule_event();
 
 		}
@@ -485,6 +493,7 @@ if ( ! class_exists( 'WP_Background_Process' ) ) {
 		protected function schedule_event() {
 			if ( ! wp_next_scheduled( $this->cron_hook_identifier ) ) {
 				wp_schedule_event( time(), $this->cron_interval_identifier, $this->cron_hook_identifier );
+				$this->debug('cron event scheduled');
 			}
 		}
 
@@ -496,6 +505,7 @@ if ( ! class_exists( 'WP_Background_Process' ) ) {
 
 			if ( $timestamp ) {
 				wp_unschedule_event( $timestamp, $this->cron_hook_identifier );
+				$this->debug('cron event unscheduled');
 			}
 		}
 
@@ -530,5 +540,18 @@ if ( ! class_exists( 'WP_Background_Process' ) ) {
 		 */
 		abstract protected function task( $item );
 
+		/**
+		 * desperate method to help debug bg processes
+		 */
+		public function debug($message) {
+			if ( !defined('TAINACAN_DEBUG_BG_PROCESS') || true !== TAINACAN_DEBUG_BG_PROCESS || !is_string($message) ) {
+				return;
+			}
+
+			$message = 'BG_PROCESS: ' . $message;
+			error_log($message);
+			
+		}
+
 	}
 }

From 408c58dc31b7a36bcb2b828d28d96331aae02653 Mon Sep 17 00:00:00 2001
From: Leo Germani <leogermani@gmail.com>
Date: Tue, 28 Aug 2018 11:05:40 -0300
Subject: [PATCH 08/12] avoid notice on test importer

---
 src/importer/class-tainacan-test-importer.php | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/importer/class-tainacan-test-importer.php b/src/importer/class-tainacan-test-importer.php
index 87939c297..66a594b4a 100644
--- a/src/importer/class-tainacan-test-importer.php
+++ b/src/importer/class-tainacan-test-importer.php
@@ -690,7 +690,9 @@ class Test_Importer extends Importer {
 		$metadatum->set_name($args['name']);
 		$metadatum->set_collection($collection);
 		$metadatum->set_metadata_type($args['type']);
-		$metadatum->set_metadata_type_options($args['options']);
+		if ( isset($args['options']) ) {
+			$metadatum->set_metadata_type_options($args['options']);
+		}
 		$metadatum->set_status('publish');
 		if ($metadatum->validate()) {
 			$metadatum = $this->metadata_repo->insert($metadatum);

From b43a599a43f55fa1df4a2d0987fcb087d386b806 Mon Sep 17 00:00:00 2001
From: Mateus Machado Luna <mateus.m.luna@gmail.com>
Date: Tue, 28 Aug 2018 13:35:37 -0300
Subject: [PATCH 09/12] Css adjustments.

---
 src/admin/admin.vue                           |  2 +-
 src/admin/components/lists/events-list.vue    |  2 +-
 src/admin/components/lists/items-list.vue     |  4 ++--
 src/admin/components/lists/processes-list.vue |  2 +-
 .../components/lists/taxonomies-list.vue      |  2 +-
 src/admin/pages/lists/items-page.vue          |  2 +-
 src/admin/scss/_buttons.scss                  |  7 +++++++
 src/admin/scss/_checkboxes.scss               |  2 +-
 .../scss/_dropdown-and-autocomplete.scss      |  6 ++++--
 src/admin/scss/_inputs.scss                   | 12 ++++++++---
 src/admin/scss/_modals.scss                   |  4 +++-
 .../scss/_repository-level-overrides.scss     |  2 +-
 src/admin/scss/_switches.scss                 |  2 +-
 src/admin/scss/_tables.scss                   |  6 ++++++
 src/admin/scss/_tags.scss                     |  2 ++
 src/admin/scss/_tainacan-form.scss            | 13 +++++-------
 src/admin/scss/_view-mode-cards.scss          | 20 +++++++++---------
 src/admin/scss/_view-mode-grid.scss           | 17 +++++++--------
 src/admin/scss/_view-mode-records.scss        | 21 ++++++++++++-------
 19 files changed, 77 insertions(+), 51 deletions(-)

diff --git a/src/admin/admin.vue b/src/admin/admin.vue
index 68f9dc0ce..f9d323ad7 100644
--- a/src/admin/admin.vue
+++ b/src/admin/admin.vue
@@ -28,7 +28,7 @@
     import TainacanHeader from './components/navigation/tainacan-header.vue';
     import TainacanRepositorySubheader from './components/navigation/tainacan-repository-subheader.vue';
 
-    export default {
+    export default { 
         name: "AdminPage",
         data(){
             return {
diff --git a/src/admin/components/lists/events-list.vue b/src/admin/components/lists/events-list.vue
index cf5bb34c5..71009b9d8 100644
--- a/src/admin/components/lists/events-list.vue
+++ b/src/admin/components/lists/events-list.vue
@@ -1,7 +1,7 @@
 <template>
     <div class="table-container">
          <div class="table-wrapper">
-            <table class="tainacan-table">
+            <table class="tainacan-table is-narrow">
                 <thead>
                     <tr>
                         <!-- Title -->
diff --git a/src/admin/components/lists/items-list.vue b/src/admin/components/lists/items-list.vue
index c583815d1..a55f96bb1 100644
--- a/src/admin/components/lists/items-list.vue
+++ b/src/admin/components/lists/items-list.vue
@@ -275,7 +275,7 @@
             <!-- RECORDS VIEW MODE -->
             <masonry 
                     :cols="{default: 4, 1919: 3, 1407: 2, 1215: 2, 1023: 1, 767: 1, 343: 1}"
-                    :gutter="42"
+                    :gutter="30"
                     class="tainacan-records-container"
                     v-if="viewMode == 'records'">
                 <div 
@@ -691,7 +691,7 @@ export default {
             }
         },
         getLimitedDescription(description) {
-            let maxCharacter = (window.innerWidth || document.documentElement.clientWidth || document.body.clientWidth) <= 480 ? 100 : 220;
+            let maxCharacter = (window.innerWidth || document.documentElement.clientWidth || document.body.clientWidth) <= 480 ? 100 : 210;
             return description.length > maxCharacter ? description.substring(0, maxCharacter - 3) + '...' : description;
         }
     }
diff --git a/src/admin/components/lists/processes-list.vue b/src/admin/components/lists/processes-list.vue
index e065536fa..02890286d 100644
--- a/src/admin/components/lists/processes-list.vue
+++ b/src/admin/components/lists/processes-list.vue
@@ -35,7 +35,7 @@
         </div>
 -->
         <div class="table-wrapper">
-            <table class="tainacan-table">
+            <table class="tainacan-table is-narrow">
                 <thead>
                     <tr>
                         <!-- Checking list -->
diff --git a/src/admin/components/lists/taxonomies-list.vue b/src/admin/components/lists/taxonomies-list.vue
index 257c4d773..e560f1d74 100644
--- a/src/admin/components/lists/taxonomies-list.vue
+++ b/src/admin/components/lists/taxonomies-list.vue
@@ -36,7 +36,7 @@
         </div>
 
         <div class="table-wrapper">
-            <table class="tainacan-table">
+            <table class="tainacan-table is-narrow">
                 <thead>
                     <tr>
                         <!-- Checking list -->
diff --git a/src/admin/pages/lists/items-page.vue b/src/admin/pages/lists/items-page.vue
index 8885aa5f8..897e0c883 100644
--- a/src/admin/pages/lists/items-page.vue
+++ b/src/admin/pages/lists/items-page.vue
@@ -571,7 +571,7 @@
                                 class="button is-secondary"
                                 :to="{ path: $routerHelper.getNewItemPath(collectionId) }">
                             {{ $i18n.getFrom('items', 'add_new') }}
-                        </router-link>
+                        </router-link> 
                     </div>
                 </section>
 
diff --git a/src/admin/scss/_buttons.scss b/src/admin/scss/_buttons.scss
index c0b1efba3..d71055c14 100644
--- a/src/admin/scss/_buttons.scss
+++ b/src/admin/scss/_buttons.scss
@@ -9,6 +9,7 @@
     display: inline-flex !important;
     cursor: pointer;
     background-color: white;
+    text-shadow: none !important;
 
     &.is-secondary:hover, &.is-secondary:focus {
         background: $secondary !important;
@@ -39,6 +40,12 @@
         cursor: not-allowed !important;
         color: $gray4 !important;
         background-color: $gray2 !important;
+        text-shadow: none !important;
+    }
+    &.is-outlined[disabled], &.is-outlined:hover[disabled] {
+        border: 1px solid $gray2 !important;
+        color: $gray4 !important;
+        background-color: white !important;
     }
     &.is-white[disabled] {
         background-color: white !important;
diff --git a/src/admin/scss/_checkboxes.scss b/src/admin/scss/_checkboxes.scss
index 9cb7db9a0..f662ec03d 100644
--- a/src/admin/scss/_checkboxes.scss
+++ b/src/admin/scss/_checkboxes.scss
@@ -8,7 +8,7 @@
         width: 12px;
         height: 12px;
         flex-shrink: 0;
-        border-radius: 1px;
+        border-radius: 2px;
         display: inline-block;
         border: 1px solid $gray4;
         transition: background 150ms ease-out;
diff --git a/src/admin/scss/_dropdown-and-autocomplete.scss b/src/admin/scss/_dropdown-and-autocomplete.scss
index 6bfab6b38..ce9b9abc5 100644
--- a/src/admin/scss/_dropdown-and-autocomplete.scss
+++ b/src/admin/scss/_dropdown-and-autocomplete.scss
@@ -91,8 +91,10 @@
     }
 }
 .selected-list-box {
-    padding: 4px 0px;
-    border: 1px solid $gray2;
+    padding: 4px 6px 0px 6px;
+    border-color: $gray2;
+    border-width: 0px 1px 1px 1px;
+    border-style: solid;
     background-color: white;
     display: flex;
     font-size: 12px;
diff --git a/src/admin/scss/_inputs.scss b/src/admin/scss/_inputs.scss
index 9ff0268d2..89875bdba 100644
--- a/src/admin/scss/_inputs.scss
+++ b/src/admin/scss/_inputs.scss
@@ -1,18 +1,24 @@
 .input, .textarea {
     font-size: 14px;
-    border: none;
     border-radius: 1px !important;
     box-shadow: none !important;
+    background-color: white;
+    border: 1px solid $gray2;
+    color: $tainacan-input-color;
+    transition: background-color 0.1s, border 0.3s;
 
     &:focus, &:active {
         box-shadow: none !important;
         background-color: white;
-        border: 1px solid $gray2 !important;
-    }    
+        border: 1px solid $gray4 !important;
+    }  
     &[disabled] {
         color: $gray4 !important;
         background-color: $gray2 !important;
     }
+    &.readonly, &[readonly] {
+        background-color: white !important;
+    }
     &::placeholder,
     &::-ms-input-placeholder,
     &:-ms-input-placeholder,
diff --git a/src/admin/scss/_modals.scss b/src/admin/scss/_modals.scss
index 618f080c9..de8d6ae47 100644
--- a/src/admin/scss/_modals.scss
+++ b/src/admin/scss/_modals.scss
@@ -108,7 +108,9 @@
                 box-shadow: none !important;
                 display: inline-flex !important;
                 cursor: pointer;
-                font-size: 13px !important;
+                height: 30px !important;
+                line-height: 20px !important;
+                font-size: 14px !important;
             }
             .button.is-success {
                 border: none;
diff --git a/src/admin/scss/_repository-level-overrides.scss b/src/admin/scss/_repository-level-overrides.scss
index 3c77ef84d..39a02e83c 100644
--- a/src/admin/scss/_repository-level-overrides.scss
+++ b/src/admin/scss/_repository-level-overrides.scss
@@ -11,7 +11,7 @@
     }
 
     .is-primary:not(.upload-draggable), .is-primary:hover, .is-primary:focus {
-        background-color: $blue4 !important;
+        background-color: $blue3 !important;
         color: white !important;
     }
     .is-secondary, .is-secondary:hover, .is-secondary:focus {
diff --git a/src/admin/scss/_switches.scss b/src/admin/scss/_switches.scss
index e20a2e276..4f6e20451 100644
--- a/src/admin/scss/_switches.scss
+++ b/src/admin/scss/_switches.scss
@@ -13,7 +13,7 @@
         }
     } 
     &:hover input[type="checkbox"]:checked + .check {
-        background-color: $turquoise2;
+        background-color: transparent;
     }
 
     input[type="checkbox"]:checked + .check {
diff --git a/src/admin/scss/_tables.scss b/src/admin/scss/_tables.scss
index e3d8877b7..2c155077f 100644
--- a/src/admin/scss/_tables.scss
+++ b/src/admin/scss/_tables.scss
@@ -20,6 +20,12 @@
             border-collapse: separate;
             border-spacing: 0;
 
+            &.is-narrow {
+                tbody td {
+                    height: 42px;
+                }
+            }
+
             .checkbox-cell {
                 min-width: 38px;
                 width: 38px;
diff --git a/src/admin/scss/_tags.scss b/src/admin/scss/_tags.scss
index 8cddb6849..e07fa3d8e 100644
--- a/src/admin/scss/_tags.scss
+++ b/src/admin/scss/_tags.scss
@@ -24,9 +24,11 @@
         &.is-delete {
             border-radius: 50px !important;
             border-radius: 50px !important;
+            color: $gray4;
 
             &:hover {
                 background-color: white !important;
+                color: $gray5;
             }
         }
     }
diff --git a/src/admin/scss/_tainacan-form.scss b/src/admin/scss/_tainacan-form.scss
index f8d32d13c..d7db7cc68 100644
--- a/src/admin/scss/_tainacan-form.scss
+++ b/src/admin/scss/_tainacan-form.scss
@@ -38,16 +38,10 @@
             color: $danger;
         }
     }
-    .input, .textarea {
-        background-color: white;
-        border: 1px solid $gray2;
-        color: $tainacan-input-color;
-        transition: background-color 0.1s;
-
+    .input, .textarea { 
         &.is-danger {
-          background-color: #e7dede;  
+          background-color: $red1;  
         }
-
     }
     .control{
         &.has-icons-right {
@@ -111,6 +105,9 @@
         }
     }
     .upload-draggable {
+        color: $gray4;
+        transition: color 0.2s background-color 0.2s;
         border-radius: 0 !important;
+        border-color: $gray2;
     }
 }
\ No newline at end of file
diff --git a/src/admin/scss/_view-mode-cards.scss b/src/admin/scss/_view-mode-cards.scss
index 721523e0a..7041a0b42 100644
--- a/src/admin/scss/_view-mode-cards.scss
+++ b/src/admin/scss/_view-mode-cards.scss
@@ -19,11 +19,11 @@
     .tainacan-card {
         padding: 0px;
         flex-basis: 0;
-        margin: 0.75rem;
+        margin: 15px;
         max-width: 425px;
         min-width: 425px;
-        min-height: 218px;
-        max-height: 218px;
+        min-height: 210px;
+        max-height: 210px;
         cursor: pointer;
 
         @media screen and (max-width: 480px) {
@@ -45,12 +45,12 @@
         .card-checkbox {
             position: absolute;
             margin-left: 1.0rem;
-            margin-top: 9px;
+            margin-top: 7px;
         }
         .actions-area {
             position: relative;
             float: right;
-            top: -7px;
+            top: -5px;
             padding-right: 12px;
             width: 80px;
             display: flex;
@@ -73,15 +73,15 @@
             visibility: hidden;
         }
         img {
-            width: 172px;
-            height: 172px;
+            width: 170px;
+            height: 170px;
             border-radius: 0px;
         }
         .metadata-title {
             flex-shrink: 0;
-            padding: 0.75rem 4.75rem 0.75rem 2.75rem;
+            padding: 0.5rem 4.75rem 0.5rem 2.75rem;
             margin-bottom: -27px;
-            min-height: 42px;
+            min-height: 40px;
             font-size: 0.875rem !important; 
             text-overflow: ellipsis;
             white-space: nowrap;
@@ -105,7 +105,7 @@
                 padding: 0.75rem 1.375rem;
                 flex: 1;
                 width: 100%;
-                height: 172px;
+                height: 170px;
                 color: $gray4;
 
                 p.metadata-description {
diff --git a/src/admin/scss/_view-mode-grid.scss b/src/admin/scss/_view-mode-grid.scss
index 1f3c2ff93..d29b062a7 100644
--- a/src/admin/scss/_view-mode-grid.scss
+++ b/src/admin/scss/_view-mode-grid.scss
@@ -14,9 +14,9 @@
     }
 
     .tainacan-grid-item {
-        max-width: 258px;
+        max-width: 255px;
         flex-basis: 0;
-        margin: 1rem 1.5rem 1.5rem 1.5rem;
+        margin: 15px;
         text-align: center;
         height: 100%;
 
@@ -33,11 +33,12 @@
             position: relative;
             float: right;
             width: 100%;
+            height: 30px;
             display: flex;
             justify-content: flex-end;
             visibility: hidden;
             opacity: 0;
-            padding: 8px;
+            padding: 3px 8px;
             transition: visibility 0.2s, opacity 0.2s;
             margin-top: -43px;
             background-color: $gray2;
@@ -51,16 +52,12 @@
         &:hover .actions-area { 
             visibility: visible;
             opacity: 1.0;
-        }
-        &.selected-grid-item {
-            .actions-area {
-                background-color: $turquoise1;
-            }
+            background-color: $gray2 !important;
         }
 
         img {
-            min-width: 258px;
-            max-width: 258px;
+            min-width: 255px;
+            max-width: 255px;
             height: auto;
             border-radius: 0px;
             margin-bottom: -5px;
diff --git a/src/admin/scss/_view-mode-records.scss b/src/admin/scss/_view-mode-records.scss
index d58eee301..08a73868a 100644
--- a/src/admin/scss/_view-mode-records.scss
+++ b/src/admin/scss/_view-mode-records.scss
@@ -14,6 +14,9 @@
         .metadata-title {
             background-color: $turquoise2;
         }
+        .media {
+            background-color: $turquoise1;
+        }
     }
 
     .tainacan-record {
@@ -35,12 +38,12 @@
         .record-checkbox {
             position: absolute;
             margin-left: 1.0rem;
-            margin-top: 9px;
+            margin-top: 7px;
         }
         .actions-area {
             position: relative;
             float: right;
-            top: -7px;
+            top: -5px;
             padding-right: 12px;
             width: 80px;
             display: flex;
@@ -64,7 +67,7 @@
         }
         .thumbnail {
             float: right;
-            margin: 0 0 1.5rem 1.5rem;
+            margin: 0 0 25px 25px;
         }
         img {
             width: 120px;
@@ -73,9 +76,9 @@
         }
         .metadata-title {
             flex-shrink: 0;
-            padding: 0.75rem 4.75rem 0.75rem 2.75rem;
+            padding: 0.5rem 4.75rem 0.5rem 2.75rem;
             margin-bottom: -27px;
-            min-height: 42px;
+            min-height: 40px;
             font-size: 0.875rem !important; 
             text-overflow: ellipsis;
             white-space: nowrap;
@@ -83,6 +86,7 @@
             color: black;
 
             p {
+                line-height: 1.7;
                 font-size: 0.875rem !important; 
                 text-overflow: ellipsis;
                 white-space: nowrap;
@@ -93,12 +97,15 @@
         &:hover .metadata-title {
             background-color: $gray2 !important;
         }
+        &:hover .media {
+            background-color: $gray1 !important;
+        }
 
         .media {
             width: 100%;
 
             .list-metadata {
-                padding: 0.75rem 1.375rem;
+                padding: 25px;
                 flex: 1;
                 font-size: 0.6875rem;
                 color: $gray4;
@@ -107,12 +114,12 @@
             
                 .metadata-label {
                     font-size: 0.75rem;
-                    margin-bottom: 0.2rem;
                     font-weight: 700;
                     color: #454647;
                 }
 
                 .metadata-value {
+                    font-size: 0.75rem;
                     margin-bottom: 1rem;
                     color: $gray4;
                 }

From 9ac1ab17f181b2a648861aa7dad8fe7285c91eef Mon Sep 17 00:00:00 2001
From: Leo Germani <leogermani@gmail.com>
Date: Tue, 28 Aug 2018 14:21:33 -0300
Subject: [PATCH 10/12] temporarily removing nonce check from bh process

---
 src/classes/libs/wp-background-process.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/classes/libs/wp-background-process.php b/src/classes/libs/wp-background-process.php
index 6c3652e71..de8ccc17b 100644
--- a/src/classes/libs/wp-background-process.php
+++ b/src/classes/libs/wp-background-process.php
@@ -178,7 +178,7 @@ if ( ! class_exists( 'WP_Background_Process' ) ) {
 				wp_die();
 			}
 			$this->debug('checking nonce');
-			check_ajax_referer( $this->identifier, 'nonce' );
+			//check_ajax_referer( $this->identifier, 'nonce' );
 			$this->debug('ok!');
 			$this->handle();
 

From 4c95763424a28fb0c0470d2ed6353e5a13a6c397 Mon Sep 17 00:00:00 2001
From: Mateus Machado Luna <mateus.m.luna@gmail.com>
Date: Tue, 28 Aug 2018 14:44:01 -0300
Subject: [PATCH 11/12] Css change to allow customizer to override link color
 on theme.

---
 src/admin/components/lists/items-list.vue | 4 ++--
 src/admin/scss/_view-mode-masonry.scss    | 2 +-
 src/admin/scss/_view-mode-records.scss    | 2 +-
 src/admin/theme-items-list.vue            | 6 +++---
 4 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/admin/components/lists/items-list.vue b/src/admin/components/lists/items-list.vue
index a55f96bb1..a87b88fc5 100644
--- a/src/admin/components/lists/items-list.vue
+++ b/src/admin/components/lists/items-list.vue
@@ -105,7 +105,7 @@
             <masonry 
                     v-if="viewMode == 'masonry'"
                     :cols="{default: 7, 1919: 6, 1407: 5, 1215: 4, 1023: 3, 767: 2, 343: 1}"
-                    :gutter="30"
+                    :gutter="25"
                     class="tainacan-masonry-container">
                 <div
                         :key="index"
@@ -275,7 +275,7 @@
             <!-- RECORDS VIEW MODE -->
             <masonry 
                     :cols="{default: 4, 1919: 3, 1407: 2, 1215: 2, 1023: 1, 767: 1, 343: 1}"
-                    :gutter="30"
+                    :gutter="30" 
                     class="tainacan-records-container"
                     v-if="viewMode == 'records'">
                 <div 
diff --git a/src/admin/scss/_view-mode-masonry.scss b/src/admin/scss/_view-mode-masonry.scss
index 5ed9b95f8..94ff7785a 100644
--- a/src/admin/scss/_view-mode-masonry.scss
+++ b/src/admin/scss/_view-mode-masonry.scss
@@ -17,7 +17,7 @@
         display: block;
         width: 100%;
         flex-basis: 0;
-        margin-bottom: 30px;
+        margin-bottom: 25px;
         text-align: center;
         text-decoration: none;
         transition: background-color 0.3s;
diff --git a/src/admin/scss/_view-mode-records.scss b/src/admin/scss/_view-mode-records.scss
index 08a73868a..5ee1b4a15 100644
--- a/src/admin/scss/_view-mode-records.scss
+++ b/src/admin/scss/_view-mode-records.scss
@@ -23,7 +23,7 @@
         background-color: #f6f6f6;
         padding: 0px;
         flex-basis: 0;
-        margin: 0 auto 42px auto;
+        margin: 0 auto 30px auto;
         width: 100%;
         max-width: 425px;
         min-height: 100px;
diff --git a/src/admin/theme-items-list.vue b/src/admin/theme-items-list.vue
index 51e6b98fc..96c7a197e 100644
--- a/src/admin/theme-items-list.vue
+++ b/src/admin/theme-items-list.vue
@@ -62,10 +62,10 @@ export default {
         display: flex;
         -webkit-overflow-scrolling: touch;
 
-        a{ color: $secondary !important }
-        a:hover {
+        a, a:not([href]){ color: $secondary }
+        a:hover, a:hover:not([href]) {
             cursor: pointer;
-            color: $secondary !important;
+            color: $secondary;
         }
         ul {
             list-style: none;

From 545d6624b472e2e7854ca6082f648468e9518f2d Mon Sep 17 00:00:00 2001
From: Mateus Machado Luna <mateus.m.luna@gmail.com>
Date: Tue, 28 Aug 2018 16:28:36 -0300
Subject: [PATCH 12/12] More Css adjustments. Disable enabled metadata
 dropwdown when in masonry view mode.

---
 .../advanced-search/advanced-search.vue       |  3 +-
 src/admin/components/other/help-button.vue    |  2 +-
 src/admin/pages/lists/items-page.vue          |  2 +-
 src/admin/pages/lists/term-items-page.vue     |  2 +-
 src/admin/scss/_variables.scss                | 53 +++++++++++++++++++
 src/admin/scss/_view-mode-masonry.scss        |  6 +--
 src/admin/theme-items-list.vue                |  1 +
 .../filter-types/tainacan-filter-item.vue     | 18 +++++--
 src/theme-helper/view-mode-masonry.vue        |  2 +-
 src/theme-helper/view-mode-records.vue        |  2 +-
 10 files changed, 75 insertions(+), 16 deletions(-)

diff --git a/src/admin/components/advanced-search/advanced-search.vue b/src/admin/components/advanced-search/advanced-search.vue
index 6f91fb405..852e27273 100644
--- a/src/admin/components/advanced-search/advanced-search.vue
+++ b/src/admin/components/advanced-search/advanced-search.vue
@@ -137,8 +137,7 @@
                     class="field column is-12">
                 <a
                     @click="addSearchCriteria"
-                    style="font-size: 12px;"
-                    class="has-text-secondary">
+                    style="font-size: 12px;">
                     <b-icon
                             class="add-i"
                             icon="plus-circle"
diff --git a/src/admin/components/other/help-button.vue b/src/admin/components/other/help-button.vue
index 6492ad4b9..0cd9edbe0 100644
--- a/src/admin/components/other/help-button.vue
+++ b/src/admin/components/other/help-button.vue
@@ -35,7 +35,7 @@ export default {
     }
 
     a.help-button .icon {
-        i, i::before { font-size: 0.9em !important; }
+        i, i::before { font-size: 0.875rem !important }
     }
 
     .help-wrapper:hover .help-tooltip {
diff --git a/src/admin/pages/lists/items-page.vue b/src/admin/pages/lists/items-page.vue
index 897e0c883..0bc16d9fe 100644
--- a/src/admin/pages/lists/items-page.vue
+++ b/src/admin/pages/lists/items-page.vue
@@ -169,7 +169,7 @@
                     <b-dropdown
                             ref="displayedMetadataDropdown"
                             :mobile-modal="true"
-                            :disabled="totalItems <= 0 || adminViewMode == 'grid'|| adminViewMode == 'cards'"
+                            :disabled="totalItems <= 0 || adminViewMode == 'grid'|| adminViewMode == 'cards' || adminViewMode == 'masonry'"
                             class="show">
                         <button
                                 class="button is-white"
diff --git a/src/admin/pages/lists/term-items-page.vue b/src/admin/pages/lists/term-items-page.vue
index 0d526e204..c85d9cb2e 100644
--- a/src/admin/pages/lists/term-items-page.vue
+++ b/src/admin/pages/lists/term-items-page.vue
@@ -171,7 +171,7 @@
                     <b-dropdown
                             ref="displayedMetadataDropdown"
                             :mobile-modal="true"
-                            :disabled="totalItems <= 0 || adminViewMode == 'grid'|| adminViewMode == 'cards'"
+                            :disabled="totalItems <= 0 || adminViewMode == 'grid'|| adminViewMode == 'cards' || adminViewMode == 'masonry'"
                             class="show">
                         <button
                                 class="button is-white"
diff --git a/src/admin/scss/_variables.scss b/src/admin/scss/_variables.scss
index 5454af88e..a7c0180a3 100644
--- a/src/admin/scss/_variables.scss
+++ b/src/admin/scss/_variables.scss
@@ -240,3 +240,56 @@ $modal-z: 9999999;
     }
 }
 
+// Filter Item Collapse 
+@keyframes filter-item-in {
+    from {
+        opacity: 0;
+        height: 0px;
+        max-height: 0px;
+        min-height: 0px;
+        -ms-transform: translate(2%, -30%); /* IE 9 */
+        -webkit-transform: translate(2%, -30%); /* Safari */
+        transform: translate(2%, -30%);
+    }
+    to {
+        height: 80px;
+        max-height: 80px;
+        min-height: 80px;
+        opacity: 1;        
+        -ms-transform: translate(0, 0); /* IE 9 */
+        -webkit-transform: translate(0, 0); /* Safari */
+        transform: translate(0, 0);
+    }
+}
+
+@keyframes filter-item-out {
+    from {
+        height: 80px;
+        max-height: 80px;
+        min-height: 80px;
+        opacity: 1;
+        -ms-transform: translate(0, 0); /* IE 9 */
+        -webkit-transform: translate(0, 0); /* Safari */
+        transform: translate(0, 0);
+    }
+    to {
+        height: 0px;
+        max-height: 0px;
+        min-height: 0px;
+        opacity: 0;
+        -ms-transform: translate(2%, -30%); /* IE 9 */
+        -webkit-transform: translate(2%, -30%); /* Safari */
+        transform: translate(2%, -30%);
+    }
+}
+
+.filter-item-enter-active {
+    animation-name: filter-item-in;
+    animation-duration: 0.1s;
+    animation-timing-function: ease;   
+}
+.filter-item-leave-active {
+    animation-name: filter-item-out;
+    animation-duration: 0.1s;
+    animation-timing-function: ease;   
+}
\ No newline at end of file
diff --git a/src/admin/scss/_view-mode-masonry.scss b/src/admin/scss/_view-mode-masonry.scss
index 94ff7785a..fcc2af7c5 100644
--- a/src/admin/scss/_view-mode-masonry.scss
+++ b/src/admin/scss/_view-mode-masonry.scss
@@ -39,12 +39,10 @@
             justify-content: flex-end;
             visibility: hidden;
             opacity: 0;
-            padding: 8px;
+            padding: 2px 8px;
             transition: visibility 0.2s, opacity 0.2s;
-            margin-top: -43px;
+            margin-top: -31px;
             background-color: $gray2;
-            border-bottom-left-radius: 2px;
-            border-bottom-right-radius: 2px;
 
             a {
                 margin-left: 12px;
diff --git a/src/admin/theme-items-list.vue b/src/admin/theme-items-list.vue
index 96c7a197e..8381baddb 100644
--- a/src/admin/theme-items-list.vue
+++ b/src/admin/theme-items-list.vue
@@ -66,6 +66,7 @@ export default {
         a:hover, a:hover:not([href]) {
             cursor: pointer;
             color: $secondary;
+            text-decoration: underline;
         }
         ul {
             list-style: none;
diff --git a/src/classes/filter-types/tainacan-filter-item.vue b/src/classes/filter-types/tainacan-filter-item.vue
index 4f28dc48f..9f8c52d35 100644
--- a/src/classes/filter-types/tainacan-filter-item.vue
+++ b/src/classes/filter-types/tainacan-filter-item.vue
@@ -5,7 +5,8 @@
             :type="filterTypeMessage">
         <b-collapse
                 class="show" 
-                :open="open">
+                :open.sync="open"
+                animation="filter-item">
             <label
                     v-tooltip="{
                                     content: filter.name,
@@ -204,16 +205,23 @@
         }
 
         .taginput-container {
-            //display: table-cell;
             border: none !important;
-            &.is-focusable:active, &.is-focusable:focus  {
+            &.is-focused, 
+            &.is-focused:active, 
+            &.is-focused:focus, 
+            &.is-focusable,
+            &.is-focusable:active 
+            &.is-focusable:focus  {
                 border: none !important;
+                input:active, input:focus {
+                    border: 1px solid $gray4 !important;
+                }
             }    
-            input, input:active, input:focus {
+            input{
                 border: 1px solid $gray2 !important;
             }
             .control.has-icons-left .icon {
-                top: 0px !important;
+                top: 3px !important;
             }
             .tags {
                 display: none !important;
diff --git a/src/theme-helper/view-mode-masonry.vue b/src/theme-helper/view-mode-masonry.vue
index 081a80fda..9ec611f16 100644
--- a/src/theme-helper/view-mode-masonry.vue
+++ b/src/theme-helper/view-mode-masonry.vue
@@ -17,7 +17,7 @@
             <!-- RECORDS VIEW MODE -->
             <masonry 
                     :cols="{default: 7, 1919: 6, 1407: 5, 1215: 4, 1023: 3, 767: 2, 343: 1}"
-                    :gutter="30"
+                    :gutter="25"
                     class="tainacan-masonry-container">
                 <a 
                         :key="index"
diff --git a/src/theme-helper/view-mode-records.vue b/src/theme-helper/view-mode-records.vue
index ff2bc5619..0f9b8f418 100644
--- a/src/theme-helper/view-mode-records.vue
+++ b/src/theme-helper/view-mode-records.vue
@@ -17,7 +17,7 @@
             <!-- RECORDS VIEW MODE -->
             <masonry 
                     :cols="{default: 4, 1919: 3, 1407: 2, 1215: 2, 1023: 1, 767: 1, 343: 1}"
-                    :gutter="42"
+                    :gutter="30"
                     class="tainacan-records-container">
                 <a 
                         :href="item.url"