Borders on empty cells

I’m trying to add borders on all cells, including empty ones. Here is the code I tried to use:

var borderStyles = {
  top: { style: "thin" },
  left: { style: "thin" },
  bottom: { style: "thin" },
  right: { style: "thin" }
};

worksheet.eachRow({ includeEmpty: true }, function(row, rowNumber) {
  row.eachCell({ includeEmpty: true }, function(cell, colNumber) {
    cell.border = borderStyles;
  });
});

I thought that using { includeEmpty: true } would do this for me, but it doesn’t seem to work, as you can see here:

I want the gaps in between the sections to have borders as well. I made those gaps by doing addRow and passing it an empty object {}, could that be causing an issue?

Or is there any other way to get borders to appear on all cells?

[HankMcCoy]

I managed to get this working by tweaking the code locally. I hope to get a PR open by mid next week.

The basic problem is that there are several places in the codebase where it checks to see if a cell has a type of Enum.ValueType.Null and if it does, it doesn’t include it in the final render, presumably as an optimization. This is, of course, problematic when that cell still have styles applied to it, despite being empty.

The short summary of my local changes is that I:

• Removed the if (cell.type != Enum.ValueType.Null) check in the get model() accessor in row.js
• Removed the spans="<%=row.min%>:<%=row.max"> attribute in the row section of sheet.xml since I had a hard time getting the min and max there to actually work (spans is an optional optimization attribute anyway).
• Added a default: case to the switch (cell.type) statement in xlsx.js to do the same thing it was already doing for case Enums.ValueType.Merge.

[edongashi]

I am facing the same problem… I found a workaround by placing empty strings to force values on cells. It works this way, although it’s causing me trouble with cell merging in MS Excel

if (!cell.value) { cell.value = ‘’; }