Coder Social home page Coder Social logo

Comments (5)

protesilaos avatar protesilaos commented on July 28, 2024 1

Thank you @vedang! I will merge your PR, subject to a stylistic change noted there.

Finally, denote-org-capture uses denote-prompts, but I didn't look at this bit of the code deeply.

I have not tested it lately, but I believe everything is okay with it.

If we have any other cases, let's discuss them here and proceed accordingly.

I will continue work on the rename-honours-denote-prompts branch as soon as possible. Then merge everything into main.

from denote.

vedang avatar vedang commented on July 28, 2024

This is so cool @protesilaos ! I looked at the code and tested the following scenarios successfully:

  • Rename a note with M-x denote-rename-file
  • Rename a note with point on filename Dired buffer
  • Rename a note with using front-matter (check that denote-prompts is not invoked)

Post that, I checked to see which code-paths could be changed to respect denote-prompts.

The following paths already call denote or other commands interactively and therefore respect denote-prompts correctly:

  • denote-region
  • denote-open-or-create
  • denote-link-or-create
  • denote-link-after-creating

The following commands use a fixed list, and therefore overwrite denote-prompts:

  • denote-type
  • denote-date
  • denote-subdirectory
  • denote-template
  • denote-signature

All of these can be modified to use denote-prompts, which is a change I am submitting on top of the current commit (see #248).

Finally, denote-org-capture uses denote-prompts, but I didn't look at this bit of the code deeply.

from denote.

protesilaos avatar protesilaos commented on July 28, 2024

I just merged everything into main. I made lots of changes/refinements to ancillary functions as well.

from denote.

protesilaos avatar protesilaos commented on July 28, 2024

Sorry, I mean I merged the branch rename-honours-denote-prompts. I will take care to delete it before the new release.

from denote.

protesilaos avatar protesilaos commented on July 28, 2024

I think we are done with this for now. If there is any command we missed, we can always update it later. Thanks!

from denote.

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.