Ich habe eine Site, auf der „Nicht kategorisierte Themen zulassen
Hast du alle Themen aus der Kategorie ‘Unkategorisiert’ entfernt?
Jep! Es geht nur um das Thema, nichts anderes. Ich verstehe nicht ganz, wie das mit den Spezifikationen, die ich annehme, überhaupt möglich sein könnte. Und es betrifft nur Mobilgeräte, nicht Desktop.
Können wir das @tshenry nachstellen? Klingt nach einem Problem, das nur auf Mobilgeräten auftritt?
Ich glaube nicht, dass wir erwarten, dass das Thema „Über uns
Ah, okay. Das hat es behoben.
Das ist ziemlich verwirrend, denn normalerweise ist es verboten, ein „Über uns“-Thema zu löschen (aber ich vermute, es ist kein echtes „Über uns“-Thema, da es keine Kategorie ist). Ich habe es dennoch gelöscht, und jetzt wird „Nicht kategorisiert“ nicht mehr angezeigt. Ich musste es auf meinem Handy tun, da ich es sonst nicht finden konnte.
„Nicht kategorisiert“ ist sehr verwirrend, da es KEINE Kategorie ist. Ich bin wirklich der Meinung, dass es standardmäßig deaktiviert sein sollte.
Nein, denn ich glaube nicht daran, von Anfang an eine künstliche Klassifizierung in Tier, Pflanze oder Mineral auf jedes einzelne Element in der Welt zu erzwingen. Entschuldigung.
Bitte beachte die Warnung, die aus gutem Grund enthalten ist:
WARNUNG: Falls es irgendeine nicht kategorisierte Themen gibt, musst du sie neu kategorisieren, bevor du dies deaktivierst.
Trotzdem: Können wir dieses Verhalten @zogstrip normalisieren, damit es zwischen Mobilgeräten und Desktop konsistent ist?
Ja. Ich verstehe dieses Argument und habe früher zugestimmt, aber ich denke, eine Standard-Kategorie würde mehr Sinn ergeben als das Fehlen einer Kategorie. Ich werde aber versuchen, das nicht wieder vorzuschlagen, es sei denn, wir sind eines Tages vielleicht im selben Raum mit
.
@nbianca, könntest du deiner Liste hinzufügen, zu untersuchen, warum die Kategorie „Nicht kategorisiert
Auf dem Desktop zeigen wir Kategorien und Themen separat an, auf mobilen Geräten hingegen kombinieren wir Kategorien mit ihren Themen. Unter Berücksichtigung dessen ist es diese Zeile, die den Fehler verursacht:
c.displayable_topics.blank? ist auf dem Desktop immer wahr, da wir Themen separat laden und displayable_topics daher stets leer ist. Auf mobilen Geräten ist es nur dann wahr, wenn in dieser Kategorie wirklich keine Themen vorhanden sind.
Wir haben hier zwei Lösungen:
-
“Nicht kategorisiert” anzeigen, wenn mindestens ein Thema vorhanden ist, unabhängig vom Wert der Einstellung “Nicht kategorisierte Themen zulassen”.
-
“Nicht kategorisiert” ausblenden, selbst wenn ein Thema vorhanden ist, falls die Einstellung “Nicht kategorisierte Themen zulassen” auf falsch gesetzt ist.
Die zweite Lösung ist einfacher umzusetzen, da wir lediglich den zweiten Teil der Bedingung entfernen müssen.
Ich finde, dass das Ausblenden der Themen in „Nicht kategorisiert
Lass uns die einfachere Variante wählen, da wir kurz vor einem Release stehen und ich nichts zu Risikoreiches unternehmen möchte.
Meine Stimme hier lautet: „So viel speziellen, nicht kategorisierten Code wie möglich entfernen. Je mehr davon wir haben, desto verwirrender wird es.
Daher würde ich für Folgendes stimmen:
Nicht kategorisierte Themen anzeigen, wenn mindestens ein Thema existiert, unabhängig vom Wert der Einstellung „Nicht kategorisierte Themen zulassen“.
Vorausgesetzt, die Implementierung entfernt den speziellen Magie-Code aus Discourse und wir haben weniger Sonderfälle zu berücksichtigen.
Im Grunde genommen die Lösung wählen, die mehr Code aus Discourse entfernt.
Ich habe einen Fix für den Fehler im ersten Beitrag eingereicht:
Ich habe mich für die erste Lösung entschieden, da diese Code aus Discourse entfernt. Die andere Lösung fügte tatsächlich Code hinzu, um zu prüfen, ob mindestens ein Thema in der Kategorie „Nicht kategorisiert“ vorhanden ist.
Außerdem habe ich alle Verwendungen von category.uncategorized? überprüft und einige Stellen gefunden, die möglicherweise nicht mehr notwendig sind, wenn wir „Nicht kategorisiert“ weniger als Einhorn und mehr als normale Kategorie behandeln:
Dies sind einige Teile, die keine anderen Funktionen beeinträchtigen, was die Änderungen weniger riskant, aber dennoch nicht einfach macht.