Comments (10)
@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.
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.
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.
@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:
- Ignore the new Emacs 28 bindings for
C-x x
and so shadow them. - Change the Perspective prefix, breaking users' muscle memory. Document in the changelog and hope that's enough.
- Option 2, but only for Emacs 28.
from perspective-el.
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.
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.
What about C-c M-p
?
from perspective-el.
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.
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.
@gcv thank you. I think this settles it, so I'll be closing the issue.
from perspective-el.
Related Issues (20)
- Marginalia annotation not working with `persp-switch-to-buffer*` HOT 4
- persp-mode does not get activated during startup HOT 3
- helm mini frame errors saying no such buffer HOT 5
- [feature request] - Option to not save modes for buffers HOT 1
- Creating pipe: Too many open files HOT 5
- Can't setup perspective layout at startup HOT 4
- Perspective breaks after enable-theme HOT 5
- Perspective hides buffers when running Emacs as server HOT 4
- Current perspective name not being propertized in the mode-line HOT 5
- Display perspective in frame title HOT 7
- `C-x 5 2 ` different behaviour when running perspective with daemon HOT 2
- Perspective.el wipes out perspectives when a emacsclient frame is closed. HOT 2
- `persp-delete-frame` alters windows of other frames when killing frame with dedicated window HOT 1
- Issue with using ediff and perspective [sorted] HOT 6
- Compare to tabspaces? HOT 2
- Support eshell state save and restore
- Would it support different themes per frame? HOT 2
- Loading perspectives upon opening emacs HOT 5
- ibuf-ext.el not loaded HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
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.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from perspective-el.