[open-ils-commits] [GIT] Evergreen ILS branch rel_2_12 updated. 363020f182873c7f7b57b7b38635f3d4577f9982
Evergreen Git
git at git.evergreen-ils.org
Wed Mar 14 16:48:23 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_2_12 has been updated
via 363020f182873c7f7b57b7b38635f3d4577f9982 (commit)
via e0e1d9a79f7fde04f73ee6a8ca798c08b72f5192 (commit)
from cc7e229d1570df89983dd55885fbbb5b705a6fef (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 363020f182873c7f7b57b7b38635f3d4577f9982
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 78a5f24..2e6d88b 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
@@ -945,6 +945,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 e0e1d9a79f7fde04f73ee6a8ca798c08b72f5192
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 fb3a05a..78a5f24 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" '+
@@ -849,12 +882,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];
@@ -870,7 +910,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,
@@ -905,7 +949,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;
@@ -1038,7 +1083,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' ?
@@ -1056,7 +1101,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) {
@@ -1065,10 +1110,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