Git Product home page Git Product logo

Comments (10)

gcv avatar gcv commented on August 16, 2024 2

@zenspider: Yes. Let's get out of the business of deciding on key prefixes for users. I looked at Projectile for inspiration, and it does not provide a default prefix.

@PauloPhagula: I agree with your reasoning about C-x x. It makes sense for users with muscle memory to keep it. Still, I think it's best for Perspective to just get out of the way and let users decide what to do.

I still don't want to kill it outright for everyone, so I kept C-x x or Emacs 27 and earlier, and blanked it for 28. There's a warning message for people who do not have the prefix set, and a way to disable the warning if someone wants the prefix off.

So overall, still a breaking change, but hopefully one easy to get past. I'd appreciate it if you two would give the change a quick look (see 7ec5a6e) and I'll merge it in a couple of days.

from perspective-el.

gcv avatar gcv commented on August 16, 2024 1

Fair point about breaking before a big new release, but it's already too late for that. :)

This branch implements option 3 and switches to C-c C-p as the prefix: https://github.com/nex3/perspective-el/tree/update-for-emacs-28

Looks like Emacs 28 has different C dependencies from 27, and my system doesn't compile it out of the box. I don't have time to figure it out right now, so I can't test the patch on 28. @PauloPhagula, please test that branch and let me know if it works on 28.

from perspective-el.

gcv avatar gcv commented on August 16, 2024

I have always considered C-x x an unfortunate default, and use C-c C-p. I don’t think it conflicts with anything popular..?

That said, it will be a breaking change in Perspective, something I try very hard to avoid. I suppose I can reduce its impact on existing users’ muscle memory by only changing the prefix for Emacs 28 and later.

from perspective-el.

gcv avatar gcv commented on August 16, 2024

@zenspider: Do you have an opinion on this? It's a helluva change to inflict on long-standing users of Perspective who never changed the prefix key. As far as I can tell, we can:

  1. Ignore the new Emacs 28 bindings for C-x x and so shadow them.
  2. Change the Perspective prefix, breaking users' muscle memory. Document in the changelog and hope that's enough.
  3. Option 2, but only for Emacs 28.

from perspective-el.

zenspider avatar zenspider commented on August 16, 2024

I don't have an opinion... I immediately changed my prefix (^z ftw!) and suspect many people do.

Unfortunately we want to keep this around, otherwise we could lean into declare w/ obsolete. As such, I suspect option 3 is probably the most gentle, but option 2 could be fine as well and prepare people BEFORE they move over (so much stuff breaks on a big new release that it'd be nice not to be a part of that).

from perspective-el.

gcv avatar gcv commented on August 16, 2024

I just realized that C-c C-p conflicts with a bunch of stuff in Org, and also markdown-mode. I'm open to other suggestions.

from perspective-el.

gcv avatar gcv commented on August 16, 2024

What about C-c M-p?

from perspective-el.

zenspider avatar zenspider commented on August 16, 2024

What about nil? Maybe don't set it and ask users to find something that works for them? I know I have mine set and thus won't even notice a version bump (nor will anyone else who already customized). Anyone who's using the default can have a message pop up or something.

from perspective-el.

PauloPhagula avatar PauloPhagula commented on August 16, 2024

Even though, it was I, who raised the issue, I'm keeping C-x x for perspective.el rather than the new buffer actions, as most of those buffer actions are in my particular view not so useful (except maybe for rename-buffer, but then again we have crux and so many hacks people made to rename a file along with its buffer). It is very seldom I'll want to perform any of the tasks listed in there.

from perspective-el.

PauloPhagula avatar PauloPhagula commented on August 16, 2024

@gcv thank you. I think this settles it, so I'll be closing the issue.

from perspective-el.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.