QuiltViewLibrary icon indicating copy to clipboard operation
QuiltViewLibrary copied to clipboard

setRowCount(int) and setColumnCount(int) shouldn't receive -1 as undefined value

Open Rafhack opened this issue 9 years ago • 2 comments

setupVertical() and setupHorizontal() methods on QuiltViewBase class, call setColumnCount(int) and setRowCount(int) passing -1 as undefined value, this makes GridLayout throw an IllegalArgumentException. Instead, these methods should pass UNDEFINED constant as undefined value.

Rafhack avatar Dec 15 '15 03:12 Rafhack

solve it by remove //this.setColumnCount(-1); and //this.setColumnCount(-1); in vertical too

bag7dad avatar Oct 10 '16 22:10 bag7dad

@medozeus you mean setRowCount in horizontal i think it's better to set it undefined: this.setRowCount(UNDEFINED); // in horizontal this.setColumnCount(UNDEFINED); // in vertical

arefbhrn avatar May 03 '17 06:05 arefbhrn