記事のソートをタイトル順にする #123
記事のソートをタイトル順にする #123
Conversation
|
@kiitosu ご提案ありがとうございます！また、ご連絡が遅くなりすみません。順次確認しますのでお待ち下さい 🙏 |
There was a problem hiding this comment.
Pull Request Overview
This PR updates the article sorting behavior to allow sorting by title instead of the default file path order.
- Introduces a new configuration (zenn-preview.sortArticle) that enables title-based sorting.
- Modifies the PreviewTreeItem.sortTreeItems method to remove emojis from labels before sorting.
- Updates package.json to define the new configuration option with enum values.
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/treeview/previewTreeItem.ts | Updated sorting logic to support title-based sorting. |
| package.json | Added configuration definition for sortArticle option. |
| const aCleanLabel = getCleanLabel(a); | ||
| const bCleanLabel = getCleanLabel(b); | ||
|
|
||
| if (aCleanLabel || bCleanLabel) { |
There was a problem hiding this comment.
[nitpick] Consider using an explicit non-empty string check (e.g., aCleanLabel !== '' || bCleanLabel !== '') to clarify the intended condition and improve code readability for future maintainers.
| if (aCleanLabel || bCleanLabel) { | |
| if (aCleanLabel !== '' || bCleanLabel !== '') { |
Copilot uses AI. Check for mistakes.
|
@kiitosu ご対応いただきありがとうございます！ 内容自体は良さそうですが、package.json のインデントが2から4に変わっているため差分が分かりにくくなっています。お手数ですがインデント2に修正をお願いいたします 🙇 |
|
すみません気づいてませんでした！ |
|
@kiitosu 忘れていないです、すいません、もう少々お待ち下さい🙇 |
Co-authored-by: Yusuke Wada <[email protected]>
|
pakcage.jsonの最後は改行で終わったほうが良いということですね。取り入れました！ |
…t_with_titles 記事のソートをタイトル順にする
…t_with_titles 記事のソートをタイトル順にする
3 participants
📑 Summary
articleの表示順をタイトル順でソートするようにしました。
以下に作業内容を整理しました。
Resolves #
📋 Tasks
プルリクエストを作成いただく際、お手数ですが以下の内容についてご確認をお願いします。
canaryブランチに対するプルリクエストであるより詳しい内容は Pull Request Policy を参照してください。