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

Consider if default theme should have different cursor colors for insert and normal mode #10568

Closed
matklad opened this issue Apr 23, 2024 · 2 comments · Fixed by #10608
Closed
Labels
A-theme Area: Theme and appearence related E-easy Call for participation: Experience needed to fix: Easy / not much E-good-first-issue Call for participation: Issues suitable for new contributors E-has-instructions Call for participation: Has instructions for fixing the issue and opening a PR

Comments

@matklad
Copy link
Contributor

matklad commented Apr 23, 2024

In default configuration helix, like vim, only shows the current mode in the modeline. This is poor UX, especially for new users, as, by just looking at the screen where you edit text, it's impossible to say which mode is active.

A nice solution here is to use different cursor color for different modes. This is available in the, e.g., ayu_evolve theme.

I suggest that the default theme also gets this cursor color distinction, as that would be, imo, a big win for usability for new users (or for old users who don't want to bother with config files).

@pascalkuthe
Copy link
Member

pascalkuthe commented Apr 25, 2024

The default cursor had mode based cursor colors forever. But a while ago we merged a PR that changed the way the highlight inheritance works (to allow theming the primary cursor different from the other cursors and support mode dependent highlights at the same time).

This broke the default theme. To fix this we would need to simply copy ui.cursor.selection and ui.cursor.insert to ui.cursor.primary.selection and ui.cursor.primary.insert

@pascalkuthe pascalkuthe added E-easy Call for participation: Experience needed to fix: Easy / not much A-theme Area: Theme and appearence related E-good-first-issue Call for participation: Issues suitable for new contributors E-has-instructions Call for participation: Has instructions for fixing the issue and opening a PR labels Apr 25, 2024
@Diogenesoftoronto
Copy link
Contributor

Diogenesoftoronto commented Apr 26, 2024

It seems no one has linked a pr. I am going to try doing this. Wish me luck will be first PR for helix. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-theme Area: Theme and appearence related E-easy Call for participation: Experience needed to fix: Easy / not much E-good-first-issue Call for participation: Issues suitable for new contributors E-has-instructions Call for participation: Has instructions for fixing the issue and opening a PR
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants