Pinned rows navigation - draft #591
Conversation
packages/qgrid-ngx/src/lib/cell-handler/cell-handler.component.ts
Outdated
Show resolved
Hide resolved
| } | ||
| return f(...args); | ||
| }, | ||
| type: entry.name |
There was a problem hiding this comment.
Adding this and factory pattern?
| addNext('mid','top'); | ||
| addNext('right','top'); | ||
| addNext('left','mid'); | ||
| addNext('mid','mid'); |
There was a problem hiding this comment.
['left', 'mid', 'right'].forEach(pin => (['top', 'mid', 'bottom'].forEach(pos => addNext(pin, pos))));
| const { table } = this.root; | ||
| const { table, model } = this.root; | ||
|
|
||
| this.model = model; |
There was a problem hiding this comment.
why u need to store this.model? u can get it from this.root always
| index += this.model.row().pinTop.length; | ||
| } | ||
| if (this.pin == 'bottom') { | ||
| index += this.model.row().pinTop.length + this.model.scene().rows.length; |
There was a problem hiding this comment.
I think model.scene should also contain pinned rows
There was a problem hiding this comment.
Also this is an interesting questions if we have page size set to 50 and have 2 pinned rows, overall we should show 50 or 52 rows?
| [attr.rowspan]="$view.body.render.rowspan($row, $column, $rowIndex, $columnIndex)" | ||
| [attr.colspan]="$view.body.render.colspan($row, $column, $rowIndex, $columnIndex)"> | ||
| <ng-container [q-grid-core-td]="$column" | ||
| <ng-container [q-grid-core-td]="$column" [q-grid-core-pin]="pin" |
There was a problem hiding this comment.
you can understand pin from TableService no need to use binding
| <mat-menu #menu="matMenu" | ||
| class="q-grid-mat-menu"> | ||
| <mat-card *ngIf="trigger.menuOpen"> | ||
| <mat-card *ngIf="trigger.menuOpen" [q-grid-stop-propagate]="['keydown', 'keypress', 'keyup']"> |
There was a problem hiding this comment.
fix formatting please
*ngIf="..."
[q-grid-stop-propagate]="..."| const pathFinder = new PathService(table.box.bag.body); | ||
| const path = eventPath(e); | ||
| return pathFinder.row(path); | ||
| return pathFinder.row(path); |
|
|
||
| if (highlight.row.canExecute(index)) { | ||
| let correctedIndex = index; | ||
| if (highlight.row.canExecute(correctedIndex)) { |
There was a problem hiding this comment.
all logic should be encapsulated under the dom table class, users should not be aware of pinned or unpinned options
| return this.getElementsCore('body') | ||
| .some(element => this.isFocusedCore(element)); | ||
| return this.getElementsCore('body').some(element => this.isFocusedCore(element)) || | ||
| this.getElementsCore('body-top').some(element => this.isFocusedCore(element)) || |
There was a problem hiding this comment.
fix formatting, add offset
3 participants
this is a draft for new functionalty - navigation for top-bottom pinned rows