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

Adding variable to define Custom browser to use #18

Merged
merged 7 commits into from
May 15, 2021
Merged

Adding variable to define Custom browser to use #18

merged 7 commits into from
May 15, 2021

Conversation

shubham-cpp
Copy link
Contributor

Add an option for user to specify using different browser(other than default one) for org/markdown previews. Its working for me right now, but will need further testing for any edge cases(if they exist).

Also updated README to include how to set the variable.

Added custom variable `grip-url-browser` to specify another browser to
be used instead of default one.

Added function grip--browser which will load preview either by using
grip-url-browser or default browser
grip-mode.el Outdated Show resolved Hide resolved
grip-mode.el Outdated Show resolved Hide resolved
All the changes have been updated as you said. I also made sure that
these changes are working as expected.

Let was giving me symbol's defination is void function something Like
that so by searching online I found setq-local.
Copy link
Owner

@seagle0128 seagle0128 left a comment

Choose a reason for hiding this comment

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

Overall it looks good now. Please make the changes, and then I will approve and merge. Thanks!

grip-mode.el Show resolved Hide resolved
grip-mode.el Outdated Show resolved Hide resolved
@seagle0128 seagle0128 merged commit 02b7a81 into seagle0128:master May 15, 2021
@seagle0128
Copy link
Owner

Thank you!

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.

2 participants