[Koha-bugs] [Bug 9571] Fix width for textareas in Edit Item screen
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9571 Chris Cormack ch...@bigballofwax.co.nz changed: What|Removed |Added Status|Pushed to Master|Pushed to Stable CC||ch...@bigballofwax.co.nz --- Comment #15 from Chris Cormack ch...@bigballofwax.co.nz --- Pushed to 3.10.x and 3.8.x will be in 3.10.5 and 3.8.12 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9571] Fix width for textareas in Edit Item screen
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9571 --- Comment #13 from Jared Camins-Esakov jcam...@cpbibliography.com --- (In reply to comment #12) What about doing both? Does the CSS correctly size the fields when the rows and cols are set? I'd like the markup to be valid. Yes. I am satisfied with this. Pushing now. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9571] Fix width for textareas in Edit Item screen
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9571 Jared Camins-Esakov jcam...@cpbibliography.com changed: What|Removed |Added Status|Passed QA |Pushed to Master --- Comment #14 from Jared Camins-Esakov jcam...@cpbibliography.com --- This patch has been pushed to master. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9571] Fix width for textareas in Edit Item screen
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9571 --- Comment #12 from Owen Leonard oleon...@myacpl.org --- What about doing both? Does the CSS correctly size the fields when the rows and cols are set? I'd like the markup to be valid. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9571] Fix width for textareas in Edit Item screen
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9571 --- Comment #11 from Jared Camins-Esakov jcam...@cpbibliography.com --- Owen, Marcel preferred the CSS version. Thoughts? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9571] Fix width for textareas in Edit Item screen
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9571 M. de Rooy m.de.r...@rijksmuseum.nl changed: What|Removed |Added Attachment #15174|0 |1 is obsolete|| --- Comment #9 from M. de Rooy m.de.r...@rijksmuseum.nl --- Created attachment 16834 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=16834action=edit Bug 9571 - Fix width for textareas in Edit Item screen This patch adds a style for textareas on the Edit Item screen of addbiblio.pl so that they are roughly the same length as the other input elements around it on the page. To test: Before applying patch... 1) Create a new item for any biblio with a URL (subfield u) of over 100 characters (you can use www.lettercount.com to count the number of characters in your URL). 2) In any browser (I've already looked at IE, Firefox, and Chrome on Windows), edit the item (either by clicking Edit Items from the Normal view of the bib record then clicking Edit next to the applicable item, or clicking on the Items view in the bib record and clicking Edit item next to the relevant item). 3) Notice that the textarea for the url is absolutely tiny. Difficult to read and difficult to interact with. Apply patch. 1) Notice that the textarea is roughly the same size as the other input elements on the page. Signed-off-by: Kyle M Hall k...@bywatersolutions.com Signed-off-by: Marcel de Rooy m.de.r...@rijksmuseum.nl -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9571] Fix width for textareas in Edit Item screen
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9571 M. de Rooy m.de.r...@rijksmuseum.nl changed: What|Removed |Added Status|Signed Off |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9571] Fix width for textareas in Edit Item screen
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9571 M. de Rooy m.de.r...@rijksmuseum.nl changed: What|Removed |Added Status|Needs Signoff |Signed Off CC||m.de.r...@rijksmuseum.nl -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9571] Fix width for textareas in Edit Item screen
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9571 M. de Rooy m.de.r...@rijksmuseum.nl changed: What|Removed |Added Status|Signed Off |Passed QA QA Contact||m.de.r...@rijksmuseum.nl --- Comment #10 from M. de Rooy m.de.r...@rijksmuseum.nl --- QA Comment: I picked the css variant. Looks just little better in this perspective than hardcoding rows and cols in code. But it is arbitrary. Passed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9571] Fix width for textareas in Edit Item screen
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9571 --- Comment #8 from Owen Leonard oleon...@myacpl.org --- (In reply to comment #7) I think it might be an idea to increase the number of cols to 69 or so, so that it is closer to the length of the other input fields Unfortunately cols doesn't map directly to text inputs' size attribute, and different browsers size them differently. I chose the size I did because it looked correct in Firefox :) Of course the size of both text inputs and textareas could be controlled by CSS, but we already have differently-sized selects and inputs, so I don't think it's much of an issue. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9571] Fix width for textareas in Edit Item screen
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9571 --- Comment #7 from David Cook dc...@prosentient.com.au --- (In reply to comment #3) Created attachment 15173 [details] [review] Bug 9571 [Alternate] Fix width for textareas in Edit Item screen This alternative patch modifies the script where the input markup is built and adds an alternative configuration for textareas. This gives the textareas explicit cols and rows attributes. I think this solution is preferable to a style fix because it solves the bug while eliminating some invalid markup (no size or maxlength attributes on textareas, which are invalid without explicit dimensions). To test, add a nice big chunk of text to an existing item's item field like Public note (952$z), save, and edit again. The field should now be displayed in a more comfortably-sized textarea. I like the idea of using explicit cols and rows. I wasn't sure whether it would be preferable to do that or use CSS, so I went with the latter. I think it might be an idea to increase the number of cols to 69 or so, so that it is closer to the length of the other input fields, but I might just be in favour of that number, because it looks better on my monitor. The same might not be true for others. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9571] Fix width for textareas in Edit Item screen
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9571 --- Comment #3 from Owen Leonard oleon...@myacpl.org --- Created attachment 15173 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=15173action=edit Bug 9571 [Alternate] Fix width for textareas in Edit Item screen This alternative patch modifies the script where the input markup is built and adds an alternative configuration for textareas. This gives the textareas explicit cols and rows attributes. I think this solution is preferable to a style fix because it solves the bug while eliminating some invalid markup (no size or maxlength attributes on textareas, which are invalid without explicit dimensions). To test, add a nice big chunk of text to an existing item's item field like Public note (952$z), save, and edit again. The field should now be displayed in a more comfortably-sized textarea. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9571] Fix width for textareas in Edit Item screen
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9571 Kyle M Hall k...@bywatersolutions.com changed: What|Removed |Added Attachment #15152|0 |1 is obsolete|| --- Comment #4 from Kyle M Hall k...@bywatersolutions.com --- Created attachment 15174 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=15174action=edit [SIGNED-OFF] Bug 9571 - Fix width for textareas in Edit Item screen This patch adds a style for textareas on the Edit Item screen of addbiblio.pl so that they are roughly the same length as the other input elements around it on the page. To test: Before applying patch... 1) Create a new item for any biblio with a URL (subfield u) of over 100 characters (you can use www.lettercount.com to count the number of characters in your URL). 2) In any browser (I've already looked at IE, Firefox, and Chrome on Windows), edit the item (either by clicking Edit Items from the Normal view of the bib record then clicking Edit next to the applicable item, or clicking on the Items view in the bib record and clicking Edit item next to the relevant item). 3) Notice that the textarea for the url is absolutely tiny. Difficult to read and difficult to interact with. Apply patch. 1) Notice that the textarea is roughly the same size as the other input elements on the page. Signed-off-by: Kyle M Hall k...@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9571] Fix width for textareas in Edit Item screen
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9571 Kyle M Hall k...@bywatersolutions.com changed: What|Removed |Added Attachment #15173|0 |1 is obsolete|| --- Comment #5 from Kyle M Hall k...@bywatersolutions.com --- Created attachment 15175 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=15175action=edit [SIGNED-OFF] Bug 9571 [Alternate] Fix width for textareas in Edit Item screen This alternative patch modifies the script where the input markup is built and adds an alternative configuration for textareas. This gives the textareas explicit cols and rows attributes. I think this solution is preferable to a style fix because it solves the bug while eliminating some invalid markup (no size or maxlength attributes on textareas, which are invalid without explicit dimensions). To test, add a nice big chunk of text to an existing item's item field like Public note (952$z), save, and edit again. The field should now be displayed in a more comfortably-sized textarea. Signed-off-by: Kyle M Hall k...@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9571] Fix width for textareas in Edit Item screen
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9571 Kyle M Hall k...@bywatersolutions.com changed: What|Removed |Added Status|Needs Signoff |Signed Off CC||k...@bywatersolutions.com --- Comment #6 from Kyle M Hall k...@bywatersolutions.com --- Both patches solve the issue. I'll leave it to QA to decent which one to use. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9571] Fix width for textareas in Edit Item screen
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9571 David Cook dc...@prosentient.com.au changed: What|Removed |Added Status|NEW |Needs Signoff Assignee|oleon...@myacpl.org |dc...@prosentient.com.au --- Comment #1 from David Cook dc...@prosentient.com.au --- Created attachment 15152 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=15152action=edit Bug 9571 - Fix width for textareas in Edit Item screen -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9571] Fix width for textareas in Edit Item screen
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9571 --- Comment #2 from David Cook dc...@prosentient.com.au --- (In reply to comment #1) Created attachment 15152 [details] Bug 9571 - Fix width for textareas in Edit Item screen To test: Before applying patch... 1) Create a new item for any biblio with a URL (subfield u) of over 100 characters (you can use www.lettercount.com to count the number of characters in your URL). 2) In any browser (I've already looked at IE, Firefox, and Chrome on Windows), edit the item (either by clicking Edit Items from the Normal view of the bib record then clicking Edit next to the applicable item, or clicking on the Items view in the bib record and clicking Edit item next to the relevant item). 3) Notice that the textarea for the url is absolutely tiny. Difficult to read and difficult to interact with. Apply patch. 1) Notice that the textarea is roughly the same size as the other input elements on the page. --- Ideally, I would like the textarea to automatically be the same length as the other input elements, but I'm not sure how to do that just with CSS and while still only using relative values. Open to hear other ideas about how this can look even better. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/