-
Notifications
You must be signed in to change notification settings - Fork 65
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
feat: logo position #62
Conversation
实验性功能会导致构建上出现一定的问题(神秘的二次文档构建 xelatex: fwrite: Broken pipe),本地暂时无法复现问题,等实验性功能稳定后本 PR 再从 draft 转换出去。 |
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.
Rest LGTM. l3build
的 typesetopts
是不是也需要加上 --shell-escape
?
src/build.lua
Outdated
@@ -16,7 +16,7 @@ end | |||
|
|||
typesetfiles = {"sjtubeamerdevguide.tex","sjtubeamer.tex"} | |||
-- typesetfiles = {"sjtubeamer.tex"} | |||
-- typesetruns = 1 -- for debug. Some reference may not be linked. | |||
typesetruns = 1 -- for debug. Some reference may not be linked. |
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.
看起来这一行不应该更改?
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.
去掉就 CI fail,第二次编译用户手册会导致写冲突,可能是某个宏包构建导致的 xelatex 冲突,我有时间试着复现这个问题...
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.
这两天 l3build
更新了 CI 就过了可还行
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.
我们用的镜像一个月更新一次,估计要等到下个月初了(
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.
没事已经过了
@@ -3,8 +3,8 @@ | |||
module = "sjtubeamer" | |||
|
|||
sourcefiledir = "source" | |||
sourcefiles = {"*.ins","*.dtx","vi/"} | |||
installfiles = {"*.sty","vi/"} | |||
sourcefiles = {"*.ins","*.dtx","vi/","latexmkrc"} |
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.
l3build 可以直接使用 latexmk 作为编译器吗?CI / 用户的编译命令是不是要换掉(
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.
这个地方还是实验性的,还没有想明白这个东西怎么提示用户去打开缓存开关,这个只是先放着(
其实我觉得 logo 位置可以用选项指定,比如 bottomleft,topright 之类的( |
我觉得可以试试加开关,位置设定:
|
感觉有 bottomright 和 topright 就够了( |
确实 但topleft是我代码写的烂导致的( |
这个 PR 已经足够长了,logo宏已经设定,实验性的缓存会在另一个 PR 稳定下来。 之后会合并到主分支。 |
徽标位置
由于底部的工具栏重写(为了不与脚注碰撞),导致 logo 需要换一个地方显示。
maxplus
会连带logo为右上角显示。max
会连带logo为右下角显示。min
会连带logo为右上角显示。my
会连带logo为右下角显示。也可以分别使用
bottomright
,topright
覆盖显示选项。缓存 tikz 图像(实验性)
如果打开了
--shell-escape
编译开关,将会对 tikz 图像进行缓存。对内部一些不必缓存的元素进行了保护。由于处于实验性阶段,默认将会关闭。