From 5a6d20f81797063cdfbc54498b38edcbe39bb0ac Mon Sep 17 00:00:00 2001
From: Johann Hoffmann <ma0646@technikum-wien.at>
Date: Thu, 5 Mar 2026 15:47:19 +0100
Subject: [PATCH] fixed scrollOffset discrepancy issue by UNDEFINING rowHeight
 and thus let tabulator detect the actual size of a row itself and not trust a
 sligthly wrong rowHeight value; define centered formatter in
 js/tabulator/formatter/centered and use it on unformatted columns for
 consistency;

---
 .../Cis/Benotungstool/Benotungstool.js         | 18 +++++++++---------
 public/js/tabulator/formatter/centered.js      |  5 +++++
 2 files changed, 14 insertions(+), 9 deletions(-)
 create mode 100644 public/js/tabulator/formatter/centered.js

diff --git a/public/js/components/Cis/Benotungstool/Benotungstool.js b/public/js/components/Cis/Benotungstool/Benotungstool.js
index 849f58678..3e3374164 100644
--- a/public/js/components/Cis/Benotungstool/Benotungstool.js
+++ b/public/js/components/Cis/Benotungstool/Benotungstool.js
@@ -9,6 +9,7 @@ import LehreinheitenModule from '../../DropdownModes/LehreinheitenModule';
 import MobilityLegende from '../../Mobility/Legende.js';
 import FhcOverlay from "../../Overlay/FhcOverlay.js";
 import {debounce} from "../../../helpers/debounce.js";
+import {centeredFormatter} from "../../../tabulator/formatter/centered.js";
 
 export const Benotungstool = {
 	name: "Benotungstool",
@@ -720,7 +721,8 @@ export const Benotungstool = {
 
 			return {
 				height: 700,
-				virtualDom: false,
+				virtualDom: true,
+				virtualDomBuffer: 5000,
 				index: 'uid',
 				layout: 'fitDataStretch',
 				placeholder: this.$capitalize(this.$p.t('global/noDataAvailable')),
@@ -728,7 +730,6 @@ export const Benotungstool = {
 				selectableRangeMode: "click", // shift+click
 				selectablePersistence: false, // reset selection on table reload
 				selectableCheck: this.selectableCheck,
-				rowHeight: 40,
 				rowFormatter: this.fixTabulatorSelectionFormatter,
 				columns: this.getColumnsDefinition(),
 				persistence: false,
@@ -796,13 +797,13 @@ export const Benotungstool = {
 				field: 'selectCol',
 				title: ''
 			})
-			columns.push({title: 'UID', field: 'uid', tooltip: false, widthGrow: 1, topCalc: this.sumCalcFunc, cssClass: 'sticky-col'})
+			columns.push({title: 'UID', field: 'uid', tooltip: false, widthGrow: 1, topCalc: this.sumCalcFunc, formatter: centeredFormatter, cssClass: 'sticky-col'})
 			columns.push({title: Vue.computed(() => this.$capitalize(this.$p.t('benotungstool/c4mail'))), field: 'email', formatter: this.mailFormatter, tooltip: false,  visible: false, widthGrow: 1, variableHeight: true})
-			columns.push({title: Vue.computed(() => this.$capitalize(this.$p.t('benotungstool/c4antrittCountv2'))), field: 'hoechsterAntritt', tooltip: false, widthGrow: 1})
-			columns.push({title: Vue.computed(() => this.$capitalize(this.$p.t('benotungstool/c4vorname'))), field: 'vorname', headerFilter: true, tooltip: false, widthGrow: 1})
-			columns.push({title: Vue.computed(() => this.$capitalize(this.$p.t('benotungstool/c4nachname'))), field: 'nachname', headerFilter: true, widthGrow: 1})
+			columns.push({title: Vue.computed(() => this.$capitalize(this.$p.t('benotungstool/c4antrittCountv2'))), field: 'hoechsterAntritt', formatter: centeredFormatter, tooltip: false, widthGrow: 1})
+			columns.push({title: Vue.computed(() => this.$capitalize(this.$p.t('benotungstool/c4vorname'))), field: 'vorname', formatter: centeredFormatter, headerFilter: true, tooltip: false, widthGrow: 1})
+			columns.push({title: Vue.computed(() => this.$capitalize(this.$p.t('benotungstool/c4nachname'))), field: 'nachname', formatter: centeredFormatter, headerFilter: true, widthGrow: 1})
 			columns.push({title: Vue.computed(() => this.$capitalize(this.$p.t('benotungstool/c4anwesenheitsquote'))), field: 'anwquote', widthGrow: 1, formatter: this.percentFormatter})
-			columns.push({title: Vue.computed(() => this.$capitalize(this.$p.t('benotungstool/c4mobility'))), field: 'mobility_zusatz', headerFilter: true, widthGrow: 1, visible: false})
+			columns.push({title: Vue.computed(() => this.$capitalize(this.$p.t('benotungstool/c4mobility'))), field: 'mobility_zusatz', formatter: centeredFormatter, headerFilter: true, widthGrow: 1, visible: false})
 			if(this.config?.CIS_GESAMTNOTE_PRUEFUNG_MOODLE_LE_NOTE) {
 				columns.push({title: Vue.computed(() => this.$capitalize(this.$p.t('benotungstool/c4teilnoten'))), field: 'teilnote', widthGrow: 1, formatter: this.teilnotenFormatter, variableHeight: true})
 			}
@@ -1668,10 +1669,9 @@ export const Benotungstool = {
 					s.freigegeben = this.checkFreigabe(s.freigabedatum, s.benotungsdatum, s.uid);
 					
 					s.lv_note = res.data[1]?.note
-					
 					const oldScrollLeft = this.$refs.notenTable.tabulator.rowManager.scrollLeft
 					const oldScrollTop = this.$refs.notenTable.tabulator.rowManager.scrollTop
-
+					
 					// add new pruefung to row
 					if(!this.pruefung) {			
 						this.handleAddNewTermin(res.data, s)
diff --git a/public/js/tabulator/formatter/centered.js b/public/js/tabulator/formatter/centered.js
new file mode 100644
index 000000000..3c38f4d4d
--- /dev/null
+++ b/public/js/tabulator/formatter/centered.js
@@ -0,0 +1,5 @@
+export function centeredFormatter (cell) 
+{
+	const val = cell.getValue()
+	return '<div style="display: flex; justify-content: center; align-items: center; height: 100%">'+val+'</div>'
+}
\ No newline at end of file