If aerc has keybinds set like
rr = reply rq = reply -q
it doesn't forward the common prefix to the underlying term
Steps to reproduce:
- Open an email
- Press "/" ( assuming less as pager)
- Type "r". Nothing comes up on the screen.
- Type "r" again. Now aerc takes this as the "Rr" command and opens a tab for you to reply.
- Close compose tab, goto step 2. Next, type "rq" in the search box. Now aerc takes this as the "Rq" command and opens a tab to reply, with quoted text.
If I try
org-fc-review-bufferon a non-org-roam file, the title is shown after the "Flip (X)" on the header line, while this doesn't occur with org-roam files (or when reviewing the entire directory).
On second thoughts, ~l3kn, should I send a patch that uses a defcustom variable to enable this feature for users? It's simply adding hooks after all.
REPORTED RESOLVED WONT_FIX
org-roam adds a huge latency to reviewing a heading, which can, fortunately be almost entirely removed by disabling the
org-roam-buffer-prepare-hookhook during review sessions.
Here is my code to do it:(add-hook 'org-fc-before-review-hook (lambda () (setq org-roam-buffer-prepare-hook nil) (setq +org-roam-open-buffer-on-find-file nil) )) (defun my/enable-org-roam-buf () (setq org-roam-buffer-prepare-hook '(hide-mode-line-mode org-roam-buffer--insert-title org-roam-buffer--insert-backlinks org-roam-buffer--insert-ref-links)) (setq +org-roam-open-buffer-on-find-file t) ) (add-hook 'org-fc-after-review-hook #'my/enable-org-roam-buf)
I'm simply adding this as reference for future users.
I think the simplest solution is fine: this issue is quite low priority compared to other possible features (in my opinion of course), and since the simplest solution is a clear low hanging fruit, we should just assume that is the scope for this feature.
Would you prefer a forecast "due now, due today, due this week (next 7 days), due this month (next 30 days)" or one for e.g. each day of the week?
The first sounds more useful than the second, so I would say the former.
Sidenote: I just realized that the "re-centering" itself is a workaround: what I really intended is the top-level heading (aka the root) to always be shown on the screen.
If we have a flashcard whose heading level is, for example, 15, then it would show that line on the center. If showing the entire tree takes up more than half of the screen vertical size, then "re-centering" would move the ancestor tree "up", meaning the root of the tree wouldn't be seen on the screen.
I don't know what's the appropriate emacs function to ensure this occurs (one could, for example, attempt to move the cursor to the bottom of the screen, which would then fix the previous example issue - but another edge case would be when the tree is so huge that it takes up more than the screen vertical size (impractical, yes, but I'm just listing out edge cases)), but I do think that the above fix is "good enough", assuming there is no issue on my end that requires me to use this.
All right, so I pulled the latest version of org-fc and I can still accomplish this using the same workflow I just showed you in the MP4 file.
The main thing you need to do to reliably replicate this using native emacs keybindings (at least according to what I know) is that you need to first move the cursor to the last line (or the line after that) of the buffer, and then press
C-l C-lwhich could move the line itself to the top of the screen, and then begin a review session. This should always cause the first flashcard to not show the entire ancestor by default unless you scroll up in the flip mode of the flashcard.
I found an emacs native method to fix this for all possible flashcards, one that you can merge into the existing code: using
recenterfunction.;; center the flashcard before showing it (setq org-fc-after-setup-hook '(recenter))
And that's it, this problem doesn't occur at all. This is my solution, and it's not even an awkward workaround (thanks for the hooks, by the way - I forgot about them until I reread your docs).
I understand that you would need to replicate this before you would understand the need for this code being merged, so please tell me if you still have issues replicating this.
: package details
Package: org-fc Source: Straight Pinned: unpinned Build: HEAD -> main, origin/main, origin/HEAD 0fd72b4 2020-10-02 11:40:07 +0200 (:host nil :repo "https://git.sr.ht/~l3kn/org-fc" :branch "main" :files ("*") :package "org-fc" :type git :local-repo "org-fc") Homepage: https://www.leonrische.me/pages/org_flashcards.html Modules: Declared by the following Doom modules: :private packages :private modules Configs: This package is configured in the following locations: