From aeb485fdac02535041a12c3bae10046434383f13 Mon Sep 17 00:00:00 2001
From: Liyulun <18217551067@163.com>
Date: Thu, 28 Jun 2018 15:57:53 +0800
Subject: [PATCH] feat(module:table): fix selector error (#1742)

close #1736
---
 components/table/nz-td.component.ts |  2 +-
 components/table/nz-td.spec.ts      | 15 +++++++++++++++
 components/table/nz-th.component.ts |  2 +-
 components/table/nz-th.spec.ts      | 15 +++++++++++++++
 4 files changed, 32 insertions(+), 2 deletions(-)

diff --git a/components/table/nz-td.component.ts b/components/table/nz-td.component.ts
index ba2f7d8b7a3..8023230e53c 100644
--- a/components/table/nz-td.component.ts
+++ b/components/table/nz-td.component.ts
@@ -12,7 +12,7 @@ import { toBoolean } from '../core/util/convert';
 
 @Component({
   // tslint:disable-next-line:component-selector
-  selector   : 'td',
+  selector   : 'td:not(.nz-disable-td)',
   templateUrl: './nz-td.component.html'
 })
 export class NzTdComponent {
diff --git a/components/table/nz-td.spec.ts b/components/table/nz-td.spec.ts
index f46c4e6ce70..ea251d0a5f4 100644
--- a/components/table/nz-td.spec.ts
+++ b/components/table/nz-td.spec.ts
@@ -123,6 +123,13 @@ describe('nz-td', () => {
       expect(td.nativeElement.classList).toContain('ant-table-td-right-sticky');
       expect(td.nativeElement.style.right).toBe('20px');
     });
+    it('should be throw error when use specific class name', () => {
+      expect(() => {
+        TestBed.configureTestingModule({
+          declarations: [ NzTestDisableTdComponent ]
+        }).createComponent(NzTestDisableTdComponent);
+      }).toThrow();
+    });
   });
 });
 
@@ -155,3 +162,11 @@ export class NzTestTdComponent {
   left;
   right;
 }
+
+@Component({
+  selector: 'nz-disable-td',
+  template: `
+    <td class="nz-disable-td" [nzShowCheckbox]="true"></td>
+  `
+})
+export class NzTestDisableTdComponent {}
diff --git a/components/table/nz-th.component.ts b/components/table/nz-th.component.ts
index 3ded5366468..715b145d977 100644
--- a/components/table/nz-th.component.ts
+++ b/components/table/nz-th.component.ts
@@ -26,7 +26,7 @@ export interface NzThItemInterface {
 
 @Component({
   // tslint:disable-next-line:component-selector
-  selector           : 'th',
+  selector           : 'th:not(.nz-disable-th)',
   preserveWhitespaces: false,
   templateUrl        : './nz-th.component.html'
 })
diff --git a/components/table/nz-th.spec.ts b/components/table/nz-th.spec.ts
index c72d2dee7ff..96e20097a83 100644
--- a/components/table/nz-th.spec.ts
+++ b/components/table/nz-th.spec.ts
@@ -248,6 +248,13 @@ describe('nz-th', () => {
       fixture.detectChanges();
       expect(th.nativeElement.classList).toContain('ant-table-expand-icon-th');
     });
+    it('should be throw error when use specific class name', () => {
+      expect(() => {
+        TestBed.configureTestingModule({
+          declarations: [ NzTestDisableThComponent ]
+        }).createComponent(NzTestDisableThComponent);
+      }).toThrow();
+    });
   });
 });
 
@@ -306,3 +313,11 @@ export class NzThTestNzTableComponent {
   filterMultiple = true;
   expand = false;
 }
+
+@Component({
+  selector: 'nz-disable-th',
+  template: `
+    <th class="nz-disable-th" [nzShowCheckbox]="true"></th>
+  `
+})
+export class NzTestDisableThComponent {}