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

做了一大堆修改... 你看看要不要合入你的分支 (可以去掉我的个人信息) #21

Closed
wants to merge 3 commits into from

Conversation

qiancy98
Copy link

修改内容:

  • \ifdef -> \if
  • \@firstname, \@lastname -> \@name
  • 照片重绘, 放到右侧

- \ifdef -> \if
- firstname, lastname -> name
- 照片重绘, 放到右侧
@qiancy98
Copy link
Author

另, 我也是上交学生, 可否认识一下www 谢谢~

@liweitianux
Copy link
Owner

liweitianux commented Sep 24, 2024

你好,我刚刚简单看了一下,有些修改不错,后面有空了再仔细看看并合并改进。谢谢!

* `\ifdef` -> `\if`

想问下这个修改有什么优势?从改动内容看,我感觉修改后的版本并没有明显优化。

* `\@firstname`, `\@lastname` -> `\@name`

个人还是偏好拆分姓氏和名字,然后可以根据中英文的不同习惯做不同的显示。

* 照片重绘, 放到右侧

照片放在左侧的主要原因是整体更加对齐,也更符合一般头像在文件左边的惯例。
当然,我觉得可以增加一个选项来让照片显示在右侧。

@liweitianux
Copy link
Owner

另, 我也是上交学生, 可否认识一下www 谢谢~

可以啊,我给你发个邮件。

@qiancy98
Copy link
Author

  • \ifdef -> \if
    感觉本质上这里只需要一个bool, 因此我感觉, 直接定义个bool值语义更清晰?

  • \@firstname, \@lastname -> \@name
    我同样设置了根据中英文的不同习惯做不同的显示, 只不过我把这个逻辑移动到了\@name的定义里. 这样不用每次用到name时都手动设置一遍顺序 (何况每次的顺序都一样, 且只和环境语言, 以及是否在元数据中有关: 已用\texorpdfstring做了区分).

    另, 我不是很理解为什么中文环境下姓名之间要加个空格, 所以我移除了. 如果需要强调姓和名之间的区别, 也许可以把名设为楷体? (瞎猜的)

  • 我觉得可以增加一个选项来让照片显示在右侧。
    OK, 我来加个选项

@liweitianux
Copy link
Owner

  • \ifdef -> \if
    感觉本质上这里只需要一个bool, 因此我感觉, 直接定义个bool值语义更清晰?

ifdef 里的 if 就说明这里是一个条件判断,就已经是 bool 型数据了。

* `\@firstname`, `\@lastname` -> `\@name`
  我同样设置了根据中英文的不同习惯做不同的显示, 只不过我把这个逻辑移动到了`\@name`的定义里. 这样不用每次用到name时都手动设置一遍顺序 (何况每次的顺序都一样, 且只和环境语言, 以及是否在元数据中有关: 已用`\texorpdfstring`做了区分).
  另, 我不是很理解为什么中文环境下姓名之间要加个空格, 所以我移除了. 如果需要强调姓和名之间的区别, 也许可以把名设为楷体? (瞎猜的)

中文的姓名之间确实不需要加上空格;应该是我当初参考英文模板做的时候觉得看着更清楚保留了空格。这个后面我调整一下。

使用字体来区分姓和名的做法,印象中没见过,所以感觉并不合适吧。

* 我觉得可以增加一个选项来让照片显示在右侧。
  OK, 我来加个选项

赞,你先试试看。不然我就得等后面有空了再折腾了(最近确实有点忙。。。)

@qiancy98
Copy link
Author

ifdef 里的 if 就说明这里是一个条件判断,就已经是 bool 型数据了。

是的, 但是 \def{\@zh}{true} 我总会担心变量内容在后面是有用的... 就, 我不知道你会不会编程时有这种担心... 不如直接定义为bool型比较让人安心...

中文的姓名之间确实不需要加上空格;应该是我当初参考英文模板做的时候觉得看着更清楚保留了空格。这个后面我调整一下。

我已经调整好了, 你可以看我的修改.

赞,你先试试看。不然我就得等后面有空了再折腾了(最近确实有点忙。。。)

我也已经改好啦, 你可以看我修改.

liweitianux added a commit that referenced this pull request Dec 5, 2024
liweitianux added a commit that referenced this pull request Dec 5, 2024
liweitianux added a commit that referenced this pull request Dec 5, 2024
liweitianux added a commit that referenced this pull request Dec 6, 2024
Extend the "\photo" command to support to place the photo on the right
side of profile, which can be controlled with the new "position"
parameter key.  Use the "keyval" package to implement the key=value
style optional parameters for the "\photo" command.

See also: PR #21
liweitianux added a commit that referenced this pull request Dec 6, 2024
@liweitianux
Copy link
Owner

针对你的一些调整,我做了重新实现并集成进去了。谢谢。

@liweitianux liweitianux closed this Dec 6, 2024
@qiancy98
Copy link
Author

qiancy98 commented Dec 6, 2024

谢谢www

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