Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ヘルプのリンクを修正 #1338

Merged
merged 1 commit into from
Jul 14, 2020
Merged

ヘルプのリンクを修正 #1338

merged 1 commit into from
Jul 14, 2020

Conversation

berryzplus
Copy link
Contributor

PR の目的

ヘルプのリンクが誤っているのを修正します。

カテゴリ

  • ドキュメント修正

PR の背景

#1336

以下、OSDNより転載します。
https://osdn.net/projects/sakura-editor/forums/34071/42606/
ヘルプファイルに不具合が数点あります。
Version: いつのバージョンからかはわかりませんが、V.2.4.1(Beta)以前、V2.2.0 などで確認できるものもあります
不具合(?)事項:
1, 検索・置換のダイアログに関するヘルプの最下部の[上検索][下検索]のリンク先がおかしい
対象:ヘルプ目次→コマンド概要→「検索(S)」メニュー→検索... または 置換... のページ
最下部にある [上検索]リンク が 次を検索 のページに、
[下検索]リンク が 前を検索 のページにそれぞれ逆に繋がっている。

PR のメリット

PR のデメリット (トレードオフとかあれば)

仕様・動作説明

テスト内容

テスト1

手順

  1. HLP000059.htmlを開く
  2. 下端の上検索をクリックして、前を検索に遷移することを確認し、ブラウザバックする。
  3. 下端の下検索をクリックして、次を検索に遷移することを確認し、ブラウザバックする。
  4. HLP000062.htmlを開く
  5. 下端の上検索をクリックして、前を検索に遷移することを確認し、ブラウザバックする。
  6. 下端の下検索をクリックして、次を検索に遷移することを確認し、ブラウザバックする。

PR の影響範囲

関連 issue, PR

#1336 OSDN転載: ヘルプファイルの不具合

参考資料

@AppVeyorBot
Copy link

@KENCHjp KENCHjp added the document ドキュメント label Jul 9, 2020
@m-tmatma
Copy link
Member

質問

上検索下検索 はどういう意味ですか?

@berryzplus
Copy link
Contributor Author

berryzplus commented Jul 13, 2020

上検索下検索 はどういう意味ですか?

機能的にはそれぞれ、前を検索と次を検索のことです。
検索ダイアログ(置換ダイアログ)のボタンキャプションが上検索・下検索だからこういう記述になっています。

上検索(前を検索)は、現在のキャレット位置を基準にして前方の一致を検索します。同一行内に一致がなく、前方に一致がある場合にはキャレットが上方向に移動する結果となるからと思われます。

下検索(次を検索)は、現在のキャレット位置を基準にして後方の一致を検索します。レイアウト的に下方向への移動となることが多いと思います。内部コマンド名は SearchNext っぽい感じになっています。

検索方向の Previous/Next を上下で捉える考え方は結構一般的だと思います。たとえば VsCode の検索ボックスについてるボタンアイコンもSearchPreviuosが上矢印でSearchNextが下矢印になっています。

@m-tmatma
Copy link
Member

上検索下検索 はどういう意味ですか?

機能的にはそれぞれ、前を検索と次を検索のことです。
検索ダイアログ(置換ダイアログ)のボタンキャプションが上検索・下検索だからこういう記述になっています。

この PR とは別件ですが、

用語を統一したほうがいいかもしません。

次を検索下検索
前を検索上検索 が混在していてわかりにくい。

@berryzplus
Copy link
Contributor Author

レビューありがとうございます。とりあえずマージしてしまいます。

@berryzplus berryzplus merged commit e746129 into master Jul 14, 2020
@berryzplus berryzplus deleted the feature/fix_help_links branch July 14, 2020 12:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
document ドキュメント
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants