From 2857e61834f393a52855f2410e9ff25219de6174 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Mar=C3=ADa=20Fernanda=20Magallanes?=
 <35668326+MaferMazu@users.noreply.github.com>
Date: Thu, 12 Oct 2023 16:08:47 -0500
Subject: [PATCH] fix: improve tooltip info, examples in the domain settings
 and doc about save before generate token (#48)

* docs: add tooltip info, examples in the domain settings and save before generate token

* style: fixed tooltip text box css

---------

Co-authored-by: Julian Ramirez <jramirezg@eafit.edu.co>
---
 admin/css/openedx-woocommerce-plugin-admin.css               | 5 ++++-
 ...class-openedx-woocommerce-plugin-enrollment-info-form.php | 4 ++--
 admin/views/class-openedx-woocommerce-plugin-settings.php    | 4 ++--
 3 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/admin/css/openedx-woocommerce-plugin-admin.css b/admin/css/openedx-woocommerce-plugin-admin.css
index cd073e1..4135ae0 100644
--- a/admin/css/openedx-woocommerce-plugin-admin.css
+++ b/admin/css/openedx-woocommerce-plugin-admin.css
@@ -108,10 +108,13 @@
     color: white;
     font-size: bold;
     border-radius: 5px;
+    width: 250px;
+    position: absolute;
+    margin-left: 5px;
 }
 
 .tooltip-icon:hover + .tooltip-text {
-    display: unset;
+    display: inline-block;
 }
 
 .setting_input {
diff --git a/admin/views/class-openedx-woocommerce-plugin-enrollment-info-form.php b/admin/views/class-openedx-woocommerce-plugin-enrollment-info-form.php
index 26f9d8d..9f00b4d 100644
--- a/admin/views/class-openedx-woocommerce-plugin-enrollment-info-form.php
+++ b/admin/views/class-openedx-woocommerce-plugin-enrollment-info-form.php
@@ -201,13 +201,13 @@ public function render_enrollment_info_form( $post ) {
 								<input class="action-checkbox" type="checkbox" id="force" name="enrollment_force" value="opcion1">
 								<label for="force"><?php esc_html_e( 'Use the "force" flag', 'wp-openedx-woocommerce-plugin' ); ?></label>
 								<span class="tooltip-icon">?</span>
-								<span class="tooltip-text"><?php esc_html_e( 'Lorem ipsum.', 'wp-openedx-woocommerce-plugin' ); ?></span>
+								<span class="tooltip-text"><?php esc_html_e( "Disregard the course's enrollment end dates.", 'wp-openedx-woocommerce-plugin' ); ?></span>
 							</td>
 							<td>
 								<input class="action-checkbox" type="checkbox" id="no_pre" name="enrollment_allowed" value="opcion1">
 								<label for="no_pre"><?php esc_html_e( "Create course enrollment allowed if the user doesn't exist", 'wp-openedx-woocommerce-plugin' ); ?></label>
 								<span class="tooltip-icon">?</span>
-								<span class="tooltip-text"><?php esc_html_e( 'Lorem ipsum.', 'wp-openedx-woocommerce-plugin' ); ?></span>
+								<span class="tooltip-text"><?php esc_html_e( 'Creates a register in the table Course Enrollment Allowed if the email we use in the request is not a user in our Open edX platform yet.', 'wp-openedx-woocommerce-plugin' ); ?></span>
 							</td>
 						</tr>
 
diff --git a/admin/views/class-openedx-woocommerce-plugin-settings.php b/admin/views/class-openedx-woocommerce-plugin-settings.php
index eb1994c..2c84bdf 100644
--- a/admin/views/class-openedx-woocommerce-plugin-settings.php
+++ b/admin/views/class-openedx-woocommerce-plugin-settings.php
@@ -218,7 +218,7 @@ public function openedx_domain_callback() {
 		<input class="setting_input" type="text" name="openedx-domain" id="openedx-domain" 
 			value="<?php echo esc_attr( $value ); ?>" required />
 
-		<p class='description'>Your Open edX platform's web address.</p>
+		<p class='description'>Your LMS Open edX platform's web address. e.g. https://lms.nightly.dedalo.edunext.link or http://local.overhang.io:8000</p>
 
 		<?php
 	}
@@ -299,7 +299,7 @@ public function openedx_jwt_token_callback() {
 
 		</div>
 
-		<p class="description"> Select the Generate Token button to obtain a JWT Token. </p>
+		<p class="description"> Select the Generate Token button to obtain a JWT Token. Remember to save your changes before creating the JWT token.</p>
 
 		<?php
 	}