Om nah hoo pez nyeetz, Gunther!

Würde es nicht reichen, dass du den gerade gelesenen Thread zuklappst?

Matthias

--
Der Unterschied zwischen Java und JavaScript ist größer als der zwischen Park und Parkinson.

Om nah hoo pez nyeetz, alle!

Wie wäre es mit einer Änderung des Textes? Der momentane Linktext in der Themenbereichsstatistik lautet „zeige Autor(inn)en-Statistik“. Für mich bezog sich das auf die darunter befindliche Liste statt auf die „Andere“. Mit „wechsle zur Autor(inn)enstatistik“ wäre zweifelsfrei klar, dass es woanders hin geht.

Stimmt!
Dann hießen aber die beiden Links, die dasselbe bewirken, unterschiedlich. Denn auf der Hauptseite der Link heißt ja "Zeige Themenbereich-Statistik".

Sie bewirken nicht dasselbe.
Der eine öffnet das Statistik-div, der andere wechselt den Inhalt dieses Elements ;-p

Deshalb bin ich auch für eine zweifelsfreie Formulierung.

Überredet! ;-)

Wie hättet ihr es denn gerne?
"Zur XYZ-Statistik wechseln"
"Wechsel zur XYZ-Statistik"
oder ... ?

Gruß
Gunther

Hallo

• Threads öffnen/schließen: per default schließen*
  Wieso sollte diese Option dafür erforderlich sein!?

Wir reden von „Gelesen-Markierung klappt Thread zu“

Ah ja, stimmt (du bist bei "zuklappen" anstatt nur die "Gelesen-Markierung" zu meinen).
Das ist natürlich unter dem Gesichtspunkt, dass es in diesem Thread um das JS-Skript etwas "kontraproduktiv" ...! :-P

Ja, man kann hier offensichtlich „viel zu viel“ einstellen. Wobei es nicht viel zu viel ist, sondern vieles, was sich mit manch anderem beißt und damit selbst unter uns angeblich „Wissenden“™ Verwirrung stiftet.

Es ist nun mal so, dass sich das Skript um das auf- und zuklappen der Threads kümmert.
Wert also, aus welchen Gründen auch immer, diese Einstellung verwenden möchte, der kann dann nicht gleichzeitig auch das JS-Skript verwenden - Punkt! ;-)

Bei mir wird es darauf hinauslaufen. Ich habe mich zu sehr an das durch Molilys Skript herbeigeführte Verhalten gewöhnt. Deine Klappfunktion für den Forumskopf, die möchte ich nämlich behalten, werde ich mir raussuchen und in Molilys Skript einfügen. *Das* sollte ich noch selbst hinbekommen.

Um bspw. neue Postings zu sehen, ist ja auch zwingend ein Reload der Seite erforderlich.

Ich habe letztens ein Board gesehen (war ein WoltlabBB), bei dem unterhalb der Postings angezeigt wird, dass zwischen dem laden des Threads und jetzt (ein) weitere(s) Posting(s) eingegangen sind. Das ist in manchen Szenarien bestimmt praktisch, mich alten Knochen hat's aber eher verwirrt.

Das heißt ja, wenn ich es richtig sehe, dann reden wir hier einzig von dem Punkt, dass Threads_ohne_ungelesene Postings "automatisch" zugeklappt werden, und das auch, wenn die Hauptseite nach dem Öffnen der ungelesenen Postings nicht neu geladen wird, richtig?

Ja. Ja, das ist ein Spezialfall.

Tschö, Auge

Hallo

Ja, da ich nun dein User-CSS statt meinem mit den diversen Ausblendungen von Bereichen nutze, ist der Forenkopf auf einem Full-HD-Display mit Browser im Vollbildmodus – man soll es nicht für möglich halten – tatsächlich höher als der Viewport des Browsers (etwa 890px).

Schon die kleinen ▼ entdeckt?

Ja, latürnich.

Das zuklappen von Kästen im Seitenkopf könnte mir ja schon gefallen …

Das ist auch nicht bierernst gemeint. Gunther hat nach unerwünschten Nebenwirkungen gefragt und diese ist, wenn auch durch mich selbst behebbar, sowohl eine Nebenwirkung als zu allem Überfluss auch unerwünscht. ;-)

