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

@<w>, @<wb>の追加 #1010

Merged
merged 7 commits into from
May 18, 2018
Merged

@<w>, @<wb>の追加 #1010

merged 7 commits into from
May 18, 2018

Conversation

kmuto
Copy link
Owner

@kmuto kmuto commented May 13, 2018

@<w>, @<wb>を追加します。

  • 辞書CSVファイル名に悩んでいます。(現時点ではwords.csv決めうち)
  • 見つからない場合にmissing wordをINFOで出すというようにしていますが、pdfmakerみたいなのだと流れて見えなくなっちゃうので、何か記号を付けるべきなのか悩んでいます。

@kdmsnr
Copy link
Collaborator

kdmsnr commented May 13, 2018

htmlbuilder#inline_icon(id) が warn と存在しないメッセージを原稿に入れる、という感じだったので、これがよいかなあと思いました。

        warn "image not bound: #{id}"
        %Q(<pre>missing image: #{id}</pre>)

@kmuto
Copy link
Owner Author

kmuto commented May 13, 2018

warnと埋込みにするようにしました。
あとはファイル名ですね…どうしようかな。config.ymlで設定?

@kdmsnr
Copy link
Collaborator

kdmsnr commented May 14, 2018

切り替えて使うユースケースを考えられないので、ひとまず決め打ちで良さそうです。

@kmuto
Copy link
Owner Author

kmuto commented May 14, 2018

「カレントのwords.csv」というのを新たに決め打ちで導入する、というところで若干ひっかかりがあるのでした。

「わりとありそうな名前なので、思いもよらず使われてCSV.loadで変なエラーが出る可能性がある?」のほか、「プロジェクトにまたがってどこかに共通で持たせたい」「contentdirのところと同じところに持たせたい」というケースがけっこうあるのではないかなと。

@kmuto
Copy link
Owner Author

kmuto commented May 17, 2018

csvfilewordcsvfileあたりの名前のパラメータで指定するようにしようかなと思っています。デフォルトは指定なしです。

@kmuto kmuto changed the title [WIP] @<w>, @<wb>の追加 @<w>, @<wb>の追加 May 18, 2018
@kmuto
Copy link
Owner Author

kmuto commented May 18, 2018

wordscsvwords_csvのほうがいいだろうか…
ご希望あります? @kdmsnr

@kdmsnr
Copy link
Collaborator

kdmsnr commented May 18, 2018

csvの文字列は実装詳細なので、ないほうがよさそうな気もしますね。たぶんないでしょうけど、jsonで管理したいとか、タブ区切りにしたいとか、あるかもしれません。

@kdmsnr
Copy link
Collaborator

kdmsnr commented May 18, 2018

なのでwords_fileですかねえ

@kmuto
Copy link
Owner Author

kmuto commented May 18, 2018

パラメータ名はwords_fileにして、拡張子.csvのもののみとりあえず処理するようにしてみました。今後拡張子判断で追加したいものが増えてきたらメソッド分けていくということで。

@kmuto
Copy link
Owner Author

kmuto commented May 18, 2018

テストエラーの1つはpygments環境差異によるもので無関係なので、これでマージします

@kmuto kmuto merged commit 5cb50f3 into master May 18, 2018
@kmuto kmuto deleted the inline_w branch May 18, 2018 10:54
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.

2 participants