UI微調整，metadata/READMEの修正,アイコンの追加 #32
Conversation
Walkthrough
Changes
TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
Outside diff range and nitpick comments (1)
dem_to_csmap.ui (1)
Line range hint
27-86: 新しいウィジェットの最小サイズが0に設定されていますが、これは視認性にとって理想的ではありません。より良いUI体験のために、適切な最小サイズを設定することを提案します。- <width>0</width> - <height>0</height> + <width>100</width> + <height>30</height>
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files ignored due to path filters (3)
imgs/howtouse.pngis excluded by!**/*.png,!**/*.pngimgs/icon.pngis excluded by!**/*.png,!**/*.pngimgs/usage.pngis excluded by!**/*.png,!**/*.png
Files selected for processing (4)
- README.md (1 hunks)
- dem_to_csmap.py (1 hunks)
- dem_to_csmap.ui (4 hunks)
- metadata.txt (1 hunks)
Files skipped from review due to trivial changes (1)
- dem_to_csmap.py
Additional context used
LanguageTool
metadata.txt
[duplication] ~8-~8: Possible typo: you repeated a word (ENGLISH_WORD_REPEAT_RULE)
Context: ...GISプラグインです。 version=0.0.1 #Plugin main icon icon=imgs/icon.png author=MIERUNE Inc. emai...
Markdownlint
README.md
8-8: Expected: h2; Actual: h3 (MD001, heading-increment)
Heading levels should only increment by one level at a time
9-9: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
12-12: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
13-13: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
14-14: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
15-15: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
9-9: Expected: 0 or 2; Actual: 1 (MD009, no-trailing-spaces)
Trailing spaces
8-8: Expected: 1; Actual: 0; Below (MD022, blanks-around-headings)
Headings should be surrounded by blank lines
11-11: Expected: 1; Actual: 0; Below (MD022, blanks-around-headings)
Headings should be surrounded by blank lines
9-9: null (MD032, blanks-around-lists)
Lists should be surrounded by blank lines
12-12: null (MD032, blanks-around-lists)
Lists should be surrounded by blank lines
6-6: null (MD039, no-space-in-links)
Spaces inside link text
3-3: null (MD045, no-alt-text)
Images should have alternate text (alt text)
17-17: null (MD045, no-alt-text)
Images should have alternate text (alt text)
Additional comments not posted (9)
metadata.txt (6)
3-3: QGISの最小バージョンを3.34に更新しています。これにより、プラグインが最新のQGIS LTRバージョンと互換性を持つようになります。
4-4: プラグインについての説明が日本語で追加され、地域ユーザーに適した情報提供が行われています。
5-5: プラグインの機能を明確に説明しているため、この変更を承認します。
6-6: バージョンが1.0から0.0.1にリセットされています。リリースノートやドキュメントにこの変更の理由を説明する注記を追加することをお勧めします。
13-15: ホームページ、トラッカー、およびリポジトリのリンクが更新され、ユーザーや開発者が情報を見つけやすくなっています。
16-16: タグが更新され、プラグインの検索や分類が容易になります。README.md (1)
5-6: プラグインの説明が明確であり、metadata.txtファイルの変更と一致しています。Tools
Markdownlint
6-6: null (MD039, no-space-in-links)
Spaces inside link textdem_to_csmap.ui (2)
18-18: QHBoxLayoutからQGridLayoutへの変更は、UI要素の柔軟かつ整理された配置を可能にします。この変更を承認します。
Line range hint
20-71: 入力ファイルと出力ファイルの選択のために追加された新しいQgsFileWidgetウィジェットは、より直感的なファイル選択プロセスを提供します。この変更を承認します。
|
一旦，インストールの部分はカットしました。 |
There was a problem hiding this comment.
Actionable comments posted: 1
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- README.md (1 hunks)
Additional context used
Markdownlint
README.md
9-9: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
10-10: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
11-11: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
12-12: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
8-8: Expected: 1; Actual: 0; Below (MD022, blanks-around-headings)
Headings should be surrounded by blank lines
9-9: null (MD032, blanks-around-lists)
Lists should be surrounded by blank lines
6-6: null (MD039, no-space-in-links)
Spaces inside link text
Additional comments not posted (3)
README.md (3)
3-3: 画像に代替テキストが追加されています。これにより、アクセシビリティが向上しています。
5-6: プラグインの説明が更新され、新しい機能とQGIS LTRの最新版との互換性が反映されています。これにより、ユーザーがプラグインの機能をより理解しやすくなっています。Tools
Markdownlint
6-6: null (MD039, no-space-in-links)
Spaces inside link text
14-14: 使用例の画像に代替テキストが追加されています。これにより、アクセシビリティが向上しています。
There was a problem hiding this comment.
Actionable comments posted: 2
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- README.md (1 hunks)
Additional context used
Markdownlint
README.md
10-10: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
11-11: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
12-12: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
13-13: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
6-6: null (MD039, no-space-in-links)
Spaces inside link text
Additional comments not posted (2)
README.md (2)
3-3: 画像に代替テキストが含まれています。これによりアクセシビリティが向上しています。
15-15: 使用例の画像に代替テキストが含まれています。これによりアクセシビリティが向上しています。
|
出典が斜体になるように修正します！ |
There was a problem hiding this comment.
Actionable comments posted: 2
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (2)
- README.md (1 hunks)
- metadata.txt (1 hunks)
Additional context used
LanguageTool
metadata.txt
[duplication] ~8-~8: Possible typo: you repeated a word (ENGLISH_WORD_REPEAT_RULE)
Context: ...GISプラグインです。 version=0.0.1 #Plugin main icon icon=imgs/icon.png author=MIERUNE Inc. emai...
Markdownlint
README.md
12-12: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
15-15: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
16-16: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
19-19: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
20-20: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
21-21: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
22-22: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
26-26: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
27-27: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
28-28: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
29-29: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
9-9: Expected: 1; Actual: 2 (MD012, no-multiple-blanks)
Multiple consecutive blank lines
14-14: Expected: 1; Actual: 0; Below (MD022, blanks-around-headings)
Headings should be surrounded by blank lines
14-14: null (MD023, heading-start-left)
Headings must start at the beginning of the line
15-15: null (MD032, blanks-around-lists)
Lists should be surrounded by blank lines
19-19: null (MD032, blanks-around-lists)
Lists should be surrounded by blank lines
7-7: null (MD039, no-space-in-links)
Spaces inside link text
16-16: null (MD039, no-space-in-links)
Spaces inside link text
Additional comments not posted (9)
metadata.txt (5)
3-3: QGISの最小バージョンが更新されました。この変更により、プラグインの互換性が向上しています。
4-4: プラグインの説明が更新され、最新のQGIS LTRバージョンでの動作が明確にされています。ユーザーにとって有益な情報です。
5-5: プラグインの主な機能についての説明が更新されました。より具体的で理解しやすくなっています。
13-16: 新しいリンクとタグが追加されました。これにより、プラグインの可視性とアクセスが向上します。
6-6: バージョン番号が下がっていることに注意してください。これは通常のバージョニングの慣習に反しています。意図的なものか、誤りかを確認してください。README.md (4)
3-3: アイコンに代替テキストが追加されています。これによりアクセシビリティが向上しました。
6-7: プラグインの説明が更新され、リンクにスペースが含まれています。これは以前のレビューで指摘された内容ですが、スペースが意図的であることが確認されています。Tools
Markdownlint
7-7: null (MD039, no-space-in-links)
Spaces inside link text
31-31: 使用例の画像に代替テキストが追加されています。これによりアクセシビリティが向上しています。
24-29: 使用方法が追加されていますが、リストのインデントが一貫性がありません。Markdownのフォーマットを整えることをお勧めします。- - アイコンをクリックして，プラグインを起動します。 + - アイコンをクリックして，プラグインを起動します。 - - 入力ファイル：DEM（GDALでサポートされている形式）を選択します。 + - 入力ファイル：DEM（GDALでサポートされている形式）を選択します。 - - 出力ファイル：出力されるCS立体図の名前を指定します。 + - 出力ファイル：出力されるCS立体図の名前を指定します。 - - `高度なオプション` 及び `処理終了後，自動でウィンドウを閉じる` の使用は任意です。 + - `高度なオプション` 及び `処理終了後，自動でウィンドウを閉じる` の使用は任意です。Likely invalid or redundant comment.
Tools
Markdownlint
26-26: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
27-27: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
28-28: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
29-29: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
There was a problem hiding this comment.
Actionable comments posted: 3
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- README.md (1 hunks)
Additional context used
Markdownlint
README.md
11-11: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
14-14: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
15-15: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
18-18: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
19-19: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
20-20: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
21-21: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
25-25: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
26-26: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
27-27: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
28-28: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
13-13: Expected: 1; Actual: 0; Below (MD022, blanks-around-headings)
Headings should be surrounded by blank lines
13-13: null (MD023, heading-start-left)
Headings must start at the beginning of the line
14-14: null (MD032, blanks-around-lists)
Lists should be surrounded by blank lines
18-18: null (MD032, blanks-around-lists)
Lists should be surrounded by blank lines
7-7: null (MD039, no-space-in-links)
Spaces inside link text
15-15: null (MD039, no-space-in-links)
Spaces inside link text
Additional comments not posted (3)
README.md (3)
3-3: 代替テキストが正しく追加されています。アクセシビリティが向上しています。
4-4: 出典が斜体になっており、ユーザーがより視覚的に区別しやすいです。この変更は文書の読みやすさを向上させます。
9-12: インストール方法が明確に記述されており、ユーザーがプラグインを簡単に見つけてインストールできるようになっています。Tools
Markdownlint
11-11: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
| ### Windowsをお使いの方へ | ||
| - Windows版のQGISでは，プラグインの読み込みエラーが発生する場合があります。<br> | ||
| - [csmap-py ](https://github.com/MIERUNE/csmap-py)によるDEMの変換処理は rasterio と numpy により実行されているので，Windows版のQGISでは rasterio を手動でインストールする必要があります。以下の手順でインストールしてください。<br> | ||
|
|
||
| rasterio インストール手順： | ||
| - Pythonコンソールを起動します。 | ||
| - `!pip install rasterio` を実行します。赤文字で `WARNING` が表示されても，最下行に `Successfully installed ~` と表示されれば問題ありません。 | ||
| - QGISを再起動して，もう一度Pythonコンソールを起動します。 | ||
| - `import rasterio` を実行すると，本プラグインのアイコンが表示されます。 |
There was a problem hiding this comment.
Windowsユーザー向けの特別なインストール指示が追加されていますが、見出しのレベルとリストのインデントに一貫性がありません。Markdownの標準に従って修正することをお勧めします。
- ### Windowsをお使いの方へ
+ ## Windowsをお使いの方へ
- - Windows版のQGISでは，プラグインの読み込みエラーが発生する場合があります。<br>
+ - Windows版のQGISでは、プラグインの読み込みエラーが発生する場合があります。<br>
- - [csmap-py ](https://github.com/MIERUNE/csmap-py)によるDEMの変換処理は rasterio と numpy により実行されているので，Windows版のQGISでは rasterio を手動でインストールする必要があります。以下の手順でインストールしてください。<br>
+ - [csmap-py](https://github.com/MIERUNE/csmap-py)によるDEMの変換処理はrasterioとnumpyにより実行されているので、Windows版のQGISではrasterioを手動でインストールする必要があります。以下の手順でインストールしてください。<br>Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| ### Windowsをお使いの方へ | |
| - Windows版のQGISでは，プラグインの読み込みエラーが発生する場合があります。<br> | |
| - [csmap-py ](https://github.com/MIERUNE/csmap-py)によるDEMの変換処理は rasterio と numpy により実行されているので，Windows版のQGISでは rasterio を手動でインストールする必要があります。以下の手順でインストールしてください。<br> | |
| rasterio インストール手順： | |
| - Pythonコンソールを起動します。 | |
| - `!pip install rasterio` を実行します。赤文字で `WARNING` が表示されても，最下行に `Successfully installed ~` と表示されれば問題ありません。 | |
| - QGISを再起動して，もう一度Pythonコンソールを起動します。 | |
| - `import rasterio` を実行すると，本プラグインのアイコンが表示されます。 | |
| ## Windowsをお使いの方へ | |
| - Windows版のQGISでは、プラグインの読み込みエラーが発生する場合があります。<br> | |
| - [csmap-py](https://github.com/MIERUNE/csmap-py)によるDEMの変換処理はrasterioとnumpyにより実行されているので、Windows版のQGISではrasterioを手動でインストールする必要があります。以下の手順でインストールしてください。<br> | |
| rasterio インストール手順： | |
| - Pythonコンソールを起動します。 | |
| - `!pip install rasterio` を実行します。赤文字で `WARNING` が表示されても，最下行に `Successfully installed ~` と表示されれば問題ありません。 | |
| - QGISを再起動して，もう一度Pythonコンソールを起動します。 | |
| - `import rasterio` を実行すると，本プラグインのアイコンが表示されます。 |
Tools
Markdownlint
14-14: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
15-15: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
18-18: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
19-19: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
20-20: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
21-21: Expected: 0; Actual: 1 (MD007, ul-indent)
Unordered list indentation
13-13: Expected: 1; Actual: 0; Below (MD022, blanks-around-headings)
Headings should be surrounded by blank lines
13-13: null (MD023, heading-start-left)
Headings must start at the beginning of the line
14-14: null (MD032, blanks-around-lists)
Lists should be surrounded by blank lines
18-18: null (MD032, blanks-around-lists)
Lists should be surrounded by blank lines
15-15: null (MD039, no-space-in-links)
Spaces inside link text
There was a problem hiding this comment.
@KeiTa4446, 承知しました。他にサポートが必要な場合は、お気軽にお知らせください。
|
LGTM！ |
Close #25
Close #31
Description（変更内容）
q: @Kanahiro
インストール方法について，後ほど確認させていただきたいです。
...
Manual Testing（手動テスト）
Summary by CodeRabbit
新機能
README.mdにインストール手順を追加バグ修正
スタイル
QHBoxLayoutからQGridLayoutに変更し、ウィジェットの位置とプロパティを調整ドキュメント