On 04/07/2025 03:56, Charles Choi wrote:
Patch v2 (c5670fc0) Worg org-protocol.org: Rename "Construct Org…" to "Using Org protocol” submitted for review.
I have no objections to these changes requested by Christian. Reaction to my comments is optional.
@@ -83,7 +83,7 @@ Take the following steps to get Org protocol working for you. 2. Setup the Org protocol scheme handler. 3. Test your Org protocol setup. 4. Setup Org protocol capture. -5. Construct an Org protocol URL request. +5. Using Org protocol.
From my point of view, "Using Org protocol" is what happens after "get Org protocol working for you", not a step on this way. "Configuring applications" may be more appropriate. Of course, I do not insist.
+******* Terminal/Shell +Users with shell access can issue the URL request via command line +utility.
I am in doubts if terminal is really relevant besides debugging. xdg-open and open are just commands that may be used if there is no better way to open URL and when emacsclient is not available due to sandboxing. Depending on the way a particular application is configured, even shell may not be involved, xdg-open and URL may be passed directly to a system call from the exec family.
-********* Opera
In my opinion, as a separate commit removing a section is more convenient. On the other hand I admit that multiple pending patches may result in mutual conflicts (e.g. due to changing the heading level).
