Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

placement of "new" comments form #57

Open
frase-git opened this issue Nov 12, 2017 · 8 comments
Open

placement of "new" comments form #57

frase-git opened this issue Nov 12, 2017 · 8 comments

Comments

@frase-git
Copy link

Hoffentlich kein Duplikat (#10)

Ich bin gerade bei einer Seite mit 2Cents und wünsche mir den Button "Neuen Kommentar verfassen" unterhalb der angezeigten Kommentare.

Ginge das als Konfig-Option?

Im Moment behelfe ich mir mit einer Änderung der Reihenfolge in "views/comments.php" - was aber nicht updatesicher ist.

@cmb69
Copy link
Owner

cmb69 commented Nov 12, 2017

Ich bin gerade bei einer Seite mit 2Cents und wünsche mir den Button "Neuen Kommentar verfassen" unterhalb der angezeigten Kommentare.

Derzeit sollte es so sein, dass dieser Schalter im Prinzip (Paginierung außen vor) dort angezeigt, wo dann auch der gepostete Kommentar steht (eben abhängig von comments_order). Passt das nicht?

@frase-git
Copy link
Author

Version: 1.0beta3
(Keine Paginierung)
Egal, ob ASC oder DESC - der Button wird immer oberhalb der Kommentare angezeigt.

@cmb69
Copy link
Owner

cmb69 commented Nov 12, 2017

Hm, bei mir (auch 1.0beta3) klappt es wie es soll. Zur Erklärung: der Button selbst wird per JS erzeugt, und sollte genau an der Stelle angezeigt werden, an der auch das Formular ausgegeben wurde. Also bitte mal temporär JS deaktivieren, und prüfen, ob das Formular richtig platziert ist. Falls nicht, dann liegt es vielleicht an einem modifizierten views/comments.php (die entsprechende Zuweisung im Controller sieht zumindest korrekt aus). Falls das Formular selbst korrekt platziert ist, dann gibt es wohl ein JS Problem, und man müsste browserseitig weiter suchen.

@frase-git
Copy link
Author

Verstehe.
Habe mit DESC angefangen und es stand oben.
Dann habe ich die comments.php manipuliert.
Habe also nicht richtig "geguckt".

Das Problem bleibt, dass es nicht konfigurierbar ist.
Ich möchte DESC haben und den Button (das Formular) unten.
Das geht momentan nur mit Manipulation.

@frase-git
Copy link
Author

frase-git commented Nov 12, 2017

Übrigens:
Ich habe einen Text über den Kommentaren, in dem auf die Kommentarmöglichkeit unten hingewiesen wird. Interner Link zum Formular.
(War eine alte Seite mit "comments". Da hatte das Formular eine ID, die man anspringen konnte.)
das geht mit Twocents gar nicht mehr - oder? (manipulieren)

Korrektur:
Der Button braucht eine ID - nicht das Formular, da es zunächst unsichtbar ist.

in twocents.min.js z.B. einfügen: button.id="twocents_write_button";
ungefähr hier:

... button.className="twocents_write_button";button.id="twocents_write_button";button.onclick= ...

Und jetzt habe ich noch ein Problem im Formular (HTML):
"I" = italic lässt sich nicht mehr ausschalten.
Die Knopf-Formatierung "inset/outset" spinnt auch.
Kannst du das nachvollziehen?

@cmb69
Copy link
Owner

cmb69 commented Nov 12, 2017

Ich möchte DESC haben und den Button (das Formular) unten.

Bedeutet halt eine zusätzliche Konfigoption, aber okay.

Ich habe einen Text über den Kommentaren, in dem auf die Kommentarmöglichkeit unten hingewiesen wird. Interner Link zum Formular.
(War eine alte Seite mit "comments". Da hatte das Formular eine ID, die man anspringen konnte.)
das geht mit Twocents gar nicht mehr - oder? (manipulieren)

Das geht leider tatsächlich nicht. Ich muss allerdings vermutlich diesbezüglich sowieso etwas umbauen (das Formular per JS zu verstecken, war wohl nicht die beste Idee). Wenn das so geändert wurde, dann könntest du direkt auf dieses Formular verlinken.

Der Button braucht eine ID - nicht das Formular, da es zunächst unsichtbar ist.

Kommt halt drauf an, ob das JS ausgeführt wird, oder nicht. Und dem Button eine ID zu verpassen, wird nicht unbedingt funktionieren, da dieser u.U. zu spät erzeugt wird, und der Browser bereits vorher versucht hat, den Anker anzuspringen.

Und jetzt habe ich noch ein Problem im Formular (HTML):
"I" = italic lässt sich nicht mehr ausschalten.
Die Knopf-Formatierung "inset/outset" spinnt auch.
Kannst du das nachvollziehen?

Das passiert wohl nur, wenn unterschiedliche Schriftarten für italic/bold definiert wurden. Da erzeugt der Browser noch zusätzlich ein entsprechendes <font> Element, entfernt es aber nicht wieder. Probier im Zweifel mal das Kommentarformular mit einer gemeinsamen Schriftart für roman/italic/bold zu stylen. Ich schau aber mal, ob ich das nicht allgemein in den Griff bekomme.

PS: inset/outset liegt am Stylesheet, das border:0 setzt.

@frase-git
Copy link
Author

frase-git commented Nov 12, 2017

Der Button braucht eine ID - nicht das Formular, da es zunächst unsichtbar ist.

Kommt halt drauf an, ob das JS ausgeführt wird, oder nicht. Und dem Button eine ID zu verpassen, wird nicht unbedingt funktionieren, da dieser u.U. zu spät erzeugt wird, und der Browser bereits vorher versucht hat, den Anker anzuspringen.

Stimmt. Bei ausgeschaltetem JS klappt eines nicht, bei eingeschaltetem das andere. :(

Das passiert wohl nur, wenn unterschiedliche Schriftarten für italic/bold definiert wurden.

Autsch! An was man alles denken muss.
Test:

.twocents_editor {
    font-family: arial,sans-serif !important;
    font-weight: normal;
    font-style: normal;
...
}
.twocents_editor b {
    font-family: arial,sans-serif !important;
    font-weight: bold;
    font-style: normal;
}
.twocents_editor i {
    font-family: arial,sans-serif !important;
    font-weight: normal;
    font-style: italic;
}

Das klappt soweit, aber was wird mit Kombinationen aus b i ?

PS
Es sind die gleichen Schriftarten, aber unterschiedliche Schriftschnitte.
Bei CSS lädt man allerdings tatsächlich unterschiedliche Dateien, sodass hier unterschiedliche Schriftarten nicht falsch ist.

@cmb69
Copy link
Owner

cmb69 commented Nov 12, 2017

Das klappt soweit, aber was wird mit Kombinationen aus b i ?

Siehe #59 (TL;DR: ich kann vermutlich fixen, aber das müsste serverseitig noch korrigiert werden). Bitte weitere Kommentare zu diesem Thema besser dort vornehmen, damit man auch später noch einen Überblick haben kann. :)

Es sind die gleichen Schriftarten, aber unterschiedliche Schriftschnitte.

Danke für die Erklärung (ich versuche mir "Schriftschnitte" zu merken). :)

Bei CSS lädt man allerdings tatsächlich unterschiedliche Dateien, sodass hier unterschiedliche Schriftarten nicht falsch ist.

Und im Extremfall sogar richtig (z.B. * {font-family:serif} i {font-family:sans-serif}), auch wenn das typografisch wohl arger Blödsinn wäre.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants