search

Seglormeister / pr0gramm.com-by-Seglor

http://pr0gramm.com
26 stars 3 forks source link

Funktioniert nicht / Webseite bleibt Schwarz #22

Open Cheatyx opened 8 years ago

Cheatyx commented 8 years ago

Geht das Script bei euch? Bei mir bleibt die Webseite dann Schwarz.

DerMozart commented 7 years ago

Ich konnte einige Stellen fixen: // zum passenden Thumb scrollen var itemId = document.URL; var itemname = '#item-' + itemId.substring(itemId.length - 6, itemId.length); var posi = 0; if ($(itemname) != null && $(itemname).offset()) { posi = $(itemname).offset().top - 52; } if ($(window).scrollTop() != posi) { //window.scrollTo(0, posi); //ohne Animation $('html').stop(); $('html').animate({scrollTop: posi}, 300, 'swing', function() { return;}); }

Das Problem bleibt jetzt bei WebMs da bekomm ich den Fehler: canPlayWebM is not defined in unknown der Fehler tritt aber wie es scheint im pr0gramm.min.js auf. Geprettyfied sieht das so aus: if (canPlayWebM) { ;_parts.push('

'); }

und genau da zerlegts ihn. Ach ja, ist im Tampermonkey in Chrome

Cheatyx commented 7 years ago

Ich kenne mit leider mit Greasemonkey Scripten nicht aus, aber guter hinweis, bei werden auch nur die webm zerschossen, ist mir gar nicht aufgefallen.

Geht leider vieles nicht mehr: Votingpunkte in den Kommentaren Kommentarbereich sit verschoben Bilder werden auf Bildschirmgröße gestreckt Die genannten webm

Lokke commented 7 years ago

"if (canPlayWebM)" - wenn ich es richtig sehe spricht das pr0gramm kein ".webm" mehr - was ich auf den ersten blick finden kann ist ".mp4"

Cheatyx commented 7 years ago

Ja du ahst recht, inzwischen wurde alles auf mp4 abgeändert.

FlorianMaak commented 7 years ago

Dann behebe das doch bitte gleich und reiche ein PR ein,

Lokke commented 7 years ago

Dann behebe das doch bitte gleich und reiche ein PR ein,

Wenn ich es könnte, hätte ich es behoben.

pwnyprod commented 7 years ago

Benutzt den Fork hier https://github.com/FlorianMaak/pr0gramm.com-by-Seglor Allerdings gibt es da auch noch einige Fehler.

Eventuell könnte man den Fork mal in die Repo hier mergen und ihm die Commitrechte geben. @Seglormeister

FlorianMaak commented 7 years ago

Edit: Hier ein Rework. Der Breitbildmodus kann anschließend über die Einstellungen aktiviert werden: https://github.com/FlorianMaak/p0weruser

Wenn ich fertig bin reiche ich ein PR ein. Der Fork funktioniert an sich, aber hat noch einige Unschönheiten, die ich vorher beheben wollte. - Arbeit eingestellt!

Wenn jemand helfen/testen möchte, so fühlt euch gerne eingeladen. Wenn ihr Fehler findet, dann eröffnet bitte ein neues Issue.

Dem Wunsch nach neuen Features komme ich auch gerne nach. Vorher möchte ich jedoch erst ein Mal eine fehlerfreie Grundversion schaffen.

Aktuelle ToDo:

Im Anschluss stehen dann folgende Improvements auf dem Plan:

Fehler gefunden?

https://github.com/FlorianMaak/pr0gramm.com-by-Seglor/issues

Neugierig wie weit die Entwicklung ist?

https://github.com/FlorianMaak/pr0gramm.com-by-Seglor/milestones

Cheatyx commented 7 years ago

Wow, vielen lieben Dank, ich hatte es mal selber versucht, bin aber mit diesem Scripten noch ein Neuling

FlorianMaak commented 7 years ago

@Cheatyx Für den Einstieg ist dieses UserScript auch definitiv nicht geeignet. (Dafür ist es zu unsauber) Ich kann dir jedoch folgendes Werk ans Herz legen, wenn du JS lernen möchtest: Speaking JavaScript

Wenn du dich anschließend für ES6 interessieren solltest:

pwnyprod commented 7 years ago

Sehr gut, bei bedarf können wir mal Messager Daten austauschen, ich habe deine Repo geforkt und wollte selber bei dir pr pushen ^^

FlorianMaak commented 7 years ago

Wenn sich schon ein weiterer Entwickler gefunden hat, würde ich vermutlich direkt die Chance ergreifen und das ganze neu schreiben. Dann kann man da ggf mal etwas Struktur reinbringen.

Seglormeister hat mir auf dem pr0 auch leider nicht auf meine Nachricht geantwortet, sodass auch weniger davon auszugehen ist, dass der Fork je seinen Weg ins Repo finden wird.

pwnyprod commented 7 years ago

Sehr schade, Ja ich hab auch erstmal alle reformater und fixer drüber laufen lassen und hat mir danach immernochnicht gefallen :D aber JS is halt immer schwer zu strukturieren, vorallem weil man ja besonders bei US keine Filestruktur anlegen kann 🙈 Wenn sich mehr Devs finden sollten könnte man sogar über nen Slack Workspace nachdenken. 😏

PS: Oder welche Messenger bevorzugst du?

FlorianMaak commented 7 years ago

Während der Entwicklung lässt sich mit mehreren @require arbeiten, um mehrere (lokale) Files einzubinden. Nimmt man sich Grunt oder andere TaskRunner zur Hilfe und lässt bei einem Release dann eine einzelne Datei draus bilden. So kann man das Projekt auch vernünftig strukturieren und beim User kommt dennoch nur eine File an. Da während der Entwicklung mit Einzeldateien gearbeitet wird, kann man auch sauber debuggen.

Ich würde mich die Tage (spätestens jedoch am Wochenende) mal an eine entsprechende Projektstruktur setzen, sie dokumentieren und sie dir zukommen lassen.

Slack wäre denke ich angemessen. Alternativ könnte ich auch einen XMPP-Server stellen.

FlorianMaak commented 7 years ago

Ich habe dir mal einen Invite für nen Repo geschickt. Für die Diskussion habe ich ein Issue erstellt, welches wir nutzen können. (Ich möchte die Entwicklung gerne so offen wie möglich halten)