Skip to content

Commit

Permalink
fix: improve tooltip info, examples in the domain settings and doc ab…
Browse files Browse the repository at this point in the history
…out 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 <[email protected]>
  • Loading branch information
MaferMazu and julianramirez2 authored Oct 12, 2023
1 parent e32f419 commit 2857e61
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 5 deletions.
5 changes: 4 additions & 1 deletion admin/css/openedx-woocommerce-plugin-admin.css
Original file line number Diff line number Diff line change
Expand Up @@ -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 {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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>

Expand Down
4 changes: 2 additions & 2 deletions admin/views/class-openedx-woocommerce-plugin-settings.php
Original file line number Diff line number Diff line change
Expand Up @@ -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
}
Expand Down Expand Up @@ -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
}
Expand Down

0 comments on commit 2857e61

Please sign in to comment.