Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

org-roam-show-graph: PREFIX to generate a graph but not display it #232

Merged
merged 1 commit into from
Mar 6, 2020

Conversation

acowley
Copy link
Contributor

@acowley acowley commented Mar 5, 2020

Motivation for this change

This lets the viewer application watch the file, or have the user manually refresh the view rather than, say, opening a new browser tab.

org-roam.el Outdated Show resolved Hide resolved
org-roam.el Outdated
(defun org-roam-show-graph ()
"Displays the generated Org-roam graph using `org-roam-graph-viewer'."
(interactive)
(defun org-roam-show-graph (prefix)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

&optional prefix

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I appreciate the improvements! FWIW, I used org-roam-insert as a style reference for the prefix argument, so perhaps we should change that, too.

This lets the viewer application watch the file, or have the user
manually refresh the view rather than, say, opening a new browser tab.
@jethrokuan jethrokuan merged commit 8f83ffc into org-roam:develop Mar 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants