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

Mac版アプリの文言を追加 #602

Merged
merged 16 commits into from
Dec 22, 2021
Merged

Mac版アプリの文言を追加 #602

merged 16 commits into from
Dec 22, 2021

Conversation

Apple-Yuki
Copy link
Contributor

@Apple-Yuki Apple-Yuki commented Dec 17, 2021

Mac版アプリの文言を追加

内容

Mac版のインストール、起動、アンインストール方法など追記

関連 Issue

(例)
ref #601

その他

関連リンク
VOICEVOX/voicevox_blog#12 (comment)

public/howtouse.md Outdated Show resolved Hide resolved
public/howtouse.md Outdated Show resolved Hide resolved
public/howtouse.md Outdated Show resolved Hide resolved
public/howtouse.md Outdated Show resolved Hide resolved
public/howtouse.md Outdated Show resolved Hide resolved
public/howtouse.md Outdated Show resolved Hide resolved
public/howtouse.md Outdated Show resolved Hide resolved
public/howtouse.md Outdated Show resolved Hide resolved
public/howtouse.md Outdated Show resolved Hide resolved
一部記載の変更を実施しました。
- 起動セッションの説明より**注記**と**重要**を削除
- アンインストールセッションより「**注記**:サードパーティーアプリ「[AppCleaner](https://freemacsoft.net/appcleaner/) 」で関連ファイルごと(隠しフォルダ内のものも含む)完全削除することもできます。」を削除
-ショートカットセッションの「Mac版:暫定」に関して「Mac版では初期状態ではショートカットが設定されていません。今後のアップデートにてご案内いたします。」に文言を変更
ご意見はこちら(#600)(#600 (comment) )から
- アンインストールセッションに「なお、「VOICEVOX」を再インストールすることで作成したプロジェクトファイルは引き続きご利用頂けます。」を追加
gatekeeperに関しての案内を追記
設定ファイルのパス:`/Users/(UserName)/Library/Application\ Support/voicevox-cpu`

設定ファイルのパス:`/Users/(UserName)/Library/Saved\ Application\ State/jp.hiroshiba.voicevox.savedState`

ログファイルのパス:`/Users/(UserName)/Library/Logs/voicevox-cpu `
public/howtouse.md Outdated Show resolved Hide resolved
public/howtouse.md Outdated Show resolved Hide resolved
public/howtouse.md Outdated Show resolved Hide resolved
Copy link
Member

@PickledChair PickledChair left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

もう一息で行けそうな感じになってきたと思います。
気になったところをチェックしました(細かすぎるところもあるかもしれません……)。特に、見出しの # の数には意味がある( html タグの <h1>, <h2>, ... と対応しており、基本的には下の階層に移るごとに一個ずつ増やす)ので、数を調整していただければと思います。よろしくお願いします。

public/howtouse.md Outdated Show resolved Hide resolved
public/howtouse.md Outdated Show resolved Hide resolved
public/howtouse.md Show resolved Hide resolved
public/howtouse.md Outdated Show resolved Hide resolved
public/howtouse.md Outdated Show resolved Hide resolved
public/howtouse.md Outdated Show resolved Hide resolved
public/howtouse.md Show resolved Hide resolved
public/howtouse.md Outdated Show resolved Hide resolved
修正内容:
★ 「してない」は口語的な言い回しなので、「していない」の方に変更(#602 (comment))
★ 「Apple Silicon搭載のMacユーザー向け」という表記から「Apple Silicon搭載のMacをお使いの場合」へ変更。(#602 (comment))
★ Mac版の説明の説明を調整しました。(#602 (comment))
★ その他、タイトルディレクトリを調整しました。
 
Thank you for your nice remarks!
Copy link
Member

@PickledChair PickledChair left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

変更の必要のない部分が変更されてしまっていたので、元に戻していただければと思います(255行目259行目# の数を4つに戻してください)。よろしくお願いします。

(同じタイトルの段落でも、場所によって必要な # の数は異なるのでご注意ください。一括で全置換したりすると期待していない結果になります)

public/howtouse.md Outdated Show resolved Hide resolved
public/howtouse.md Outdated Show resolved Hide resolved
Fixes:
-- Added Mac description to user guide and Corrections were made to the pointed out areas. ( #601 #602)

Thank you for your nice remarks!
Copy link
Member

@PickledChair PickledChair left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

細かい部分の修正・調整が大変だったと思います。お疲れ様でした。対応ありがとうございました。

Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!!

細かいとこ(〜版の統一)だけこちら側で修正させて頂いてからマージしたいと思います!

public/howtouse.md Outdated Show resolved Hide resolved
public/howtouse.md Outdated Show resolved Hide resolved
public/howtouse.md Outdated Show resolved Hide resolved
@Hiroshiba Hiroshiba merged commit b53f878 into VOICEVOX:main Dec 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants