ピン留めされたトピックを分ける

こんにちは、Teriseさん。

コードを見ましたが、あと一歩です!

フィードバックをいくつかご紹介します。

https://github.com/discourse/discourse/blob/main/app/assets/javascripts/discourse/app/components/topic-list/list.gjs#L126-L136

これは次のようになります。

export default class PinnedTopicBody extends Component {
  get pinnedTopics() {
    return this.outletArgs.topics.filter((t) => t.get("pinned"));
  }
}

そしてテンプレート:

{{#if this.pinnedTopics}}
  <tbody class="pinned-topics">
    {{#each this.pinnedTopics as |topic|}}
      <TopicListItem
        @topic={{topic}}
        @bulkSelectEnabled={{@outletArgs.bulkSelectEnabled}}
        @lastVisitedTopic={{@outletArgs.lastVisitedTopic}}
        @selected={{@outletArgs.selected}}
        @hideCategory={{@outletArgs.hideCategory}}
        @showPosters={{true}}
      />
    {{/each}}
    <tr class="table-break">
      <td colspan="10">&nbsp;</td>
    </tr>
  </tbody>
{{/if}}

これは、一意のIDを持つdivラッパーを生成し、CSSを混乱させます。ただし、要素にアクションをバインドしたい場合は必要です。

ラッパーをカスタマイズするには、tagName と classNames を使用できます。

import { classNames, tagName } from "@ember-decorators/component";

@tagName("tbody")
@classNames("pinned-topics")
export default class PinnedTopicBody extends Component {
  ...
}
{{#if this.pinnedTopics}}
  {{#each this.pinnedTopics as |topic|}}
    ...
  {{/each}}
  <tr class="table-break">
    <td colspan="10">&nbsp;</td>
  </tr>
{{/if}}

(要素に対して何も行わないことがわかっている場合は、テンプレートの tbody を保持し、tagName("") を使用できます)。

すると、次のようになります。

より良い方法があるかどうか、徹底的に調べたわけではありません。
あなたのコードを機能させる手助けをしたかっただけです! :smile: