Skip to content

Commit 117f902

Browse files
raffisgajus
authored andcommitted
fix: do not draw new line on the top (borderless) (#102)
1 parent 35f360d commit 117f902

File tree

2 files changed

+20
-1
lines changed

2 files changed

+20
-1
lines changed

src/drawBorder.js

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,12 +35,18 @@ const drawBorder = (columnSizeIndex, parts) => {
3535
* @returns {string}
3636
*/
3737
const drawBorderTop = (columnSizeIndex, parts) => {
38-
return drawBorder(columnSizeIndex, {
38+
const border = drawBorder(columnSizeIndex, {
3939
body: parts.topBody,
4040
join: parts.topJoin,
4141
left: parts.topLeft,
4242
right: parts.topRight
4343
});
44+
45+
if (border === '\n') {
46+
return '';
47+
}
48+
49+
return border;
4450
};
4551

4652
/**

test/drawBorder.js

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,19 @@ describe('drawBorderTop', () => {
3838
expect(drawBorderTop([1, 1], parts)).to.equal('╔═╤═╗\n');
3939
expect(drawBorderTop([5, 10], parts)).to.equal('╔═════╤══════════╗\n');
4040
});
41+
42+
it('no leading new line if borderless', () => {
43+
const parts = {
44+
topLeft: '',
45+
topRight: '',
46+
topBody: '',
47+
topJoin: ''
48+
};
49+
50+
expect(drawBorderTop([1], parts)).to.equal('');
51+
expect(drawBorderTop([1, 1], parts)).to.equal('');
52+
expect(drawBorderTop([5, 10], parts)).to.equal('');
53+
});
4154
});
4255

4356
describe('drawBorderJoin', () => {

0 commit comments

Comments
 (0)