You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Given that we have evil motions for going backwards (for defun: evil-backward-section-end) and forward for defun, it would be helpful to have the same for defclass as well. This would ideally make it easier to have a keymap to go back to previous/next class directly instead of moving between functions.
Going through the code-base, I didn't find functions that would support defclass like they do for defun (like beginning-of-defun, end-of-defun). Please point out if I missed anything here.
@krshrimali can you give an example of a defclass that would be incorrectly navigated with the motions evil supplies?
also,
was there any reasoning to not support motions for defclass?
Evil primarily emulates vim functionality. As this is not functionality in vim, it would have to be a common emacs-specific need to be addressed. And defuns are much more commonly used than defclasses. For example, evil has zero usages.
Issue type
Pitch
Given that we have evil motions for going backwards (for
defun
:evil-backward-section-end
) and forward fordefun
, it would be helpful to have the same fordefclass
as well. This would ideally make it easier to have a keymap to go back to previous/next class directly instead of moving between functions.Going through the code-base, I didn't find functions that would support
defclass
like they do fordefun
(likebeginning-of-defun
,end-of-defun
). Please point out if I missed anything here.Environment
Emacs version: 29.4
Operating System: Endeavour OS
Evil version: 1.15.0
Evil installation type: MELPA
Graphical/Terminal: Terminal (/both)
Further notes
Happy to discuss this forward and any thoughts on implementations (can help with a PR if necessary).
Was just curious, was there any reasoning to not support motions for
defclass
? Maybe I'm missing some context here.The text was updated successfully, but these errors were encountered: