Attachment 151982 Details for Bug 33883 – Bug 33883: "Make sure to copy your API secret" message overlaps text

[patch] Bug 33883: "Make sure to copy your API secret" message overlaps text

Bug-33883-Make-sure-to-copy-your-API-secret-messag.patch (text/plain), 3.34 KB, created by Owen Leonard on 2023-06-02 14:08:43 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Owen Leonard

Created: 2023-06-02 14:08:43 UTC

Size: 3.34 KB

patch

obsolete

>From 20c00d8c9386c3cd4472c0671a770888934ed54b Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Fri, 2 Jun 2023 14:00:34 +0000
>Subject: [PATCH] Bug 33883: "Make sure to copy your API secret" message
> overlaps text
>MIME-Version: 1.0
>Content-Type: text/plain; charset=UTF-8
>Content-Transfer-Encoding: 8bit
>
>This patch updates the markup on the patron API keys page so that it
>uses standard classes for dialogs and page sections.
>
>To test you must have RESTOAuth2ClientCredentials enabled.
>
>- Apply the patch and locate a patron record.
>- From their checkout or detail page, choose More -> Manage API keys
>- Click "Generate a new client id/secret pair"
>- Enter a description and click Save.
>- You should see a message in a standard message-style dialog, "Make
>  sure to copy your API secret now. You wonât be able to see it again!"
>- The API key information should be displayed in a .page-section div.
>- The table of existing keys should be displayed in a .page-section div.
>---
> .../prog/en/modules/members/apikeys.tt           | 16 ++++++++--------
> 1 file changed, 8 insertions(+), 8 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/apikeys.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/apikeys.tt
>index fb6167487e..5ac62bba13 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/apikeys.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/apikeys.tt
>@@ -36,8 +36,8 @@
> 
>             [% IF fresh_api_key %]
>                 [%# A fresh api key was generated, display the secret, only once %]
>-                <div class="rows">
>-                    <span class="alert">Make sure to copy your API secret now. You wonât be able to see it again!</span>
>+                <div class="dialog message">Make sure to copy your API secret now. You wonât be able to see it again!</div>
>+                <div class="page-section rows">
>                     <ol>
>                         <li>
>                             <span class="label">Description: </span>
>@@ -72,8 +72,8 @@
>                     </fieldset>
>                 </form>
> 
>-                <div id="keys">
>-                    [% IF api_keys.count %]
>+                [% IF api_keys.count %]
>+                    <div class="page-section" id="keys">
>                         <p>
>                             <button class="btn btn-default toggle_element" type="submit" id="show-api-form" data-element="#add-api-key"><i class="fa fa-plus"></i> Generate a new client id/key pair</button>
>                         </p>
>@@ -117,10 +117,10 @@
>                                 [% END %]
>                             </tbody>
>                         </table>
>-                    [% ELSE %]
>-                        <div class="dialog message">No keys defined for the current patron. <a href="#" class="toggle_element" data-element="#add-api-key" id="show-api-form"><i class="fa fa-plus"></i> Generate a new client id/secret pair</div>
>-                    [% END %]
>-                </div>
>+                    </div>
>+                [% ELSE %]
>+                    <div class="dialog message">No keys defined for the current patron. <a href="#" class="toggle_element" data-element="#add-api-key" id="show-api-form"><i class="fa fa-plus"></i> Generate a new client id/secret pair</div>
>+                [% END %]
>             </main>
>         </div> 
> 
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 33883: 151982 | 151983 | 151985 | 152033