[Bug 40418] New: Update the item type translation process to avoid Greybox modal
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40418 Bug ID: 40418 Summary: Update the item type translation process to avoid Greybox modal Change sponsored?: --- Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: System Administration Assignee: oleonard@myacpl.org Reporter: oleonard@myacpl.org QA Contact: testopia@bugs.koha-community.org CC: caroline.cyr-la-rose@inlibro.com, gmcharlt@gmail.com, testopia@bugs.koha-community.org Depends on: 14100 I propose that we update the item type translation process in order to avoid using a Greybox modal. We can just as easily move from page to page as the process progresses. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14100 [Bug 14100] Add base to make item types, branch names, sysprefs values and authorised values translatable -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40418 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40418 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |40422 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40422 [Bug 40422] Remove Greybox assets from the staff interface -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40418 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart@gmail.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40418 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff Patch complexity|--- |Small patch -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40418 --- Comment #1 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 184188 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=184188&action=edit Bug 40418: Update the item type translation process to avoid Greybox modal This patch updates the item type translation process so that it takes place on a separate standalone page instead of in a Greybox modal. Although currently only item types can be translated in this way, the original plan was to expand the range of data that could be translated. For this reason I used [% SWITCH %] in localization.tt for determining whether to use the term "item type." This accommodates, I hope, expansion of the feature. To test you must have more than one installed translation. - Apply the patch and go to Administration -> Item types. - Edit an item type. - On the "Description" line, click "Translate into other languages." - You should be redirected to a new page, "Add translations for item type 'XXX.' - Confirm that the page title, breadcrumbs, and heading are correct. - Test the process of adding and deleting a translation. - Test the process of performing an inline edit of an existing translation by clicking the table cell. - Test that the "Return to item type 'XXX'" button returns you to the item type you were editing. Sponsored-by: Athens County Public Libraries -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40418 --- Comment #2 from Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> --- I had some trouble with "Test the process of performing an inline edit of an existing translation by clicking the table cell." I had never noticed this was possible! It seems you can't modify it right after adding it? - From the item types list, click "Edit" next to Maps - Click "Translate into other languages" - Choose language (fr-CA) - Enter "Cartes" in translation - Click "Add" - Double click "Cartes" to change it --> Nothing happens - Click "Return to item type 'MP'" - Click "Translate into other languages" - Double click "Cartes" to change it --> Now it's OK, but not very visible that the cell is editable now - Change to "Cartes géographiques" - Click outside the table I wonder if this is a functionality worth keeping. It's very rare that table cells are editable in Koha in general (only point of sale uses it I think?) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40418 --- Comment #3 from Owen Leonard <oleonard@myacpl.org> --- I think you're absolutely right that the feature is unusual. I find that the issue with not being able to edit a newly-added translation happens in master as well. I think these are issues which should be addressed outside this bug. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40418 Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40418 Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #184188|0 |1 is obsolete| | --- Comment #4 from Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> --- Created attachment 184376 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=184376&action=edit Bug 40418: Update the item type translation process to avoid Greybox modal This patch updates the item type translation process so that it takes place on a separate standalone page instead of in a Greybox modal. Although currently only item types can be translated in this way, the original plan was to expand the range of data that could be translated. For this reason I used [% SWITCH %] in localization.tt for determining whether to use the term "item type." This accommodates, I hope, expansion of the feature. To test you must have more than one installed translation. - Apply the patch and go to Administration -> Item types. - Edit an item type. - On the "Description" line, click "Translate into other languages." - You should be redirected to a new page, "Add translations for item type 'XXX.' - Confirm that the page title, breadcrumbs, and heading are correct. - Test the process of adding and deleting a translation. - Test the process of performing an inline edit of an existing translation by clicking the table cell. - Test that the "Return to item type 'XXX'" button returns you to the item type you were editing. Sponsored-by: Athens County Public Libraries Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40418 --- Comment #5 from Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> --- Alright! I added my so -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40418 --- Comment #6 from Jonathan Druart <jonathan.druart@gmail.com> --- (In reply to Owen Leonard from comment #3)
I think you're absolutely right that the feature is unusual. I find that the issue with not being able to edit a newly-added translation happens in master as well.
Yes, it has always been like that. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40418 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |40453 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40453 [Bug 40453] Allow newly-added item type translations to be edited -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40418 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #7 from Jonathan Druart <jonathan.druart@gmail.com> --- 1. There is a JS error: "Uncaught TypeError: antiClickjack is null" on http://localhost:8081/cgi-bin/koha/admin/localization.pl?entity=itemtypes&code=BK 2. "Return to item type" is a button, shouldn't it be a link instead? 3. When I edit and save: Entity itemtypes (code BK) for lang fr-FR has correctly been updated with 'Français (fr-FR)English (en)हिन्दी (hi)Deutsch (de-DE)Català (ca-ES)Español (es-ES)Svenska (sv-SE)' 4. The confirmation message shows up then hides. Shouldn't we keep it displayed? (not a behaviour change) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40418 --- Comment #8 from Owen Leonard <oleonard@myacpl.org> --- (In reply to Jonathan Druart from comment #7)
1. There is a JS error: "Uncaught TypeError: antiClickjack is null"
Will fix in a follow-up.
2. "Return to item type" is a button, shouldn't it be a link instead?
We're not consistent, and there isn't an official guideline for this. Sometimes we use a button and sometimes we use a link.
3. When I edit and save: Entity itemtypes (code BK) for lang fr-FR has correctly been updated with 'Français (fr-FR)English (en)हिन्दी (hi)Deutsch (de-DE)Català (ca-ES)Español (es-ES)Svenska (sv-SE)'
This is an existing bug which is fixed in Bug 40453
4. The confirmation message shows up then hides. Shouldn't we keep it displayed? (not a behaviour change)
I think that's outside the scope of this bug, but worth a discussion. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40418 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40418 --- Comment #9 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 184447 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=184447&action=edit Bug 40418: (follow-up) Remove popup flag from INCLUDE -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40418 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|testopia@bugs.koha-communit |jonathan.druart@gmail.com |y.org | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40418 David Flater <flaterdavid@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #184447|0 |1 is obsolete| | --- Comment #10 from David Flater <flaterdavid@gmail.com> --- Created attachment 185105 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185105&action=edit Bug 40418: (follow-up) Remove popup flag from INCLUDE Signed-off-by: David Flater <flaterdavid@gmail.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40418 David Flater <flaterdavid@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |flaterdavid@gmail.com Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40418 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40418 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #184376|0 |1 is obsolete| | Attachment #185105|0 |1 is obsolete| | --- Comment #11 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 186124 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=186124&action=edit Bug 40418: Update the item type translation process to avoid Greybox modal This patch updates the item type translation process so that it takes place on a separate standalone page instead of in a Greybox modal. Although currently only item types can be translated in this way, the original plan was to expand the range of data that could be translated. For this reason I used [% SWITCH %] in localization.tt for determining whether to use the term "item type." This accommodates, I hope, expansion of the feature. To test you must have more than one installed translation. - Apply the patch and go to Administration -> Item types. - Edit an item type. - On the "Description" line, click "Translate into other languages." - You should be redirected to a new page, "Add translations for item type 'XXX.' - Confirm that the page title, breadcrumbs, and heading are correct. - Test the process of adding and deleting a translation. - Test the process of performing an inline edit of an existing translation by clicking the table cell. - Test that the "Return to item type 'XXX'" button returns you to the item type you were editing. Sponsored-by: Athens County Public Libraries Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40418 --- Comment #12 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 186125 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=186125&action=edit Bug 40418: (follow-up) Remove popup flag from INCLUDE Signed-off-by: David Flater <flaterdavid@gmail.com> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40418 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Version(s)| |25.11.00 released in| | Status|Passed QA |Pushed to main -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40418 --- Comment #13 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Nice work everyone! Pushed to main for 25.11 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40418 Laura Escamilla <Laura.escamilla@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to main |Needs documenting CC| |Laura.escamilla@bywatersolu | |tions.com --- Comment #14 from Laura Escamilla <Laura.escamilla@bywatersolutions.com> --- This is an enhancement and will not be backported to the 25.05.x branch. -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org