-
-
Notifications
You must be signed in to change notification settings - Fork 819
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
parent mode for magit-log-edit-mode #424
Comments
My initial idea is to create advice function for magit-log-edit to replace text-mode into markdown-mode. But I failed since markdown-mode also uses text-mode. :(
|
You would have to do something like this: (if (require 'markdown-mode nil t)
(define-derived-mode magit-log-edit-mode markdown-mode "Magit Log Edit")
(define-derived-mode magit-log-edit-mode text-mode "Magit Log Edit"
;; Recognize changelog-style paragraphs
(set (make-local-variable 'paragraph-start)
(concat paragraph-start "\\|*\\|(")))) |
No discussion on this one for a while. Here is another idea: could we make log-edit a minor mode? |
@lunaryorn does git-commit-mode support something like this. |
@tarsius Git Commit Mode will not derive from Markdown Mode. Git commit messages are no Markdown generally. However, I might refactor parts of Git Commit Mode into a minor mode, or find some other means of interacting with Markdown Mode, if there is need. Please open a corresponding issue in the Git Modes repository. |
@lunaryorn I agree that I won't add such a feature to magit's own commit (log-edit) mode, it should stay simple. Please use |
Since github uses markdown markup for commit message, it would be interesting if the
magit-log-edit-mode
could syntax highlight for the markdown syntax.The easiest way would be change the parent mode for magit-log-edit-mode to markdown-mode, however it will be bad decision because not all users uses
markdown-mode
.I cannot provide clean solution for this, but will you kindly consider to support to use XXX mode as the parent mode for
magit-log-edit-mode
, or to provide clean solution for this?Thank you.
The text was updated successfully, but these errors were encountered: