[open-ils-commits] [GIT] Evergreen ILS branch rel_3_0 updated. 404ccc6908f0fd0ba57f681402b03c0360bfb140
Evergreen Git
git at git.evergreen-ils.org
Wed Mar 14 16:47:24 EDT 2018
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Evergreen ILS".
The branch, rel_3_0 has been updated
via 404ccc6908f0fd0ba57f681402b03c0360bfb140 (commit)
via f987c1ade801d0102c46ada465448d873413a634 (commit)
from ad4802eedb3c2d05f4d0136c1724883968c34b6f (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
- Log -----------------------------------------------------------------
commit 404ccc6908f0fd0ba57f681402b03c0360bfb140
Author: Galen Charlton <gmc at equinoxinitiative.org>
Date: Mon Feb 5 18:04:12 2018 -0500
LP#1691263: discourage input of newlines in MARC editor
This patch catches and discards enter keydown events when the
focus is in a contenteditable subfield value div; otherwise, newlines
can infect the MARC record.
Signed-off-by: Galen Charlton <gmc at equinoxinitiative.org>
Signed-off-by: Kathy Lussier <klussier at masslnc.org>
diff --git a/Open-ILS/web/js/ui/default/staff/cat/services/marcedit.js b/Open-ILS/web/js/ui/default/staff/cat/services/marcedit.js
index 3dd9691..f35cc27 100644
--- a/Open-ILS/web/js/ui/default/staff/cat/services/marcedit.js
+++ b/Open-ILS/web/js/ui/default/staff/cat/services/marcedit.js
@@ -946,6 +946,12 @@ angular.module('egMarcMod', ['egCoreMod', 'ui.bootstrap'])
addDatafield(event, event.shiftKey); // shift key inserts before
event_return = false;
+ } else if (event.which == 13 &&
+ ($(event.target).hasClass('marcsf') || $(event.target.parentNode).hasClass('marcsf'))
+ ) {
+ // bare return; don't allow it
+ event_return = false;
+
} else if (event.which == 46 && event.ctrlKey) { // ctrl+del, remove field
deleteDatafield(event);
event_return = false;
commit f987c1ade801d0102c46ada465448d873413a634
Author: Cesar Velez <cesar.velez at equinoxinitiative.org>
Date: Mon Dec 4 12:45:35 2017 -0500
LP#1691263: make webstaff MARC editor wrap long fields
This patch makes the MARC editor wrap long fields (e.g.,
bibliographic 505 fields) so that they fit the width of the enclosing
window or modal. The approach taken is replacing the text input
elements with contenteditable divs, which in turn can be better
styled.
To test
-------
[1] Apply the patch.
[2] Locate a record with a long 505 field and open it in the
MARC editor. Verify that the contents of the field wrap.
[3] Verify that record editing and saving work as expected.
Signed-off-by: Cesar Velez <cesar.velez at equinoxinitiative.org>
Signed-off-by: Galen Charlton <gmc at equinoxinitiative.org>
Signed-off-by: Kathy Lussier <klussier at masslnc.org>
diff --git a/Open-ILS/src/templates/staff/cat/share/t_marcedit_editable.tt2 b/Open-ILS/src/templates/staff/cat/share/t_marcedit_editable.tt2
new file mode 100644
index 0000000..1e5888f
--- /dev/null
+++ b/Open-ILS/src/templates/staff/cat/share/t_marcedit_editable.tt2
@@ -0,0 +1,20 @@
+<span style="all:unset;">
+<input
+ ng-show="itype != 'sfv'"
+ ng-disabled="{{isInputDisabled}}"
+ ng-class="['marcedit', {'marcsfcode': itype == 'sfc','marcind': itype == 'ind' || itype == 'tag', 'focusable': itype != 'sfv'}]"
+ style="font-family: 'Lucida Console', Monaco, monospace; min-width: 1ch; margin: 0 -2px;"
+ ng-model="content"
+ size="{{content.length * 1.1}}"
+ maxlength="{{max}}"
+ class=""
+ type="text">
+</input>
+<div contenteditable
+ class=""
+ ng-class="['marcedit', {'marcsfvalue': itype == 'sfv', 'focusable': itype == 'sfv'}]"
+ ng-show="itype == 'sfv'"
+ style="font-family: 'Lucida Console', Monaco, monospace; display: inline-block; min-width: 1ch; margin: 0 -1px; padding: 0;"
+ ng-model="content"
+>{{content}}</div>
+</span>
diff --git a/Open-ILS/web/js/ui/default/staff/cat/services/marcedit.js b/Open-ILS/web/js/ui/default/staff/cat/services/marcedit.js
index 89dabb1..3dd9691 100644
--- a/Open-ILS/web/js/ui/default/staff/cat/services/marcedit.js
+++ b/Open-ILS/web/js/ui/default/staff/cat/services/marcedit.js
@@ -44,18 +44,34 @@ angular.module('egMarcMod', ['egCoreMod', 'ui.bootstrap'])
}
}])
+.directive("contenteditable", function() {
+ return {
+ restrict: "A",
+ require: "ngModel",
+ link: function(scope,element,attrs,ngModel){
+
+ function read(){
+ // save new text into model
+ var elhtml = element.text();
+ ngModel.$setViewValue(elhtml);
+ }
+
+ ngModel.$render = function(){
+ element.text(ngModel.$viewValue || "");
+ };
+
+ element.bind("blur.c_e keyup.c_e change.c_e", function(){
+ scope.$apply(read);
+ });
+ }
+ };
+})
+
.directive("egMarcEditEditable", ['$timeout', '$compile', '$document', function ($timeout, $compile, $document) {
return {
restrict: 'E',
replace: true,
- template: '<input '+
- 'style="font-family: \'Lucida Console\', Monaco, monospace;" '+
- 'ng-model="content" '+
- 'size="{{content.length * 1.1}}" '+
- 'maxlength="{{max}}" '+
- 'class="" '+
- 'type="text" '+
- '/>',
+ templateUrl: './cat/share/t_marcedit_editable',
scope: {
field: '=',
onKeydown: '=',
@@ -66,11 +82,12 @@ angular.module('egMarcMod', ['egCoreMod', 'ui.bootstrap'])
max: '@',
itype: '@',
selectOnFocus: '=',
- advanceFocusAfterInput: '='
+ advanceFocusAfterInput: '=',
+ isDisabled: "="
},
controller : ['$scope',
function ( $scope ) {
-
+ $scope.isInputDisabled = $scope.isDisabled == 'disabled';
if ($scope.contextItemContainer && angular.isArray($scope.$parent[$scope.contextItemContainer]))
$scope.item_container = $scope.$parent[$scope.contextItemContainer];
else if ($scope.contextItemGenerator)
@@ -146,9 +163,24 @@ angular.module('egMarcMod', ['egCoreMod', 'ui.bootstrap'])
if (Boolean(scope.selectOnFocus)) {
element.addClass('noSelection');
- element.bind('focus', function () { element.select() });
+ element.bind('focus', function (e) {
+ var el = $(e.target).children('input').first();
+ if (el.select) { el.select(); }
+ });
}
+ element.children("div[contenteditable]").each(function() {
+ $(this).focus(function(e) {
+ var tNode = e.target.firstChild;
+ var range = document.createRange();
+ range.setStart(tNode, 0);
+ range.setEnd(tNode, tNode.length);
+ var sel = window.getSelection();
+ sel.removeAllRanges();
+ sel.addRange(range);
+ });
+ });
+
function findCaretTarget(id, itype) {
var tgt = null;
if (itype == 'tag') {
@@ -349,6 +381,7 @@ angular.module('egMarcMod', ['egCoreMod', 'ui.bootstrap'])
'/></span>'+
'<span><eg-marc-edit-editable '+
'itype="sfv" '+
+ 'select-on-focus="true" '+
'class="marcedit marcsf marcsfvalue" '+
'field="field" '+
'subfield="subfield" '+
@@ -589,7 +622,7 @@ angular.module('egMarcMod', ['egCoreMod', 'ui.bootstrap'])
'content="tag" '+
'on-keydown="onKeydown" '+
'id="leadertag" '+
- 'disabled="disabled"'+
+ 'is-disabled="disabled"'+
'/></span>'+
'<span><eg-marc-edit-editable '+
'class="marcedit marcdata" '+
@@ -850,12 +883,19 @@ angular.module('egMarcMod', ['egCoreMod', 'ui.bootstrap'])
index_sf = event.data.scope.subfield[2];
new_sf = index_sf + 1;
- var start = event.target.selectionStart;
- var end = event.target.selectionEnd - event.target.selectionStart ?
- event.target.selectionEnd :
- event.target.value.length;
+ var start = event.target.selectionStart || getCaretPosEditableDiv(element);
+ var end;
+ if (event.target.value){
+ end = event.target.selectionEnd - event.target.selectionStart ?
+ event.target.selectionEnd :
+ event.target.value.length;
+ } else {
+ end = element.text().length;
+ }
- move_data = event.target.value.substring(start,end);
+ move_data = element.value ?
+ element.value.substring(start,end) :
+ element.text().substring(start, end);
} else if (element.hasClass('marcsfcode')) {
index_sf = event.data.scope.subfield[2];
@@ -871,7 +911,11 @@ angular.module('egMarcMod', ['egCoreMod', 'ui.bootstrap'])
event.data.scope.field.subfields.forEach(function(sf) {
if (sf[2] >= new_sf) sf[2]++;
- if (sf[2] == index_sf) sf[1] = event.target.value.substring(0,start) + event.target.value.substring(end);
+ if (sf[2] == index_sf) {
+ sf[1] = event.target.value ?
+ event.target.value.substring(0,start) + event.target.value.substring(end) :
+ element.text().substring(0, start);
+ }
});
event.data.scope.field.subfields.splice(
new_sf,
@@ -906,7 +950,8 @@ angular.module('egMarcMod', ['egCoreMod', 'ui.bootstrap'])
deleteDatafield(event);
event_return = false;
- } else if (event.which == 46 && event.shiftKey && $(event.target).hasClass('marcsf')) { // shift+del, remove subfield
+ } else if (event.which == 46 && event.shiftKey && ($(event.target).hasClass('marcsf') || $(event.target.parentNode).hasClass('marcsf'))) {
+ // shift+del, remove subfield
var sf = event.data.scope.subfield[2] - 1;
if (sf == -1) sf = 0;
@@ -1039,7 +1084,7 @@ angular.module('egMarcMod', ['egCoreMod', 'ui.bootstrap'])
event_return = false;
} else { // Assumes only marc editor elements have IDs that can trigger this event handler.
- $scope.current_event_target = $(event.target).attr('id');
+ $scope.current_event_target = $(event.target).hasClass('focusable') ? $(event.target) : null;//.attr('id');
if ($scope.current_event_target) {
$scope.current_event_target_cursor_pos =
event.target.selectionDirection=='backward' ?
@@ -1057,7 +1102,7 @@ angular.module('egMarcMod', ['egCoreMod', 'ui.bootstrap'])
if (!$scope.current_event_target_cursor_pos_end)
$scope.current_event_target_cursor_pos_end = $scope.current_event_target_cursor_pos
- var element = $('#'+$scope.current_event_target).get(0);
+ var element = $('#'+$scope.current_event_target + " .focusable").get(0);
if (element) {
element.focus();
if (element.setSelectionRange) {
@@ -1066,10 +1111,24 @@ angular.module('egMarcMod', ['egCoreMod', 'ui.bootstrap'])
$scope.current_event_target_cursor_pos_end
);
}
- $scope.current_event_cursor_pos_end = null;
- $scope.current_event_target = null;
+ }
+ $scope.current_event_cursor_pos_end = null;
+ $scope.current_event_target = null;
+ }
+ }
+
+ function getCaretPosEditableDiv(editableDiv){
+ var caretPos = 0, sel, range;
+ if (window.getSelection) {
+ sel = window.getSelection();
+ if (sel.rangeCount) {
+ range = sel.getRangeAt(0);
+ if (range.commonAncestorContainer.parentNode == editableDiv[0]) {
+ caretPos = range.endOffset;
+ }
}
}
+ return caretPos;
}
function loadRecord() {
-----------------------------------------------------------------------
Summary of changes:
.../staff/cat/share/t_marcedit_editable.tt2 | 20 ++++
.../js/ui/default/staff/cat/services/marcedit.js | 111 ++++++++++++++++----
2 files changed, 108 insertions(+), 23 deletions(-)
create mode 100644 Open-ILS/src/templates/staff/cat/share/t_marcedit_editable.tt2
hooks/post-receive
--
Evergreen ILS
More information about the open-ils-commits
mailing list