Project

General

Profile

Actions

Bug #14506

closed

tablesorter: apply sort on page load

Added by Carsten Rose over 1 year ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Carsten Rose
Target version:
Start date:
05.11.2022
Due date:
12.12.2022
% Done:

0%

Estimated time:
Discuss:
Prio Planung:
No
Vote:

Description

  • Scheinbar merkt sich der Tablesorter die Spalte und Sortierrichtung auch wenn die Seite neu geladen wird.
  • Aber: es wird nicht sortiert!

  • Falsch:


Files


Related issues

Related to QFQ - Bug #14997: Tablesorter: Filter value shown but no effect after 'Browser Back'ClosedEnis Nuredini05.11.2022

Actions
Actions #1

Updated by Carsten Rose over 1 year ago

  • Due date set to 27.07.2022
  • Assignee changed from Carsten Rose to Enis Nuredini
  • Priority changed from Normal to High
  • Target version changed from next3 to 355
Actions #2

Updated by Carsten Rose over 1 year ago

  • Tracker changed from Support to Bug
  • Prio Planung set to No
Actions #3

Updated by Enis Nuredini over 1 year ago

  • Status changed from New to Priorize
Actions #4

Updated by Enis Nuredini over 1 year ago

  • Status changed from Priorize to In Progress
Actions #5

Updated by Carsten Rose over 1 year ago

  • Target version changed from 355 to Check if 'high' is still necessary
Actions #6

Updated by Enis Nuredini over 1 year ago

  • Status changed from In Progress to Feedback
  • Assignee changed from Enis Nuredini to Carsten Rose

Die bisherige Implementierung erlaubte es nicht die View 'Clear', welche als Default zur Verfügung steht, verändert zu speichern. Da es sich um ein standardmässig gesetztes Public View handelt macht es durchaus Sinn dass die Veränderung nicht bestehen bleibt nach dem Page Reload. Dafür war das Speichern der View über den Edit Button oben rechts von der Tabelle gedacht. Die dort gespeicherte Ansicht bleibt weiterhin bestehen. Ein Manko dabei ist es dass nach jeder Änderung (z.b. sortieren) die View erneut gespeichert werden muss um die Änderung bei Page Reload zu behalten. Die Pfeile des Tablesorters haben keine direkte Verbindung zur Funktionalität des View Savers, diese werden unabhängig davon von der tablesorter library selbst getriggert.

Meine Lösung:
Via JS lasse ich nebst dem bisherigen Public View 'Clear' auch einen Private View 'Default' erzeugen falls dieser nicht existiert. Dabei wird der Default als Standard bei Page Load ausgewählt falls nichts anderes zuletzt gewählt wurde. Jeder Klick auf den Sorter führt den Save für die View im Hintergrund aus. So bleibt die View in der DB-Tabelle Setting immer auf dem aktuellsten Stand, ohne das sie manuell jedes mal gespeichert werden muss.

Weiteres Problem mit neuer Typo3 Version behoben:
Die relative Pfadangabe des API URLs vom Tablesorter funktioniert mit den neueren Typo3 Versionen nicht mehr (Gleiches Problem wie bei den anderen API Aufrufen, die behoben wurden). Daher gebe ich die base url als eigenes Attribut zur Table mit, wenn tablesorter-view-saver verwendet wird.

Actions #7

Updated by Enis Nuredini over 1 year ago

  • Status changed from Feedback to In Progress
  • Assignee changed from Carsten Rose to Enis Nuredini
Actions #8

Updated by Enis Nuredini over 1 year ago

  • Status changed from In Progress to Feedback
  • Assignee changed from Enis Nuredini to Carsten Rose

Erneute Anpassung des JS. Mit der neuen Implementierung bleibt der tablesorter und tablefilter bestehen bis eine neue View über das Dropdown ausgewählt wurde, dies erfolgt über die interne Funktion des tablesorters. Eine gespeicherte View bestitzt auch eine eigene Sortierungskonfiguration. Nach der View-Wahl kann wieder wie gewohnt der Sorter und Filter verwendet werden, welche nach Page Load weiterhin bestehen bleiben bis wieder eine andere View ausgewählt wird.

Actions #9

Updated by Carsten Rose over 1 year ago

Bei Aufruf einer Seite mit einem tablesorter geht ein Dialog auf:

Das ist gar nicht gut: Ein User wird nach etwas gefragt ohne das er versteht warum! Es ist auch nicht das Ziel das QFQ den Entwickler zwingt fuer jede Tabelle einen Default View anzulegen.

Das Ticket ging nicht um die Funktion von gespeicherten Views (sondern wie in #14997 beschrieben). Mit ist nicht klar ob wir diese Code Aenderung hier wirklich wollen/benoetigen.

Actions #10

Updated by Carsten Rose over 1 year ago

  • Related to Bug #14997: Tablesorter: Filter value shown but no effect after 'Browser Back' added
Actions #11

Updated by Carsten Rose over 1 year ago

  • Due date set to 12.12.2022
  • Assignee changed from Carsten Rose to Enis Nuredini
Actions #12

Updated by Enis Nuredini over 1 year ago

  • Status changed from Feedback to Ready to sync (develop)
  • Assignee changed from Enis Nuredini to Carsten Rose

Verhalten war nicht beabsichtigt. Sollte eigentlich nur die View ohne User input automatisch erstellen. Hab jetzt den 'Default' View entfernt da wahrscheinlich nicht wirklich benötigt wird.

Actions #13

Updated by Carsten Rose about 1 year ago

  • Status changed from Ready to sync (develop) to Closed
  • Target version changed from Check if 'high' is still necessary to 22.12.1
Actions

Also available in: Atom PDF