@syncfusion/ej2-richtexteditor - npm Package Compare versions

Comparing version 17.4.39 to 17.4.40

CHANGELOG.md

@@ -9,2 +9,10 @@ # Changelog
 
+- `#254606` - Web accessibility related issues have been resolved.
+
+## 17.4.39 (2019-12-17)
+
+### RichTextEditor
+
+#### Bug Fixes
+
 - `#256452` - The Rich Text Editor is no longer allow to resize the image when `readonly` is enabled.
@@ -11,0 +19,0 @@

dist/global/index.d.ts

 /*!
 *  filename: index.d.ts
-*  version : 17.4.39
+*  version : 17.4.40
 *  Copyright Syncfusion Inc. 2001 - 2019. All rights reserved.
@@ -5,0 +5,0 @@ *  Use of this code is subject to the terms of our license.

package.json

 {
   "_from": "@syncfusion/ej2-richtexteditor@*",
-  "_id": "@syncfusion/ej2-richtexteditor@17.1.38",
+  "_id": "@syncfusion/ej2-richtexteditor@17.4.39",
   "_inBundle": false,
-  "_integrity": "sha512-7jbWl31iUv+F9IxEQyHRXeUkG9ok4bGeZ6NCA+/InZSriGheuXEhWkqJtX549kp8nDiYkyeWQmkuls3veHXTHg==",
+  "_integrity": "sha512-UUUrNFSKoelyZ1hK0LocXJ5zz9rO/yoRq3Tv+BkP6L7LwDYQT2kLIdIDjh2ZUoYBDrxAZQOhaj5CnBPehnAAcQ==",
   "_location": "/@syncfusion/ej2-richtexteditor",
@@ -29,4 +29,4 @@ "_phantomChildren": {},
   ],
-  "_resolved": "http://nexus.syncfusion.com/repository/ej2-release/@syncfusion/ej2-richtexteditor/-/ej2-richtexteditor-17.1.38.tgz",
-  "_shasum": "60b837d3c09add96672cbadcac0b38651eac0ea8",
+  "_resolved": "http://nexus.syncfusion.com/repository/ej2-hotfix/@syncfusion/ej2-richtexteditor/-/ej2-richtexteditor-17.4.39.tgz",
+  "_shasum": "d7bd1e2d8eaf11dbcc40341d264cc0b7e2b6a8d7",
   "_spec": "@syncfusion/ej2-richtexteditor@*",
@@ -73,4 +73,4 @@ "_where": "/jenkins/workspace/ation_hotfix_17.2.0.34_Vol2-DDPJ6C2YEWTWLLKKNA7AG2JBE6DYTCTNGMHGORWGAYAVCL6L7OLQ/packages/included",
   "typings": "index.d.ts",
-  "version": "17.4.39",
+  "version": "17.4.40",
   "sideEffects": false
 }

src/editor-manager/plugin/formats.js

@@ -32,51 +32,53 @@ import * as EVENTS from './../../common/constant';
     Formats.prototype.onKeyDown = function (e) {
-        var range = this.parent.nodeSelection.getRange(this.parent.currentDocument);
-        var startCon = (range.startContainer.textContent.length === 0 || range.startContainer.nodeName === 'PRE')
-            ? range.startContainer : range.startContainer.parentElement;
-        var endCon = (range.endContainer.textContent.length === 0 || range.endContainer.nodeName === 'PRE')
-            ? range.endContainer : range.endContainer.parentElement;
-        var preElem = closest(startCon, 'pre');
-        var endPreElem = closest(endCon, 'pre');
-        var liParent = !isNOU(preElem) && !isNOU(preElem.parentElement) && preElem.parentElement.tagName === 'LI';
-        if (liParent) {
-            return;
-        }
-        if (((isNOU(preElem) && !isNOU(endPreElem)) || (!isNOU(preElem) && isNOU(endPreElem)))) {
-            e.event.preventDefault();
-            this.deleteContent(range);
-            this.removeCodeContent(range);
-            range = this.parent.nodeSelection.getRange(this.parent.currentDocument);
-            this.parent.nodeSelection.setCursorPoint(this.parent.currentDocument, endCon, 0);
-        }
-        if (e.event.which === 13 && !isNOU(preElem) && !isNOU(endPreElem)) {
-            e.event.preventDefault();
-            this.deleteContent(range);
-            this.removeCodeContent(range);
-            range = this.parent.nodeSelection.getRange(this.parent.currentDocument);
-            var lastEmpty = range.startContainer.childNodes[range.endOffset];
-            var lastBeforeBr = range.startContainer.childNodes[range.endOffset - 1];
-            var startParent = range.startContainer;
-            if (!isNOU(lastEmpty) && !isNOU(lastBeforeBr) && isNOU(lastEmpty.nextSibling) &&
-                lastEmpty.nodeName === 'BR' && lastBeforeBr.nodeName === 'BR') {
-                this.paraFocus(range.startContainer);
+        if (e.event.which === 13) {
+            var range = this.parent.nodeSelection.getRange(this.parent.currentDocument);
+            var startCon = (range.startContainer.textContent.length === 0 || range.startContainer.nodeName === 'PRE')
+                ? range.startContainer : range.startContainer.parentElement;
+            var endCon = (range.endContainer.textContent.length === 0 || range.endContainer.nodeName === 'PRE')
+                ? range.endContainer : range.endContainer.parentElement;
+            var preElem = closest(startCon, 'pre');
+            var endPreElem = closest(endCon, 'pre');
+            var liParent = !isNOU(preElem) && !isNOU(preElem.parentElement) && preElem.parentElement.tagName === 'LI';
+            if (liParent) {
+                return;
             }
-            else if ((startParent.textContent.charCodeAt(0) === 8203 &&
-                startParent.textContent.length === 1) || startParent.textContent.length === 0) {
-                //Double enter with any parent tag for the node
-                while (startParent.parentElement.nodeName !== 'PRE' &&
-                    (startParent.textContent.length === 1 || startParent.textContent.length === 0)) {
-                    startParent = startParent.parentElement;
+            if (((isNOU(preElem) && !isNOU(endPreElem)) || (!isNOU(preElem) && isNOU(endPreElem)))) {
+                e.event.preventDefault();
+                this.deleteContent(range);
+                this.removeCodeContent(range);
+                range = this.parent.nodeSelection.getRange(this.parent.currentDocument);
+                this.parent.nodeSelection.setCursorPoint(this.parent.currentDocument, endCon, 0);
+            }
+            if (e.event.which === 13 && !isNOU(preElem) && !isNOU(endPreElem)) {
+                e.event.preventDefault();
+                this.deleteContent(range);
+                this.removeCodeContent(range);
+                range = this.parent.nodeSelection.getRange(this.parent.currentDocument);
+                var lastEmpty = range.startContainer.childNodes[range.endOffset];
+                var lastBeforeBr = range.startContainer.childNodes[range.endOffset - 1];
+                var startParent = range.startContainer;
+                if (!isNOU(lastEmpty) && !isNOU(lastBeforeBr) && isNOU(lastEmpty.nextSibling) &&
+                    lastEmpty.nodeName === 'BR' && lastBeforeBr.nodeName === 'BR') {
+                    this.paraFocus(range.startContainer);
                 }
-                if (!isNOU(startParent.previousSibling) && startParent.previousSibling.nodeName === 'BR' &&
-                    isNOU(startParent.nextSibling)) {
-                    this.paraFocus(startParent.parentElement);
+                else if ((startParent.textContent.charCodeAt(0) === 8203 &&
+                    startParent.textContent.length === 1) || startParent.textContent.length === 0) {
+                    //Double enter with any parent tag for the node
+                    while (startParent.parentElement.nodeName !== 'PRE' &&
+                        (startParent.textContent.length === 1 || startParent.textContent.length === 0)) {
+                        startParent = startParent.parentElement;
+                    }
+                    if (!isNOU(startParent.previousSibling) && startParent.previousSibling.nodeName === 'BR' &&
+                        isNOU(startParent.nextSibling)) {
+                        this.paraFocus(startParent.parentElement);
+                    }
+                    else {
+                        this.isNotEndCursor(preElem, range);
+                    }
                 }
                 else {
+                    //Cursor at start and middle
                     this.isNotEndCursor(preElem, range);
                 }
             }
-            else {
-                //Cursor at start and middle
-                this.isNotEndCursor(preElem, range);
-            }
         }
@@ -83,0 +85,0 @@ };

src/rich-text-editor/actions/toolbar.d.ts

@@ -126,2 +126,4 @@ import { Toolbar as tool } from '@syncfusion/ej2-navigations';
     private dropDownBeforeOpenHandler;
+    private tbFocusHandler;
+    private tbKeydownHandler;
     private toolbarMouseDownHandler;
@@ -128,0 +130,0 @@ protected wireEvents(): void;

src/rich-text-editor/actions/toolbar.js

@@ -471,2 +471,23 @@ import { addClass, Browser, EventHandler, detach, removeClass, select, selectAll, KeyboardEvents } from '@syncfusion/ej2-base';
     };
+    Toolbar.prototype.tbFocusHandler = function (e) {
+        var activeElm = document.activeElement;
+        var isToolbaractive = closest(activeElm, '.e-rte-toolbar');
+        if (activeElm === this.parent.getToolbarElement() || isToolbaractive === this.parent.getToolbarElement()) {
+            var toolbarItem = this.parent.getToolbarElement().querySelectorAll('.e-expended-nav');
+            for (var i = 0; i < toolbarItem.length; i++) {
+                if (isNOU(this.parent.getToolbarElement().querySelector('.e-insert-table-btn'))) {
+                    toolbarItem[i].setAttribute('tabindex', '0');
+                }
+                else {
+                    toolbarItem[i].setAttribute('tabindex', '1');
+                }
+            }
+        }
+    };
+    Toolbar.prototype.tbKeydownHandler = function (e) {
+        if (e.target.classList.contains('e-dropdown-btn') ||
+            e.target.getAttribute('id') === this.parent.getID() + '_toolbar_CreateTable') {
+            e.target.setAttribute('tabindex', '0');
+        }
+    };
     Toolbar.prototype.toolbarMouseDownHandler = function (e) {
@@ -496,5 +517,9 @@ var trg = closest(e.target, '.e-hor-nav');
         EventHandler.add(this.tbElement, 'click mousedown', this.toolbarMouseDownHandler, this);
+        EventHandler.add(this.tbElement, 'focusin', this.tbFocusHandler, this);
+        EventHandler.add(this.tbElement, 'keydown', this.tbKeydownHandler, this);
     };
     Toolbar.prototype.unWireEvents = function () {
         EventHandler.remove(this.tbElement, 'click mousedown', this.toolbarMouseDownHandler);
+        EventHandler.remove(this.tbElement, 'focusin', this.tbFocusHandler);
+        EventHandler.remove(this.tbElement, 'keydown', this.tbKeydownHandler);
     };
@@ -501,0 +526,0 @@ Toolbar.prototype.addEventListener = function () {

src/rich-text-editor/renderer/table-module.js

@@ -790,4 +790,7 @@ import { detach, closest, Browser, isNullOrUndefined as isNOU, isBlazor } from '@syncfusion/ej2-base';
             className: 'e-insert-table-btn', id: this.rteID + '_insertTable',
-            attrs: { type: 'button' }
+            attrs: { type: 'button', tabindex: '0' }
         });
+        if (!isNOU(this.parent.getToolbarElement().querySelector('.e-expended-nav'))) {
+            this.parent.getToolbarElement().querySelector('.e-expended-nav').setAttribute('tabindex', '1');
+        }
         this.dlgDiv.appendChild(btnEle);
@@ -794,0 +797,0 @@ var button = new Button({

src/rich-text-editor/renderer/toolbar-renderer.js

@@ -308,2 +308,3 @@ import { addClass, Browser, removeClass, EventHandler, formatUnit, isNullOrUndefined } from '@syncfusion/ej2-base';
         dropDown.element.onmousedown = function () { proxy.parent.notify(events.selectionSave, {}); };
+        dropDown.element.onkeydown = function () { proxy.parent.notify(events.selectionSave, {}); };
         return dropDown;
@@ -310,0 +311,0 @@ };

src/rich-text-editor/renderer/view-source.js

@@ -71,2 +71,8 @@ import { removeClass, selectAll, isNullOrUndefined, EventHandler } from '@syncfusion/ej2-base';
                 break;
+            case 'toolbar-focus':
+                if (this.parent.toolbarSettings.enable) {
+                    var selector = '.e-toolbar-item[aria-disabled="false"][title] [tabindex]';
+                    this.parent.toolbarModule.baseToolbar.toolbarObj.element.querySelector(selector).focus();
+                }
+                break;
         }
@@ -73,0 +79,0 @@ };

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

12416040

increased by0.22%

Lines of code

109820

increased by0.12%

No dependency changes