-
Notifications
You must be signed in to change notification settings - Fork 397
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
[GnoVM] Make error line clickable #3517
Comments
I'm not sure how editors make it work honestly; but I think it needs to be either 1. at the beginning of a newline or 2. an actual relative file, rather than being a package path. Would be good to find some references to how to actually make this "clickable" in mainstream editors. |
Usually, using the format Or, we can also use OSC 81 hyperlink escape sequences enables more explicit handling. This method can also be work over a external terminal, such as iTerm. For example, it takes the following form (we can use other schemes also):
However, I believe that in most cases, simply applying the first method with a consistent format should be sufficient without any issues. Footnotes |
To share a bit my progress, my idea on how to make this happen is to replace the String() representation of Location type to replace the package path (gno.lang/p/demo/ownable) by the actual location of the package inside the file system. I started a draft to make it happen, however after more considerations. The issue that rise is that I need to store the filepath inside every nodes so the string will be duplicated too much. So far, I am trying to find another cleaner way. |
Description
Would be cool and good for UX to make the error line clickable so it takes you to the location of the error, such as here:
Do you think this is feasible/practical?
The text was updated successfully, but these errors were encountered: