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

@<b>の展開として\textbfそのままではなく抽象名\reviewboldとする #792

Closed
wants to merge 2 commits into from

Conversation

kmuto
Copy link
Owner

@kmuto kmuto commented Jun 25, 2017

ref #768
実体としては\textbfに展開されるので互換性は問題ないはず。

結局これだけでは本文/見出し/コード内などでの書体は指定するのは難しいので、ユーザがカスタマイズするにはクラスファイルレベルでの用意が必要になりそうではある…。

layout.tex.erbをカスタマイズして利用していたユーザは、reviewboldが未定義となり、コンパイルが動かなくなる影響が生じる。互換性を保つには、

\newcommand{\reviewbold}[1]{\textbf{#1}}

の追加が必要。

@kmuto kmuto mentioned this pull request Jun 29, 2017
1 task
@kauplan
Copy link

kauplan commented Jan 10, 2018

このpull requestをmasterブランチにマージする予定はありますか?
個人的な希望としては、ぜひマージしてほしいです。

Re:VIEW 2.3で試したところ、@<b>{...}\textbf{...}になりました。

@kmuto kmuto mentioned this pull request Feb 6, 2018
19 tasks
@kmuto
Copy link
Owner Author

kmuto commented Feb 28, 2018

#950 のブランチで含めているけれども、追加の必要な対象はわりと少なめだった。

\newcommand{\reviewit}[1]{\textit{#1}}
\newcommand{\reviewbold}[1]{\textbf{#1}}
\newcommand{\reviewcode}[1]{\texttt{#1}}
\newcommand{\reviewtt}[1]{\texttt{#1}}

@takahashim
Copy link
Collaborator

takahashim commented Jun 17, 2018

遅ればせながらこの辺は3.0で取り込まれることになりそうです(masterには反映済み)

@takahashim
Copy link
Collaborator

@kmuto これは閉じて良いですか?

@kmuto
Copy link
Owner Author

kmuto commented Jun 18, 2018

とじましょう

@kmuto kmuto closed this Jun 18, 2018
@takahashim takahashim deleted the reviewboldfont branch June 19, 2018 10:36
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