[Opal] Automatisches 'alle anzeigen' bei Foreneinträgen und Kurslisten

[kingature]

Das Feature
In Opal werden Foreneinträge oder Kurslisten standardmäßig auf 10 Elemente pro Seite beschränkt. Es gibt jedoch auch die Option, sich alle Elemente als lange Liste auszugeben, indem man auf alle anzeigen klickt. Ich würde mir wünschen, dass das TUfast automatisch erledigen könnte.

[OliEfr]

Hey @kingature ,

das ist tatsächlich ohne Weiteres möglich, bzw wird auch an manchen Stellen schon gemacht. (ZB beim Laden der Kurslisten in das Dashboard.

Der Grund warum ich mich bisher dagegen entschieden habe es überall zu machen war der, dass es etwas länger lädt, wenn man in viele Kurse eingeschrieben ist. Das wollte ich nicht jedem zumuten.

Idee: man könnte es als Option einbauen

@C0ntroller Meinung?
@Noxdor die Option müsste durch dich in das Front-End eingebaut werden

Relevant Stelle im Code (aus /develop):

TUfast_TUD/src/contentScripts/other/opal/parseCourses.ts

Line 78 in ad00d64

(document.getElementsByClassName('pager-showall')[0] as HTMLAnchorElement | undefined)?.click()

[C0ntroller]

Also Foreneinträge verstehe ich, damit machen wir nichts, aber eigentlich sollte derzeit auch schon auf dem master und dementsprechend im stable bei jedem Besuch der Kurslisten das ganze angeklickt werden. Siehe hier:

TUfast_TUD/src/contentScripts/parseOpal.js

Lines 17 to 25 in 8103c98

	if (!document.getElementsByClassName('pager-showall')[0]) {
	chrome.runtime.sendMessage({ cmd: 'save_courses', course_list: parseCoursesFromWebPage() })
	parsedCourses = true
	// if not --> load all courses
	} else {
	document.getElementsByClassName('pager-showall')[0].click()
	chrome.runtime.sendMessage({ cmd: 'save_clicks', click_count: 1 })
	parsedCourses = false
	}

Falls das nicht der Fall ist, meinst du vielleicht etwas anderes? Meinst du Einschreibelisten in Kursen drinnen? @kingature

[kingature]

Ich meine generell Seiten in Opal, die aus mehreren Subpages bestehen. Ich weiß nicht genau, wie ich das beschreiben soll. Hier ein Screenshot. Das hast du sicherlich schon einmal gesehen.

Und genau bei so etwas möchte ich nicht immer auf alle anzeigen klicken. Ist nervig, wenn man sich verschiedene Foreneinträge aus hinteren Seiten anschaut und beim Verlassen eines jeden Foreneintrags wieder prompt auf Seite 1 geschickt wird. Vielleicht gibt es das Feature schon und ich habe es nur (noch) nicht gefunden.

[C0ntroller]

OK, damit kann man arbeiten.

Pauschal gibts das noch nicht, wie gesagt nur auf der Seite Meine Kurse und Favoriten.

Ich denke, man kann es auf jeden Fall mit auf die Liste setzen.

[Noxdor]

@Noxdor die Option müsste durch dich in das Front-End eingebaut werden

Das sollte relativ straight forward sein, einfach ein Toggle Button ob man die Funktion nutzen will oder nicht würde ich sagen?

[OliEfr]

@Noxdor ja
@C0ntroller okay, ich bin auch dafür. Sollen wir den PR zu einer ToDo in einem Projekt hinzufügen?

[C0ntroller]

Mir reicht es als offenes Issue, ich würde gern erstmal #103 fertig haben