[gnome-shell/wip/paging] Fixed bad calculations to rowsup and rowsdown at expanding collection view
- From: Carlos Soriano <csoriano src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-shell/wip/paging] Fixed bad calculations to rowsup and rowsdown at expanding collection view
- Date: Sat, 13 Jul 2013 11:44:30 +0000 (UTC)
commit 1b1700b640d317b3dd04d533c4c17d1989c365a8
Author: Carlos Soriano <csoriano src gnome org>
Date: Sat Jul 13 13:38:17 2013 +0200
Fixed bad calculations to rowsup and rowsdown at expanding collection
view
js/ui/appDisplay.js | 10 +++++++---
js/ui/iconGrid.js | 2 +-
2 files changed, 8 insertions(+), 4 deletions(-)
---
diff --git a/js/ui/appDisplay.js b/js/ui/appDisplay.js
index cd83257..dc669e0 100644
--- a/js/ui/appDisplay.js
+++ b/js/ui/appDisplay.js
@@ -218,6 +218,7 @@ const AppPages = new Lang.Class({
let rows = this._grid.pageRows(currentPage);
this._translatedRows = rows;
for(let rowIndex in rows) {
+ global.log("row " + rows[rowIndex]);
isMainIconRow = mainIconYPosition == rows[rowIndex][0].y;
if(isMainIconRow)
mainIconRowReached = true;
@@ -233,6 +234,8 @@ const AppPages = new Lang.Class({
rowsDown.push(rows[rowIndex]);
}
}
+ //The last page can have space without rows
+ let emptyRows = folderNVisibleRowsAtOnce + 1 - rows.length ;
let panViewUpNRows = 0;
let panViewDownNRows = 0;
if(side == St.Side.BOTTOM) {
@@ -245,11 +248,12 @@ const AppPages = new Lang.Class({
}
} else {
// There's not need to pan view up
- if(rowsDown.length >= folderNVisibleRowsAtOnce)
+ if(rowsDown.length + emptyRows >= folderNVisibleRowsAtOnce){
panViewDownNRows = folderNVisibleRowsAtOnce;
+ }
else {
- panViewDownNRows = rowsDown.length;
- panViewUpNRows = folderNVisibleRowsAtOnce - rowsDown.length;
+ panViewDownNRows = rowsDown.length + emptyRows;
+ panViewUpNRows = folderNVisibleRowsAtOnce - rowsDown.length - emptyRows;
}
}
this._panViewForFolderView(rowsUp, rowsDown, panViewUpNRows, panViewDownNRows, iconActor);
diff --git a/js/ui/iconGrid.js b/js/ui/iconGrid.js
index 2da474f..d00e812 100644
--- a/js/ui/iconGrid.js
+++ b/js/ui/iconGrid.js
@@ -589,7 +589,7 @@ const IconGrid = new Lang.Class({
// Positioning to the first element of the page
while(children[index] != this._firstPagesItems[pageNumber])
index++;
- for( let rowIndex = 0; rowIndex < this._rowsPerPage; rowIndex++) {
+ for( let rowIndex = 0; rowIndex < this._rowsPerPage && index < children.length; rowIndex++) {
rows[rowIndex] = [];
while(index < children.length && children[index].y == currentItem.y ) {
rows[rowIndex].push(children[index]);
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]