Skip to content

Conversation

@phillip-simons
Copy link

This PR adds the evil-fringe-mark package.

I configured it to display both user and special marks globally.

Thanks

@sunlin7
Copy link
Contributor

sunlin7 commented Dec 30, 2024

Have you try it with holy mode? And how about its performance on a large file with several marks? I remember this package have some issues on these two questions. correct me if I was wrong. Thank you.

(defun spacemacs-evil/init-evil-fringe-mark ()
(use-package evil-fringe-mark
:config
(setq-default evil-fringe-mark-show-special t)
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think it would be better to set options in the :init or :custom block. It makes them easier to override by users when the package is deferred.

(use-package evil-fringe-mark
:config
(setq-default evil-fringe-mark-show-special t)
(global-evil-fringe-mark-mode)))
Copy link
Collaborator

Choose a reason for hiding this comment

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

If we add this package, it should be deferred, and the mode should not be enabled by default. It would make sense to add a toggle as for evil-visual-mark-mode, SPC t ' seems logical as a key binding.

Copy link
Collaborator

@smile13241324 smile13241324 left a comment

Choose a reason for hiding this comment

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

Not a bad idea though I think it makes sense to have this configurable in the layer and to have a toggle for this feature as well.

Given the performance concerns from @sunlin7 the package should be disabled by default. Also please only install the package when it is used.

@smile13241324 smile13241324 self-assigned this Feb 24, 2025
@smile13241324
Copy link
Collaborator

Looks like evil fridge mode had not received any commit in the last 6 years. Given thst the package seems dead I am hesitant to add it to Spacemacs.

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.

4 participants