Summary: | New and edit identity provider UIs inconsistent | ||
---|---|---|---|
Product: | Koha | Reporter: | David Cook <dcook> |
Component: | Authentication | Assignee: | David Cook <dcook> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | trivial | ||
Priority: | P5 - low | CC: | dpavlin, tomascohen |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: |
22.11.00
|
|
Circulation function: | |||
Bug Depends on: | 31378 | ||
Bug Blocks: | |||
Attachments: |
Bug 32141: Fix textbox formatting on edit IdP screen
Bug 32141: Add missing fieldset legends and titlecase "matchpoint" Bug 32141: Fix textbox formatting on edit IdP screen Bug 32141: Add missing fieldset legends and titlecase "matchpoint" |
Description
David Cook
2022-11-09 00:37:55 UTC
Created attachment 143455 [details] [review] Bug 32141: Fix textbox formatting on edit IdP screen This patch makes the textbox have the same number of rows and columns on the "Edit" page as it does on the "New" page for Identity Providers. Created attachment 143456 [details] [review] Bug 32141: Add missing fieldset legends and titlecase "matchpoint" This patch adds missing fieldset legends and titlecases "matchpoint", so that the "Edit" UI is consistent with the "New" UI for adding and modifying Identity Providers. Test plan: 1) Apply patches 2) Go to http://localhost:8081/cgi-bin/koha/admin/identity_providers.pl?op=add_form 3) Fill out the fields to add an identity provider 4) Click "Edit" next to the Identity Provider in the list 5) In a new tab, go to http://localhost:8081/cgi-bin/koha/admin/identity_providers.pl?op=add_form 6) Compare the two screens and note that all the labels, inputs, and help text are the same on both screens. (In reply to David Cook from comment #0) > When adding an Identity Provider, the textboxes are 75 columns by 10 rows. > > But when editing an Identity Provider, the boxes don't have explicit sizing, > and they're too small and always need to be manually resized. > > Not a big drama but it's annoying. Yeah, I added that on my last follow-up on the bug, and forgot to tweak the edit page. Thanks! Created attachment 143525 [details] [review] Bug 32141: Fix textbox formatting on edit IdP screen This patch makes the textbox have the same number of rows and columns on the "Edit" page as it does on the "New" page for Identity Providers. Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Created attachment 143526 [details] [review] Bug 32141: Add missing fieldset legends and titlecase "matchpoint" This patch adds missing fieldset legends and titlecases "matchpoint", so that the "Edit" UI is consistent with the "New" UI for adding and modifying Identity Providers. Test plan: 1) Apply patches 2) Go to http://localhost:8081/cgi-bin/koha/admin/identity_providers.pl?op=add_form 3) Fill out the fields to add an identity provider 4) Click "Edit" next to the Identity Provider in the list 5) In a new tab, go to http://localhost:8081/cgi-bin/koha/admin/identity_providers.pl?op=add_form 6) Compare the two screens and note that all the labels, inputs, and help text are the same on both screens. Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Pushed to master for 22.11. Nice work everyone, thanks! |