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

【PgFlow】シーン冒頭の空白が削除される #217

Closed
snext1220 opened this issue Aug 5, 2020 · 10 comments
Closed

【PgFlow】シーン冒頭の空白が削除される #217

snext1220 opened this issue Aug 5, 2020 · 10 comments
Labels
bug バグ報告* Testing テスト中(現在のテスト対象項目)

Comments

@snext1220
Copy link
Owner

snext1220 commented Aug 5, 2020

snext1220/snext-management#18 (comment) 由来のIssueです。

・各シーンの最初の空白(段落下げ)が反映されない?データをリロードするたびに段落下げがなくなっている。

.xmlファイルをPgFlow形式(.json)に変換する際に前後の空白を除去していたことが原因でした。末尾の空白だけを除去するように改めました。GitHubのみ反映しております。

@snext1220 snext1220 added bug バグ報告* Testing テスト中(現在のテスト対象項目) labels Aug 5, 2020
@snext1220 snext1220 added this to the October 2020 Update milestone Aug 5, 2020
@toki-sor1
Copy link
Collaborator

動作確認環境: Win 8.1/84.0.4147.105(64bit)

今度は.xmlファイルの一行目の段落が意図したとおり下がっていることを確認しました!
早速のご対応ありがとうございました&お疲れ様でした!
(返信内容が #215 (comment) と重複するため、同文で失礼いたします~)

@cocotori
Copy link
Collaborator

chromeで確認しました。
文頭の空白は消されず、末尾のみ消されており、
仕様通りになっていました。

ちなみに、末尾の方も、
空白を消してしまう理由ってありますかね?
何か機能上、そのほうがやりやすいならばいいのですが、
特に意図がないなら、製作者さんの意図通りの記述が
できたほうがよいのではと思いました。

例えば、シーン移動のボタンをすごく下にずらしたいときなど。

@snext1220
Copy link
Owner Author

snext1220 commented Aug 30, 2020

ご確認ありがとうございます!

末尾~については、本文の末尾に意図しない空行が入っていた時(経験上、意外と意味のない空行が混じることは多い)、シーン移動ボタンが下にずれたりずれなかったりとばらつくのが目的でした。

仰る通り、製作者のご意図ママとするのがあるべき、というご意見も理解できるのですが…システム的には、「ボタンをすごく下にずらしたい」のような積極的なご要望がないのであれば、現状維持の方が無難ではないかというスタンスではあります。

勿論、ご要望次第で「空白処理は一切しない」はできますので、今一度ご意見いただければ幸いです。

@toki-sor1
Copy link
Collaborator

仰る通り、製作者のご意図ママとするのがあるべき、というご意見も理解できるのですが…システム的には、「ボタンをすごく下にずらしたい」のような積極的なご要望がないのであれば、現状維持の方が無難ではないかというスタンスではあります。

空白処理は一切しないの方が自然で分かりやすいかなと思いました。

現状では、HTMLタグを使うことで意図的に2行以上の空行を作ることも可能なようですが、キーを押すだけで2行以上の空行を作れないので、その辺りの操作はちょっと直感的ではなく、分かりにくい印象です。
(私も最初それに気付かず、やたら回りくどいことをして空行を入れていました…お恥ずかしい…)

おそらく自分が気付いていないだけで、自作シナでも末尾の無駄な空行を自動的に処理していただいているとは思いますし、その機能がなければボタン位置がバラついて「感覚的に気持ちの悪い状態」になりやすくなるかもしれません。

しかし、タグを入れずにそのまま意図したとおりに空行を入れられる(表示できる)方が作者側としても楽かなと思いますし、「あれ? どうして空白が消えちゃうの?」という混乱が起きないのも良いかなぁと。

@Salvadors-cabin
Copy link
Collaborator

Win10+Edge環境で空白の処理が仕様通りである事を確認しました。

末尾の空白を消す処理についてですが、意図せず空白が残ってしまっても人手で消せばよいと思うので、処理しないに一票ですね。

@snext1220
Copy link
Owner Author

末尾の空白について、空白除去処理はなしとのご意見が大勢でしたので、処理を破棄いたしました(GitHub Only)。
ご確認戴けますと幸いです~

@toki-sor1
Copy link
Collaborator

Win 8.1/Chrome 85.0.4183.83(64bit)にて動作確認。
末尾に空白を追加した場合、空白が残っていることを確認しました。

ただ、すみません。いくつか分からないことがあり、お聞きしたいことがあります;

  1. 空白ではなく2行以上の空行については、今までと同じく反映されないのでしょうか?
  2. 空白除去の処理破棄後に今までのscenario.xmlをPg Flowで読み込んだところ、一行目が空行になっていました。
    scene id等の属性の続きから文章を書いていない場合(二行目から文章を書いた場合)は、一行目が空白になってしまうのでしょうか?

#もしかして1の仕様は2の処理に影響しているのでしょうか…?

@snext1220
Copy link
Owner Author

ご質問の件、取り急ぎ。

  1. については、各行にタブなどを加えて戴くと複数行の空白を認識してくれるかと思います(改行のみだと、たぶん束ねられてしまいます)。

  2. については仰る通りかと思います。空白(空行)処理がなくなったため、そのまま改行が出力される形となります。

@toki-sor1
Copy link
Collaborator

お忙しいところ、早速のお返事ありがとうございます!
1と2の仕様&解決方法について承知しました!
2行以上の改行はタグを使用し、文章の冒頭はテキストエディタ等での編集時もPg Flowの仕様に合わせて一行目から書き始めるようにしますね。

@snext1220
Copy link
Owner Author

ご確認ありがとうございます~

文章の冒頭>PgFlowネイティブで進めている場合はほぼ問題にはならないかと思いますが、環境混在の場合はそのようにお願いいたします…!

# 今回、空白については一旦修正しておりますが、システム的にはそちこちで空白処理を行っております(属性はほとんどのものを処理しています)。機能外したことで問題に感じる面も出てくるかと思いますが、改めて復活させることは可能なので、必要に応じてお声がけください。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug バグ報告* Testing テスト中(現在のテスト対象項目)
Projects
None yet
Development

No branches or pull requests

4 participants