Skip to content
GitLab
Projects
Groups
Snippets
Help
Loading...
Help
Help
Support
Community forum
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Sign in / Register
Toggle navigation
Open sidebar
Boris Kocherov
sdkjs
Commits
544ff7de
Commit
544ff7de
authored
7 years ago
by
Alexander.Trofimov
Browse files
Options
Download
Email Patches
Plain Diff
fix position column data headers
parent
f5f0660f
Changes
2
Hide whitespace changes
Inline
Side-by-side
Showing
2 changed files
with
18 additions
and
15 deletions
+18
-15
cell/model/PivotTables.js
cell/model/PivotTables.js
+5
-2
cell/model/Workbook.js
cell/model/Workbook.js
+13
-13
No files found.
cell/model/PivotTables.js
View file @
544ff7de
...
...
@@ -2759,8 +2759,11 @@ CT_pivotTableDefinition.prototype.contains = function (col, row) {
CT_pivotTableDefinition.prototype.getRange = function () {
return this.location && this.location.ref;
};
CT_pivotTableDefinition.prototype.getFirstHeaderRow = function () {
return this.location && this.location.firstHeaderRow;
CT_pivotTableDefinition.prototype.getFirstHeaderRow0 = function () {
return this.location && (this.location.firstHeaderRow + this.getColumnFieldsCount() - 1);
};
CT_pivotTableDefinition.prototype.getFirstDataCol = function () {
return this.location && this.location.firstDataCol;
};
CT_pivotTableDefinition.prototype.getColumnFieldsCount = function (withoutValues) {
var res = 0;
...
...
This diff is collapsed.
Click to expand it.
cell/model/Workbook.js
View file @
544ff7de
...
...
@@ -5211,8 +5211,8 @@
}
};
Worksheet.prototype.updatePivotTablesStyle = function (range) {
var pivotTable, pivotRange, styleInfo, style, wholeStyle, cells, j, r, pos, firstHeaderRow0,
countC
,
countCWValues, countR, countD, stripe1, stripe2, items, item, start,
var pivotTable, pivotRange, styleInfo, style, wholeStyle, cells, j, r, pos, firstHeaderRow0,
firstDataCol0
,
countC,
countCWValues, countR, countD, stripe1, stripe2, items, item, start,
emptyStripe = new Asc.CTableStyleElement();
var dxf, dxfLabels, dxfValues, grandColumn;
for (var i = 0; i < this.pivotTables.length; ++i) {
...
...
@@ -5255,7 +5255,8 @@
continue;
}
firstHeaderRow0 = pivotTable.getFirstHeaderRow() + countC - 1;
firstHeaderRow0 = pivotTable.getFirstHeaderRow0();
firstDataCol0 = pivotTable.getFirstDataCol();
countD = pivotTable.getDataFieldsCount();
countCWValues = pivotTable.getColumnFieldsCount(true);
...
...
@@ -5266,7 +5267,7 @@
if (styleInfo.showColStripes) {
stripe1 = style.firstColumnStripe || emptyStripe;
stripe2 = style.secondColumnStripe || emptyStripe;
start = pivotRange.c1 +
countR
;
start = pivotRange.c1 +
firstDataCol0
;
if (stripe1.dxf) {
cells = this.getRange3(pivotRange.r1, start, pivotRange.r2, pivotRange.c2);
cells.setTableStyle(stripe1.dxf, new Asc.CTableStyleStripe(stripe1.size, stripe2.size));
...
...
@@ -5294,8 +5295,8 @@
// First Column
dxf = style.firstColumn && style.firstColumn.dxf;
if (styleInfo.showRowHeaders && countR && dxf) {
cells = this.getRange3(pivotRange.r1, pivotRange.c1, pivotRange.r2,
pivotRange.c1 + Math.max(0, countR
- 1));
cells = this.getRange3(pivotRange.r1, pivotRange.c1, pivotRange.r2,
pivotRange.c1 +
Math.max(0, firstDataCol0
- 1));
cells.setTableStyle(dxf);
}
...
...
@@ -5310,14 +5311,14 @@
dxf = style.firstHeaderCell && style.firstHeaderCell.dxf;
if (styleInfo.showColHeaders && styleInfo.showRowHeaders && countCWValues && (countR + countD) && dxf) {
cells = this.getRange3(pivotRange.r1, pivotRange.c1, pivotRange.r1 + firstHeaderRow0 - (countR ? 1 : 0),
pivotRange.c1 + Math.max(0,
countR
- 1));
pivotRange.c1 + Math.max(0,
firstDataCol0
- 1));
cells.setTableStyle(dxf);
}
// Subtotal Column
items = pivotTable.getColItems();
if (items) {
start = pivotRange.c1 +
countR
;
start = pivotRange.c1 +
firstDataCol0
;
for (j = 0; j < items.length; ++j) {
dxf = null;
item = items[j];
...
...
@@ -5380,7 +5381,7 @@
// Column Subheading
items = pivotTable.getColItems();
if (items) {
start = pivotRange.c1 +
countR
;
start = pivotRange.c1 +
firstDataCol0
;
for (j = 0; j < countCWValues; ++j) {
if (0 === j) {
dxf = style.firstColumnSubheading;
...
...
@@ -5456,7 +5457,7 @@
};
Worksheet.prototype.getPivotTableButtons = function (range) {
var res = [];
var pivotTable, pivotRange, j, pos, countC, countCWValues, countR,
firstHeaderRow0,
cell;
var pivotTable, pivotRange, j, pos, countC, countCWValues, countR, cell;
for (var i = 0; i < this.pivotTables.length; ++i) {
pivotTable = this.pivotTables[i];
if (!pivotTable.intersection(range)) {
...
...
@@ -5476,16 +5477,15 @@
countCWValues = pivotTable.getColumnFieldsCount(true);
countR = pivotTable.getRowFieldsCount(true);
pivotRange = pivotTable.getRange();
firstHeaderRow0 = pivotTable.getFirstHeaderRow() + countC - 1;
if (countR) {
cell = new AscCommon.CellBase(pivotRange.r1 +
f
irstHeaderRow0, pivotRange.c1);
cell = new AscCommon.CellBase(pivotRange.r1 +
pivotTable.getF
irstHeaderRow0
()
, pivotRange.c1);
if (range.contains2(cell)) {
res.push(cell);
}
}
if (countCWValues) {
cell = new AscCommon.CellBase(pivotRange.r1, pivotRange.c1 +
countR
);
cell = new AscCommon.CellBase(pivotRange.r1, pivotRange.c1 +
pivotTable.getFirstDataCol()
);
if (range.contains2(cell)) {
res.push(cell);
}
...
...
This diff is collapsed.
Click to expand it.
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
.
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment