org-gtd.el icon indicating copy to clipboard operation
org-gtd.el copied to clipboard

UI Suggestion for v3. Put the transient menu and horizon view in the clarify screen.

Open maikol-solis opened this issue 2 years ago • 7 comments

Currently, if you try to clarify something, you see this screen

image

However, I feel that a better UI could be something like this since the beginning:

image

Here, you have everything ready to start handling your tasks.

Otherwise, you'll need to assign org-gtd-organize and org-gtd-clarify-toggle-horizon-window to some keybindings or call them manually each time.

WDYT?

maikol-solis avatar May 07 '23 18:05 maikol-solis

Well, there's a few things here.

First, regarding the horizons window, there is a custom variable you can set to have it show by default.

Second, regarding the transient menu, I'm not actually sure how to give control to another window while it's up, so invoking it right away would probably mean the user has to close it anyway (if you know how to clarify your item while the menu is up, let me know!)

Third, this is not great for people working on possibly small monitors, so I'd rather leave as much as possible here to configuration, like displaying the horizons buffer by default.

And, finally, the org-gtd-clarify-map exists precisely for the sake of binding functions to keyboard shortcuts in this particular step of the process, including for instance the new function org-gtd-clarify-project-insert-template, still obviously undocumented as well, but requested in #121

You sound like you hesitate or don't want to use keybindings here, would you help me understand why that is?

Trevoke avatar May 08 '23 15:05 Trevoke

Yes. I wasn't aware of org-gtd-clarify-show-horizons at all. Thanks for pointing out.

Just a little remark here. In the variable docstring

https://github.com/Trevoke/org-gtd.el/blob/ba42650860ed087ca9d0eb10c3459dd7a281ac2f/org-gtd-clarify.el#L38-L44

The non-nil (t) value causes an error. It should be the position where you want the popup-buffer (right, left, top or bottom). I set it with this line:

  (setq org-gtd-clarify-show-horizons 'right)

Now, the transient menu it's just thinking about having the options right away when you start the clarifying process. But it's optional at the end. I have org-gtd-clarify bound to C-c C-c for easy ergonomics.

maikol-solis avatar May 09 '23 18:05 maikol-solis

Oh! You're right, the logic I have written here is incorrect (and obviously not properly tested). Thanks for finding that, I'll fix the docstring and the logic. I was trying to be "friendly" but that's .. Probably more harmful than helpful.

Trevoke avatar May 10 '23 13:05 Trevoke

Well, in the end, it's just giving the users all the options right away.

But it's your call, of course.

Thanks.

maikol-solis avatar May 10 '23 14:05 maikol-solis

What I mean is -- please try to open the organize menu and then try to clarify the item. If you manage, let me know how you did it, please.

On Wed, May 10, 2023 at 10:32 AM Maikol Solís @.***> wrote:

Well, in the end, it's just giving the users all the options right away.

But it's your call, of course.

Thanks.

— Reply to this email directly, view it on GitHub https://github.com/Trevoke/org-gtd.el/issues/143#issuecomment-1542317158, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAQSSM64VHBZFRBLAKCLGTXFORI7ANCNFSM6AAAAAAXZBPFPY . You are receiving this because you commented.Message ID: <Trevoke/org-gtd. @.***>

Trevoke avatar May 11 '23 01:05 Trevoke

I don't mind somehow telling the user what all the options are, I think that's probably a good idea, and it might even be worth doing before they start clarifying, but I don't think it can be done with the transient menu.

On Wed, May 10, 2023 at 9:05 PM Aldric @.***> wrote:

What I mean is -- please try to open the organize menu and then try to clarify the item. If you manage, let me know how you did it, please.

On Wed, May 10, 2023 at 10:32 AM Maikol Solís @.***> wrote:

Well, in the end, it's just giving the users all the options right away.

But it's your call, of course.

Thanks.

— Reply to this email directly, view it on GitHub https://github.com/Trevoke/org-gtd.el/issues/143#issuecomment-1542317158, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAQSSM64VHBZFRBLAKCLGTXFORI7ANCNFSM6AAAAAAXZBPFPY . You are receiving this because you commented.Message ID: <Trevoke/org-gtd .@.***>

Trevoke avatar May 11 '23 01:05 Trevoke

Oh, right! I get what do you mean. Jumping across the transient menu and the clarify buffer isn't as smooth as I expected. I think that binding the menu to something simple like C-c C-c should do the work. Sorry for the noise.

maikol-solis avatar May 17 '23 11:05 maikol-solis