smartyellow/docx-js
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Breaking Change. Swap arguments around and make default WidthType

Browse Source
This commit is contained in:
Dolan Miu
2018-11-15 03:00:26 +00:00
parent 28556f277b
commit e19890e27a
7 changed files with 50 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
src/file/table/table-properties/table-properties.spec.ts
View File

@ -15,13 +15,21 @@ describe("TableProperties", () => {
});
describe("#setWidth", () => {
it("adds a table width property", () => {
const tp = new TableProperties().setWidth(WidthType.DXA, 1234);
it("should add a table width property", () => {
const tp = new TableProperties().setWidth(1234, WidthType.DXA);
const tree = new Formatter().format(tp);
expect(tree).to.deep.equal({
"w:tblPr": [{ "w:tblW": [{ _attr: { "w:type": "dxa", "w:w": 1234 } }] }],
});
});
it("should add a table width property with default of AUTO", () => {
const tp = new TableProperties().setWidth(1234);
const tree = new Formatter().format(tp);
expect(tree).to.deep.equal({
"w:tblPr": [{ "w:tblW": [{ _attr: { "w:type": "auto", "w:w": 1234 } }] }],
});
});
});
describe("#setFixedWidthLayout", () => {

4
src/file/table/table-properties/table-properties.ts
View File

@ -17,8 +17,8 @@ export class TableProperties extends XmlComponent {
this.root.push(this.cellMargin);
}
public setWidth(type: WidthType, w: number | string): TableProperties {
this.root.push(new PreferredTableWidth(type, w));
public setWidth(width: number | string, type: WidthType = WidthType.AUTO): TableProperties {
this.root.push(new PreferredTableWidth(type, width));
return this;
}