kanboard-backlog icon indicating copy to clipboard operation
kanboard-backlog copied to clipboard

Backlog column width doesn't take care of collapsed columns

Open slavkoja opened this issue 6 years ago • 4 comments

I use this master and kanboard 1.2.8 and it seems, that the main Backlog column width doesn't take care about collapsed columns in board.

I have main board with 5 columns, from which two are collapsed, but backlog column has width set to 16,6...%, which seems to be result of 100% / (5 + 1). But this simple calculation is bad, if columns are collapsed, backlog column has real width 240px (from min-width), but other not collapsed columns has circa 360px, the two collapsed columns has 28px, by its CSS.

I tried to find way to get collapsed columns, but my knowledge is not enough for this and it seems, that it is handled via JavaScript.

slavkoja avatar Mar 29 '19 12:03 slavkoja

I don't know a lot about JS either, but, i think if you put an incremental variable in this if statement: https://github.com/vistree/kanboard-backlog/blob/d0d904d20032cd0d941053e6a9b6aacfb24063dd/Assets/backlog.js#L2-L5

you will have a variable that contains the number of collapsed columns to use in simple calculation of: https://github.com/vistree/kanboard-backlog/blob/d0d904d20032cd0d941053e6a9b6aacfb24063dd/Assets/backlog.js#L12

creecros avatar Apr 01 '19 19:04 creecros

I also found this, to get the count of elements using a class. document.querySelectorAll('#main-div .specific-class').length;

so maybe this would work as well:

document.querySelectorAll('#kanboard-column th .board-column-header-collapsed').length;

creecros avatar Apr 01 '19 19:04 creecros

I still think, why the separate table is created. IMO, when it will inject cell into first row with rowspan set to appropriate count of swimlanes (which will create whole table column) and with the appropriate classes. Then whole backlog column can be in this cell and its width then will be calculated as needed with other columns.

But i do not know, if it is possible to inject cell by this way...

slavkoja avatar Apr 02 '19 10:04 slavkoja

I will try to check this on the weekend!

vistree avatar Apr 03 '19 08:04 vistree