Comments (20)
Yeah, that simply was not implemented at all. While at it, I also fixed the strange scrolling behaviour (I had to press the cursor key a bunch of times before it started scrolling).
Try it out with the latest commit!
from pter.
Hm, good one. The happy paths worked π
Try with this: ef741a1d03
from pter.
I think the problem is that the help bar does not actually show the commands available in the help screen, but keeps showing the general pter shortcuts π€
The fact that it accepts both quit and cancel is β¦ uhm, historical? Not much thinking ever went into the help bar on the help screen; for kinda obvious reasons, I think. But yeah, it should still be correct, helpful, and consistent.
from pter.
You know what? quit
should actually quit (talk about being consistent) and cancel
should only close the help screen.
So the help bar stays as it is, but quit actally quits and refresh now works.
from pter.
Ah, I see, but doesn't pter already check for
quit = nop
andcancel = nop
? I just checked and it already refuses to start if either is unset.
What? You're right, it already does⦠I had no idea. git blame
says it's been a year though, so that's way beyond the event horizon.
True, true. So many new features
It's time to release them, then ;)
It really is. I'll still remove quit
from the help screen functions though. And let's see if the scrolling can be fixed into submission.
from pter.
Works, thank you!
from pter.
I'll plan to release 3.16 then later this week, hoping that nothing else gets in the way. I'm halfway considering giving this release the nickname 'andrei-a-papou'.
from pter.
Closing this, so I don't have to feel bad to release with pending issues in the milestone.
from pter.
Tested again, things seem to be working fine here!
from pter.
Oh, interesting! Iβll have a look at that.
from pter.
Chained keybindings on Help screen mostly work, but there are some hiccups:
- I use
^Xq
to exit Help screen. The combination works, but after returning to the main screen,^X
remains painted at the bottom of the screen. - On the Help screen, if I press
^X
, there's no way to cancel (except by hittingq
and exiting). On the main screen, I can press^X
and then a random unused key (or simply do a^C
) to break the chain. - There's no feedback for non-existing keys on the Help screen. On the main screen, pter prints a No such keybinding error and cancels the chain. This works for chained and non-chained keybindings.
from pter.
This is much nicer, thank you!
Strangely, though ^L
doesn't seem to refresh things and pter shows the No such keybinding error.
As a side note, do we really need Quit and Refresh screen on the Help screen? Personally, I don't mind Refresh screen (if it works), but maybe we should remove the Quit hint? It functions the same as Cancel anyway, so it's a tiny bit confusing.
from pter.
You know what? quit should actually quit (talk about being consistent) and cancel should only close the help screen.
In terms of consistency, I agree 100%!
However, maybe I'm missing something, but why would you want to quit the app from the Help screen? Especially with the default q
binding that's all too easy to hit -- and then it's good-bye my complex unsaved search, for example.
-
I'm not sure making hints on Help screen configurable is worth it, so maybe just removing the "Quit" hint is easiest?
-
Or at least moving it to the far right, so it's not the first hint/action? In which case (also for consistency) I would make the "Quit" action come last on the main screen too (I'm talking about the default hints here).
What do you think?
from pter.
Maybe not just removing the quit hint, but the functionality altogether π€ but then I'll have to protect the user from locking themselves in (like it's done with the quit
function).
Having quit as the first help option in the main screen is intentional: sure, the help bar could deploy some backpack-problem solution to ensure that the last help option is quit
, but it's so much easier to show it as the first option. pter shouldn't be the vi/vim of TUI task managers ;)
from pter.
but then I'll have to protect the user from locking themselves in
I'm not sure I follow... You can always "Cancel"/^C
, go back to main screen and quit from there, no?
Having quit as the first help option in the main screen is intentional
It's your call -- after all, the main screen hints are fully configurable :)
from pter.
but then I'll have to protect the user from locking themselves in
I'm not sure I follow... You can always "Cancel"/
^C
, go back to main screen and quit from there, no?
Oh, allow me to elaborate: since a user might configure q = nop
, pter will not start if there is no shortcut to quit.
Similarly pter should deny to start if there is no shortcut for cancel
, because otherwise a user might get stuck in the help screen since quit
won't be accessible in the help screen either.
Having quit as the first help option in the main screen is intentional
It's your call -- after all, the main screen hints are fully configurable :)
True, true. So many new features; it's admittedly hard to keep track π
from pter.
Ah, I see, but doesn't pter already check for quit = nop
and cancel = nop
? I just checked and it already refuses to start if either is unset.
True, true. So many new features
It's time to release them, then ;)
from pter.
It really is. I'll still remove quit from the help screen functions though.
I just tried the latest commit, and the quit
function on Help screen no longer works via keybinding -- so that's good. But the hint for it is still there, FYI
from pter.
Argh. Yeah, fixed that.
from pter.
Thank you, but no need: you've already mentioned me in git commits, this is plenty :)
from pter.
Related Issues (20)
- Distraction-free task editing/creation HOT 3
- Patch for help screen HOT 5
- Mouse-wheel scrolling broken after editing a task externally HOT 38
- Patch to fix caret key issues HOT 1
- Patch to improve line editing some more HOT 6
- Patch: UI proofreading/minor editing HOT 7
- Patch to invert completion highlighting HOT 2
- Helpbar improvements (configurable hints and key sorting) HOT 23
- Regression: pter crashes when choosing a filename to save the task to HOT 1
- Distraction-free mode should show task list on "jump-to" action HOT 6
- What does "N" (edit note) do? HOT 6
- Request: Add filename in task format? HOT 13
- Add file name completion when searching HOT 8
- Bug: Editor keybindings not dumped? HOT 2
- Centralised archive with multiple todo.txt file HOT 1
- Support for different sources of tasks
- qpter not finding PyQt5 under python3 (No longer easily available under pip install) HOT 4
- new qpter issue (at least on my end!) HOT 4
- Bug: When creating a note for a task, the folder where it is stored does not respect the general configuration and uses the name of the todo file. HOT 5
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 pter.