Hinzufügen des topic-list-after-title-Outlet zum mobilen Template

fzngagan · 17. Juni 2020 um 03:37

Im Events-Plugin überschreiben wir derzeit die Mobile-Vorlage. Ich bin sehr dankbar für die kürzlich hinzugefügten Outlet-Erweiterungen in der Mobile-Vorlage, die die Sache bereits deutlich verbessern. Ich frage mich nur, ob wir auch auf Mobile den Outlet topic-list-after-title einfügen können, damit wir unsere Überschreibung einfach entfernen können? Dieser Outlet ist übrigens bereits auf dem Desktop verfügbar.

Falls die Anfrage sinnvoll ist, erstelle ich gerne einen PR dafür.

david · 17. Juni 2020 um 08:40

Klar, das klingt vernünftig – mir fällt kein Grund ein, warum es auf Mobilgeräten fehlen sollte.

fzngagan · 17. Juni 2020 um 08:44

github.com/discourse/discourse

app/assets/javascripts/discourse/app/templates/mobile/list/topic-list-item.hbr

45eb97c20


      
            This causes the topic-post-badge to be considered the same word as "text"
            at the end of the link, preventing it from line wrapping onto its own line.
          --}}
          {{~raw-plugin-outlet name="topic-list-before-link"}}
          <div class='main-link'>
            {{~raw-plugin-outlet name="topic-list-before-status"}}
            {{~raw "topic-status" topic=topic~}}
            {{~topic-link topic class="raw-link raw-topic-link"}}
            {{~#if topic.featured_link~}}
            {{~topic-featured-link topic~}}
            {{~/if~}}
            {{~#if topic.unseen~}}
              <span class="topic-post-badges">&nbsp;<span class="badge-notification new-topic"></span></span>
            {{~/if~}}
            {{~#if expandPinned~}}
            {{~raw "list/topic-excerpt" topic=topic~}}
            {{~/if~}}
          </div>
          <div class='pull-right'>
            {{raw "list/post-count-or-badges" topic=topic postBadgesEnabled=showTopicPostBadges}}
          </div>

Ist dies der richtige Ort, um es hinzuzufügen? Nur zur Bestätigung.

david · 17. Juni 2020 um 08:46

Das scheint gut mit der Desktop-Vorlage übereinzustimmen

fzngagan · 17. Juni 2020 um 08:56

Hier ist der PR dafür. DEV: Added topic-list-after-title outlet in the mobile template by fzngagan · Pull Request #10059 · discourse/discourse · GitHub

fzngagan · 17. Juni 2020 um 09:30

@david danke für den Merge.

Thema		Antworten	Aufrufe
Missing "topic-list-after-columns" raw plugin outlet Development	2	422	28. Juni 2023
Unable to replace topic-list-item.raw.hbs / topic-list-item.hbr on mobile Development	10	1476	13. Februar 2020
Customizing the topic list Developer Guides	0	766	4. Februar 2025
Locked, solved, and new topic adornments are misaligned on longer titles on mobile UX	5	128	6. Dezember 2025
Event date is no longer styled in topic list (mobile view) Support calendar-and-event	1	391	11. Januar 2024

Hinzufügen des topic-list-after-title-Outlet zum mobilen Template

Verwandte Themen