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

Support for embedded URLs could be better #7

Open
sinic opened this issue Nov 25, 2022 · 1 comment
Open

Support for embedded URLs could be better #7

sinic opened this issue Nov 25, 2022 · 1 comment

Comments

@sinic
Copy link
Owner

sinic commented Nov 25, 2022

This is a point mentioned in a YouTube video by @NapoleonWils0n, first brought to my attention in issue #5. In the video, plain URLs are inserted in the message body, but a better way to communicate URLs would be to use XML-based markup. Unfortunately, EDNC currently doesn't do any parsing of notification bodies at all (which is allowed by the standard as long as it doesn't claim to be able to do so).

Either way, notifications from Firefox commonly open an URL as their default action, so I don't know how pressing of an issue it really is.

@sinic
Copy link
Owner Author

sinic commented Nov 25, 2022

As a workaround, the following will automatically highlight all URLs in ednc-view-mode:

(add-hook 'ednc-view-mode-hook #'goto-address-mode)

I'm wary of adding this to EDNC, however, because if markup parsing is implemented at some point, it might interfere with goto-address-mode (e.g., if the text contains one URL, but the href another).

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

No branches or pull requests

1 participant