-
Notifications
You must be signed in to change notification settings - Fork 163
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
HTML Help 中のリンクを GitHub のものに変える #383
HTML Help 中のリンクを GitHub のものに変える #383
Conversation
下記のソースコード中にリンクがある事に気付きました。 sakura/sakura_core/util/shell.cpp Line 610 in 4c58aeb
https://sakura-editor.github.io/help/ ヘルプIDは、https://github.com/sakura-editor/sakura/blob/master/sakura_core/sakura.hh sakura-editor/sakura-editor.github.io#5 (comment) |
この PR とは別件なので、別の issue で議論したほうがいいと思います |
レビューお願いします |
どうしたいのかがよく分っていないです。 除外理由に挙げられている内容で「履歴だから」というのは、ニュアンスというか、分からなくないんですが、「過去そうだった」という理由で現在は消滅したリンクを残す意味があるのかないのか、です。 たとえば #383 を適用した状態で sakura っぽい Web ページアドレスを検索した結果が以下になります。
旧サイト sourceforge のアドレスが結構残ってます。 |
http://sakura.qp.land.to に関しては代替のコンテンツが github にないため対象にしていません。 |
少し分かりにくいですが、 ビルド成果物のダウンロード(バイナリ、インストーラなど) から辿れると思うのでその記述を無くしても良いのではないかと。 |
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTMです。
追加説明のあった除外条件を元に変更が漏れなく行われているかチェックしました。
問題なく変更できているという判断です。
修正済み(5箇所)
C:\gitroot\sakura\help\plugin\Text\index.html(20,18) [SJIS]: <center><a href="https://sakura-editor.github.io/" target=_blank>https://sakura-editor.github.io/</a></center>
C:\gitroot\sakura\help\sakura\_RESOURCE\HLP000001.html(20,18) [SJIS]: <center><a href="https://sakura-editor.github.io/" target=_blank>https://sakura-editor.github.io/</a></center>
C:\gitroot\sakura\help\sakura\_RESOURCE\HLP000104.html(30,4) [SJIS]: https://sakura-editor.github.io/<br />
C:\gitroot\sakura\help\sakura\_RESOURCE\HLP000112.html(17,34) [SJIS]: <a href="https://sakura-editor.github.io/" target=_blank>https://sakura-editor.github.io/</a><br>
C:\gitroot\sakura\help\sakura\_RESOURCE\HLP_UR000.html(18,34) [SJIS]: <a href="https://sakura-editor.github.io/" target=_blank>https://sakura-editor.github.io/</a><br>
除外条件を満たすもの(8箇所)
C:\gitroot\sakura\help\plugin\Text\index.html(21,18) [SJIS]: <center><a href="http://sakura.qp.land.to/?Plugin" target=_blank>http://sakura.qp.land.to/?Plugin</a></center>
C:\gitroot\sakura\help\plugin\Text\overview.html(23,33) [SJIS]: <strong>プラグイン</strong>は<a href="http://sakura.qp.land.to/?Plugin%2F%C5%EA%B9%C6">SakuraEditorWiki</a>で公開されています。
C:\gitroot\sakura\help\sakura\_RESOURCE\HLP000003.html(24,12) [SJIS]: <a href = "http://sourceforge.net/p/sakura-editor/wiki/64bit/" target = "_blank">開発者Wiki/64bit</a>
C:\gitroot\sakura\help\sakura\_RESOURCE\HLP000010.html(36,16) [SJIS]: <li><a href = "http://sakura.qp.land.to/?Customize%2FToolIconMaker" target = "_blank">http://sakura.qp.land.to/?Customize%2FToolIconMaker</a></li>
C:\gitroot\sakura\help\sakura\_RESOURCE\HLP000110.html(41,22) [SJIS]: プラグインに関する情報:<a href="http://sakura.qp.land.to/?Plugin">http://sakura.qp.land.to/?Plugin</a><br />
C:\gitroot\sakura\help\sakura\_RESOURCE\HLP_HISTORY.html(15,26) [SJIS]: helpのコンパイル前の最新は <a href="http://sourceforge.net/p/sakura-editor/code/" target="_blank" rel="nofollow">http://sourceforge.net/p/sakura-editor/code/</a> help/trunk2より閲覧できます。<br>
C:\gitroot\sakura\help\sakura\_RESOURCE\HLP_HISTORY.html(49,21) [SJIS]: ・テキストエディタ非公式版の紹介ページ(http://members.at.infoseek.co.jp/sakura_editor/intro.html)の内容を機能概要に追加<br>
C:\gitroot\sakura\help\sakura\_RESOURCE\HLP_HISTORY.html(50,18) [SJIS]: ・ダイレクトタグジャンプの使い方(http://members.at.infoseek.co.jp/sakura_editor/doc/direct_tagjump.html)のページを追加<br>
除外条件を満たさない疑いがあるもの(1箇所)
C:\gitroot\sakura\help\sakura\_RESOURCE\HLP_UR015.html(275,16) [SJIS]: <li>オンラインヘルプは http://sakura-editor.sourceforge.net/htmlhelp2/ を参照する (svn:1943 unicode:1621 ryoji)</li>
→ 変更履歴(リリースノート)の一部なので、除外条件を満たしていると判断して良さそう。
…k-in-help HTML Help 中のリンクを GitHub のものに変える
#380: HTML Help 中のリンクを GitHub のものに変える
sakura/help/sakura/_RESOURCE/HLP_HISTORY.html
Lines 13 to 15 in 4c58aeb