fix: ignore sorter click if event is canceled

packages/grid/src/vaadin-grid-sorter.js

@@ -224,6 +224,11 @@ class GridSorter extends ThemableMixin(DirMixin(PolymerElement)) {
 
   /** @private */
   _onClick(e) {
+    if (e.defaultPrevented) {
+      // Something else has already handled the click event, do nothing.
+      return;
+    }
+
     const activeElement = this.getRootNode().activeElement;
     if (this !== activeElement && this.contains(activeElement)) {
       // Some focusable content inside the sorter was clicked, do nothing.

packages/grid/test/sorting.test.js

@@ -16,7 +16,7 @@ import {
 
 describe('sorting', () => {
   describe('sorter', () => {
-    let sorter, button, orderIndicator;
+    let sorter, title, button, orderIndicator;
 
     beforeEach(() => {
       sorter = fixtureSync(`
@@ -25,6 +25,7 @@ describe('sorting', () => {
         </vaadin-grid-sorter>
       `);
       button = sorter.querySelector('button');
+      title = sorter.querySelector('.title');
       orderIndicator = sorter.shadowRoot.querySelector('[part="order"]');
     });
 
@@ -47,6 +48,12 @@ describe('sorting', () => {
       expect(sorter.direction).to.equal(null);
     });
 
+    it('should not toggle if click event is already consumed', () => {
+      title.addEventListener('click', (e) => e.preventDefault());
+      click(title);
+      expect(sorter.direction).to.equal(null);
+    });
+
     it('should fire a sorter-changed event', () => {
       const spy = sinon.spy();
       sorter.addEventListener('sorter-changed', spy);