From 0f828dd0bf41aec0f9eb59b49ac2f340a404b4e7 Mon Sep 17 00:00:00 2001
From: Sean Morley <zipsm15@gmail.com>
Date: Mon, 13 Jan 2025 10:51:33 -0500
Subject: [PATCH] fix: update conditionals in AdventureModal and
 CategoryDropdown for improved logic

---
 frontend/src/lib/components/AdventureModal.svelte   | 2 +-
 frontend/src/lib/components/CategoryDropdown.svelte | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/frontend/src/lib/components/AdventureModal.svelte b/frontend/src/lib/components/AdventureModal.svelte
index 2fd2470..322e67f 100644
--- a/frontend/src/lib/components/AdventureModal.svelte
+++ b/frontend/src/lib/components/AdventureModal.svelte
@@ -832,7 +832,7 @@ it would also work to just use on:click on the MapLibre component itself. -->
 											{$t('adventures.mark_visited')}
 										</button>
 									{/if}
-									{#if (willBeMarkedVisited && !reverseGeocodePlace.region_visited) || (!reverseGeocodePlace.city_visited && willBeMarkedVisited)}
+									{#if (willBeMarkedVisited && !reverseGeocodePlace.region_visited && reverseGeocodePlace.region_id) || (!reverseGeocodePlace.city_visited && willBeMarkedVisited && reverseGeocodePlace.city_id)}
 										<div role="alert" class="alert alert-info mt-2">
 											<svg
 												xmlns="http://www.w3.org/2000/svg"
diff --git a/frontend/src/lib/components/CategoryDropdown.svelte b/frontend/src/lib/components/CategoryDropdown.svelte
index d509370..1de8638 100644
--- a/frontend/src/lib/components/CategoryDropdown.svelte
+++ b/frontend/src/lib/components/CategoryDropdown.svelte
@@ -86,7 +86,7 @@
 						bind:value={new_category.icon}
 					/>
 					<button on:click={toggleEmojiPicker} type="button" class="btn btn-secondary">
-						{isEmojiPickerVisible ? $t('adventures.hide') : $t('adventures.hide')}
+						{!isEmojiPickerVisible ? $t('adventures.show') : $t('adventures.hide')}
 						{$t('adventures.emoji_picker')}
 					</button>
 					<button on:click={custom_category} type="button" class="btn btn-primary">