Tschö, Auge

Hallo

Da fällt mir ein: Gunnar, du hast dich noch nicht zu meinem Vorschlag über Github geäußert. …

schon wieder dafür

Tschö, Auge

hi Matthias,

Om nah hoo pez nyeetz, tami!

finde aber aus didaktisch-pädagogischen gründen durchaus einen reiz darin, auch als "lehrbeispiel" für selfhtml ...; aber ich rieche schon, dass es dafür "um die ohren" gibt ...;

Warum sollte es?

SELFHTML hat ganz klar einen Bildungsauftrag. Und dazu zählt auch ein sauber eingerückter Code. In den Wikibeispielen achten wir sehr darauf, bei den anderen Sachen ist es nicht immer möglich.

Also ich bin bei Zeile 120. Es ist auch nicht nur die Einrückung, wie ich schrieb. typeof soll weg, das war das eine mal, wo ich in den Code eingeriffen habe.

"test.js:122:17:Unexpected 'typeof'. Use '===' to compare directly with undefined."

Und die dreifachen Gleichheitszeichen:

configChecked === 'false'
mit
configChecked = sessionStorage.getItem('config_checked');

ich gehe davon aus, das .getItem einen String in dem Fall zurück gibt.

"test.js:125:67:Expected '===' and instead saw '=='."

Für ein Gefühl, was jslint mir ausspuckt:

C:\Program Files\nodejs\node.exe C:\jslint-reporter\wrapper.js test.js

test.js:127:25:Combine this with the previous 'var' statement.
test.js:128:23:Expected exactly one space between 'if' and '('.
test.js:129:29:Combine this with the previous 'var' statement.
test.js:130:29:Combine this with the previous 'var' statement.
test.js:131:27:Expected exactly one space between 'if' and '('.
test.js:135:25:Unnecessary 'else' after disruption.
test.js:136:29:Expected 'sessionStorage' at column 33, not column 29.
test.js:137:25:Expected '}' at column 29, not column 25.
test.js:140:16:Unexpected character '(space)'.
test.js:145:19:Expected exactly one space between 'if' and '('.
test.js:145:49:Expected '{' and instead saw 'Storage'.
test.js:145:49:Stopping. (7% scanned).

Also ja: über 50% sind Einrückungen, es sind aber auch fehlende Semikola nach Funktionsbeendigungen, fehlende Blanks (und zuviele Blanks hinter einem Semikolon), Zusammenfassung der var-Statements am Anfang (das ist etwas mühselig, die alle da oben reinzuschreiben ...).

Ich paste hier mal die ersten 172 Zeilen jslint-konform und sehe grade, dass er mir einen Fehler gibt, den ich nicht zuordnen kann:

test.js:116:61:Uninitialized 'threads_open'.
test.js:116:75:Uninitialized 'Config'.

beide mit "undefined" zu initialisieren ist blödsinn, dass will "er" nicht:

test.js:116:74:It is not necessary to initialize 'threads_open' to 'undefined'.

ich hoffe zudem dass:
test.js:148:37:['HighlightVisitedPostings'] is better written in dot notation.
test.js:148:91:['HighlightOwnPostings'] is better written in dot notation.

mit dot notation auch "funzt":

