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

Add valign-mode #1

Merged
merged 4 commits into from May 29, 2020
Merged

Add valign-mode #1

merged 4 commits into from May 29, 2020

Conversation

tumashu
Copy link
Contributor

@tumashu tumashu commented May 29, 2020

No description provided.

@casouri
Copy link
Owner

casouri commented May 29, 2020

非常感谢!我手动merge一下。

@casouri
Copy link
Owner

casouri commented May 29, 2020

哦对,有个问题,能不能用Emacs的格式写commit信息然后force push一下?我好像改不了你的commit信息。

@tumashu
Copy link
Contributor Author

tumashu commented May 29, 2020

未来有没有入 gnu elpa 的打算? 有的话, 协议是否签署就应该注意了

@casouri
Copy link
Owner

casouri commented May 29, 2020

有,签过了,我知道你也签过了。

@casouri
Copy link
Owner

casouri commented May 29, 2020

现在惟一的一个问题(也是我一开始没有写实现valign-mode的原因)是关闭valign-mode不会把已经对齐的表格变回去。不过大概这个功能也不是很必要。以后有人提要求再说吧。

* valign.el: Add URL, Version, Keywords and Package-Requires.
(valign--pixel-width-from-to, valign--beginning-of-table)
(valign--end-of-table, valign--align-separator-row): Fix package-lint and checkdoc warns
* valign.el (valign-lighter): New variable.
(valign-org-mode-hook): New function.
(valign-mode): New minor mode.
(valign-setup): Use valign-mode.
@tumashu
Copy link
Contributor Author

tumashu commented May 29, 2020

现在惟一的一个问题(也是我一开始没有写实现valign-mode的原因)是关闭valign-mode不会把已经对齐的表格变回去。不过大概这个功能也不是很必要。以后有人提要求再说吧。

我觉得这个功能是必要的,最起码 C-c C-c 之后,要变动,我目前测试发现,valign-mode 切换后需要重新打开文件才能生效

@casouri
Copy link
Owner

casouri commented May 29, 2020

是的,切换后同步效果可以我来写,你把目前的整理好我就merge。

@tumashu
Copy link
Contributor Author

tumashu commented May 29, 2020

应该可以合并了。

Copy link
Owner

@casouri casouri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

能不能把标题改成 Re add org-agenda support?加个空格

* valign.el (valign-org-agenda-finalize-hook): New function.
(valign-mode): use valign-org-agenda-finalize-hook.
@tumashu
Copy link
Contributor Author

tumashu commented May 29, 2020

ok

@casouri casouri merged commit 1d915f7 into casouri:master May 29, 2020
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.

None yet

2 participants