Die Optionen der Umfrage hatten keinen Layout-Container, wodurch @mention-Pillen das Optionszeichen oder Kontrollkästchen überlappen konnten.
Diese Änderung macht Umfrageoptionen zu Flex-Zeilen, sodass das Zeichen und der Inhalt immer nebeneinander angeordnet werden.
Nur CSS, keine Verhaltens- oder Markup-Änderungen.
Reproduktion:
Erstellen Sie eine Umfrage mit Optionen, die @Erwähnungen (@mention pills) enthalten.
In der gekochten Themenansicht kann das Optionszeichen/Kontrollkästchen das Erwähnungspill überlappen.
Korrektur:
Machen Sie die Listenelemente der Umfrageoptionen zu einer Flex-Zeile, sodass das Zeichen/Kontrollkästchen und der Optionsinhalt mit konsistentem Abstand nebeneinander angeordnet werden.
Screenshots zeigen die Überlappung vor der Änderung.
Hallo, ich frage mich nur, wann dies wahrscheinlich in den Hauptzweig übernommen wird? Ich möchte meinen Core nicht mit meiner eigenen Korrektur patchen, um zukünftige Merge-Konflikte mit Updates zu vermeiden.
Ich ziehe es vor, auf den Merge des Commits mit dem Hauptzweig zu warten, frage mich aber auch, wie der weitere Prozess aussehen wird und wann er stattfinden wird.
Vielen Dank im Voraus.
Hmm, mir scheint, dass der von Ihnen verlinkte Pull Request auf Ihren eigenen Fork und nicht auf das Haupt-Repository discourse/discourse abzielt. Gibt es einen bestimmten Grund, warum Sie den Pull Request nicht stattdessen gegen das Haupt-Repository geöffnet haben?
Normalerweise genehmigt und merged das Team dort Pull Requests. Ich bin mir nicht sicher, ob sie sich PRs auf Forks ansehen. Vielleicht hat @zogstrip das bei seiner Überprüfung nicht bemerkt.
ah, okay. Dachte, es würde sowieso zum Haupt-Branch führen oder dort erscheinen, um die Änderungen zu akzeptieren. Aber damit bin ich nicht sehr vertraut und bräuchte hier Unterstützung.
Ist der aktuelle Zustand eine echte Sackgasse, wenn ich bestimmte Schritte nicht befolge, oder kann jemand das trotzdem in den Haupt-Branch mergen?