if (Config.Settings.HighlightVisitedPostings !== 'yes' || Config.Settings.HighlightOwnPostings !== 'yes') {

Ansonsten hier die ersten 172 Zeilen (inkl. o.g. Fehler mit nichtinitialisierten Variablen);

  
/*jslint browser : true, devel: true*/  
/*!  
JavaScript fuer die benutzerspezifische Ansicht des SELFHTML-Forums (http://forum.de.selfhtml.org/)  
Author: Mathias Schaefer (molily)  
Aenderungen & Ergaenzungen: Gunther Pilz (Gunther)  
Lizenz: MIT License  
*/  
  
(function () {  
  
    'use strict';  
  
// ####################################################################################  
  
// Check browser support  
  
    var  
        div = document.createElement('div'),  
        supported =  
        div.querySelector && div.querySelectorAll &&  
        div.classList && div.classList.contains &&  
        div.textContent === '' &&  
        div.addEventListener &&  
        window.sessionStorage &&  
        window.localStorage &&  
        window.getComputedStyle &&  
        window.XMLHttpRequest &&  
        Function.prototype.bind,  
        firstVisit = false,  
        lastVisit = sessionStorage.getItem('session_start'),  
        lastReload = sessionStorage.getItem('last_reload'),  
        forEach, escapeHTML, slice, timeout, clearIntervals, getLength, Modules;  
  
    if (!supported) {  
        if (window.console && console.error) {  
            console.error('JavaScript-Erweitungen: Sie verwenden einen nicht unterstuetzten Browser. Bitte verwenden Sie eine aktuelle Browserversion.');  
        }  
        return;  
    }  
    div = undefined;  
  
    if (!lastVisit) {  
        //lastVisit = Date.now();  
        lastVisit = localStorage.getItem('logged_out');  
        if (!lastVisit) {  
            lastVisit = Date.now();  
        }  
        localStorage.setItem('last_visit', lastVisit);  
        sessionStorage.setItem('session_start', Date.now());  
        sessionStorage.setItem('config_checked', 'false');  
        firstVisit = true;  
    } else {  
        lastVisit = localStorage.getItem('last_visit');  
    }  
  
    if (!lastReload) {  
        lastReload = lastVisit;  
    }  
  
    // ####################################################################################  
  
    // Helfer  
  
    forEach = function (object, func) {  
        var property;  
        for (property in object) {  
            if (object.hasOwnProperty(property)) {  
                func.call(object, property, object[property]);  
            }  
        }  
    };  
  
    escapeHTML = function (string) {  
        return string  
            .replace(/&/g, '&')  
            .replace(/</g, '&lt;')  
            .replace(/>/g, '&lt;')  
            .replace(/'/g, '&quot;')  
            .replace(/'/g, '&#39;');  
    };  
  
    slice = Array.prototype.slice;  
  
    if (!Array.from) {  
        Array.from = function (list) {  
            return slice.call(list);  
        };  
    }  
  
    timeout = function (id) {  
        window.clearTimeout(id);  
    };  
  
    clearIntervals = function (id) {  
        window.clearInterval(id);  
    };  
  
    getLength = function (arr) {  
        var len = arr.length;  
        console.log('Config.activeXMLHttpRequests.length: ' + len);  
        return len;  
    };  
    // ####################################################################################  
  
    // Modulverwaltung  
  
    Modules = {};  
  
    Modules.queue = [];  
  
    Modules.add = function (module) {  
        Modules.queue.push(module);  
    };  
  
    Modules.init = function () {  
        var configMismatch = false, configChecked, root, a, threads_open, Config, tmp, ConfigDialog, ret;  
        if (document.body.id === 'selfforum-hauptseite') {  
            configChecked = sessionStorage.getItem('config_checked');  
            root = document.getElementById('root');  
            a = root.querySelector('a');  
  
            if ((threads_open === undefined || threads_open === false) || !a || !a.onclick) {  
                configMismatch = true;  
            } else if (firstVisit || !configChecked || configChecked === 'false') {  
                Config.Check('Name', true);  
                if (!Config.Settings.Name) {  
                    tmp = sessionStorage.getItem('name');  
                    if (!tmp) {  
                        ConfigDialog = 'Du hast keinen (Posting)Namen angegeben!\nWillst du jetzt die Benutzer-Einstellungen aufrufen,\num einen Namen einzugeben?';  
                        ret = window.confirm(ConfigDialog);  
                        if (ret) {  
                            window.location = 'http://forum.de.selfhtml.org/cgi-bin/user/fo_userconf#neuepostings';  
                            return;  
                        }  
                        sessionStorage.setItem('name', '');  
                    }  
                }  
  
                sessionStorage.setItem('name', Config.Settings.Name);  
                Config.Check('HighlightVisitedPostings', true);  
                Config.Check('HighlightOwnPostings', true);  
                Config.Check('SortThreads', true);  
                if (Config.Settings.SortThreads) {  
                    Storage.setValue('thread_sort', Config.Settings.SortThreads);  
                } else {  
                    Storage.setValue('thread_sort', 'newestfirst');  
                }  
                if (Config.Settings.HighlightVisitedPostings !== 'yes' || Config.Settings.HighlightOwnPostings !== 'yes') {  
                    configMismatch = true;  
                } else {  
                    sessionStorage.setItem('config_checked', 'true');  
                }  
            }  
  
        }  
        if (!configMismatch) {  
            Modules.queue.forEach(function (module) {  
                if (module.documentType === 'all' || document.body.id === 'selfforum-' + module.documentType) {  
                    module.init();  
                }  
            });  
        } else {  
            Config.Show('mismatch');  
        }  
    };  
  
    document.addEventListener('DOMContentLoaded', Modules.init);  
  
    // ####################################################################################  
  
    // Hauptobjekt mit Caches  
  

Matthias

mfg

tami

hi Gunther,

wenn nicht alle das völlig sinnlos fänden, hätte ich u.u. lust (habe grade schon angefangen, aber das ist ja doch recht mühsam), da script komplett jslint-konform zu "kriegen. macht natürlich nur dann sinn, wenn da grade mal ne atempause ist, und danach dann an der konformen version weitergearbeitet würde. wie ich es jetzt sehe, betrifft das in den ersten 60 zeilen:

• umlaute
• einrückungen
• zusammenfassung der var-statements
• prüfung auf hasOwnProperty bei dem property in object.

ist vielleicht auch blödsinn, bei fast 2000 zeilen code, ich bin jetzt erst bei 4% angekommen, als ich es eben mal probiert habe. finde aber aus didaktisch-pädagogischen gründen durchaus einen reiz darin, auch als "lehrbeispiel" für selfhtml ...;

Dafür dann ausgerechnet ein von mir "vermurxtes" Skript zu nehmen ...! :-P
Ich bin kein "Javascript'ler" und dann auch noch ohne jQuery.

Ohne molilys sehr "solides Grundgerüst", wäre ich keinen Meter weit gekommen.

Aber du solltest vlt. besser warten, bis wir sozusagen die endgültige Version stehen haben (mir schweben da noch ein paar kleine Änderungen/ Ergänzungen vor). ;-)

Und ggf. reicht ja auch JSHint statt JSLint?

Also wenn es nach mir ginge, dann würde ich das "Original" nehmen. Habe ja grade die ersten 172 Zeilen gepostet. Würde jetzt erstmal sowieso nix machen weiter, war nur ein Testlauf und mal gucken, wie die Resonanz so ist. Geht aber eigentlich dann doch ganz fix. Habe es mit in meine SciTE so eingerichtet, dass er mit mit F5 das jslint ausspuckt und mit F4 springe ich dann zum ersten "Fehler", den er mir angibt.

Einrückungen sind ja schnell beseitigt (keine Tabs, 4 Blanks Standard-Indentation). Die vars will "er" immer all zusammen am Funktionsanfang haben, kommasepariert, egal, ob sie da initialisiert werden oder nicht. Wie ich aber grade sehe, besteht er schon darauf, dass eine var dann überhaupt initialisiert wird, bevor sie gegen "undefined" oder ähnliches getestet wird, was ja auch sinnvoll ist ...;

mfg

tami

Moin Auge,

… ich wollte nicht auf die Abbildung der Realität hinaus. Es ging mir eher um die Performance auf dem Server. Auch wenn heutzutage weit weniger im Forum los ist, als z.B. vor etwa acht bis zehn jahren, kann ich mir vorstellen, dass zu Stoßzeiten viele solcher Anfragen mehr oder minder gleichzeitig bearbeitet werden müssten. Inwieweit das ein Problem darstellt, weiß ich natürlich nicht. Kann ja auch sein, dass das im Grundrauschen unterginge.

Warum sollte es den Server belasten, wenn man via JS (Client-seitig) .visited setzt? Der wird da doch gar nicht befragt.

Ausserdem muss der das ab können, das neue Forum ist ja in Ruby mit Rails geschrieben, und da werden viel mehr Ressourcen gebraucht.

Das frühere Verhalten, ein Posting in der Hauptansicht binnen ein bis zwei Sekunden nach dem Aufruf des Postings in einem anderen Tab mit den Regeln für die serverseitige Markierung als gelesen zu markieren, lässt mich vermuten, dass das schon einmal genau so funktionierte.

Das ist ein Irrtum. Das ist ein Browser-Feature gewesen, das Links entsprechend :visited eingefärbt hat.

Das kann nicht sein™. Wenn binnen ein oder zwei Sekunden ohne Reload der Hauptseite in dieser ein soeben in einem anderen Tab geöffnetes Posting mit der für die serverseitige Markierung festgelegten CSS-Regel versehen wird, die sich definitiv von der für a:visisted unterscheidet, ist da mehr passiert. Das hat bis vor etwa drei oder vier Jahren so funktioniert.

Die ein bis zwei Sekunden erklären sich anhand der load time des Postings. Ansonsten kann ich dazu nur sagen: ich habe da nichts eingebaut. Wenn da wirklich etwas via JS passiert ist, dann musst du von irgendwo her JS eingebunden haben, dass das implementiert hat.

LG,
 CK

Om nah hoo pez nyeetz, Auge!

Das kann nicht sein™. Wenn binnen ein oder zwei Sekunden ohne Reload der Hauptseite in dieser ein soeben in einem anderen Tab geöffnetes Posting mit der für die serverseitige Markierung festgelegten CSS-Regel versehen wird, die sich definitiv von der für a:visisted unterscheidet, ist da mehr passiert. Das hat bis vor etwa drei oder vier Jahren so funktioniert.

aktuelles CSS:

.visited, a.visited, #root a:visited {  
    color: #888;  
}

In den Threadbäumen der Thread-Ansicht greift nur

a:link, a:visited {  
    color: #3481CD;  
}

Allgemein lässt sich sagen, dass bei Verwendung der Thread-Ansicht die einzelnen Postings serverseitig als gelesen markiert werden, bei Verwendung der nested- oder Listenansicht alle, die es gerade im aktuellen Thread gibt.

Matthias

--
Der Unterschied zwischen Java und JavaScript ist größer als der zwischen Not und Note.

Om nah hoo pez nyeetz, Gunther!

Versteh' ich jetzt nicht ganz ...!?
Die Gelesen-Markierung wird/ würde ja nur bei dem Posting eingefügt, welches du auf der Hauptseite auch angeklickt hast.

Wenn du also Posting Nr. 21 angeklickt hast, und per Back-Button zurück gehst, ist Posting Nr. 75 doch (immer noch) als ungelesen da!

Genau. Wenn ich aber in der nested-Ansicht unterwegs bin, werden alle Postings des Threads, die zu diesem Zeitpunkt vorhanden sind, als gelesen markiert, was ja auch sinnvoll ist, denn ich hatte sie ja alle auf dem Schirm (=vom Server angefordert). Du kannst eben nicht mit vertretbarem Aufwand prüfen, ob sich ein bestimmtes Posting ganz oder teilweise im Viewport befand. (nested-Ansicht!)

Matthias

--
Der Unterschied zwischen Java und JavaScript ist größer als der zwischen Barbie und Barbier.

Hallo

Oder würde es "Sinn machen", quasi das von Auge gewünschte Verhalten zu implementieren, und die Klasse 'visited' vorab per JS zu ergänzen (beim nächsten Reload müsste sie ja dann eh bereits serverseitig vorhanden sein)?

Das wäre möglich und aus meiner Sicht sinnvoll, kollidiert aber u.U. mit gewünschtem Verhalten.

Wie ich gerade schon in meinem anderen Posting geschrieben habe, reicht diese Maßnahme alleine ja nicht aus.

*Wozu* reicht sie nicht aus? Die Frage ist mMn an dieser Stelle, wozu das Setzen der Klasse ohne Reload dienen soll. Ist es dazu da, deiner clientseitigen Prüfung auf offene Postings vor dem zuklappen eines Threads zu genügen, reicht die Maßnahme. Für andere Anwendungsfälle, mag das nicht zutreffen …

Kann es eigentlich sein, dass die Benutzer-Einstellung "Listed-/Nested-Darstellung: In der Listen- bzw. Nested-Darstellung den kompletten Thread als gelesen markieren" nicht funktioniert?

… und die Markierung funktioniert für die Nestedansicht eh nicht, wie sie mal für die Threadansicht entwickelt wurde.

Tschö, Auge