-
Notifications
You must be signed in to change notification settings - Fork 1
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
FR new command: E - edit variable #5
Comments
It seems the "C -> change variable" is already available, but only from the variable view. E may be preferable to C as C is already taken in the main view. |
Okay. I got the idea. Thank you. |
Hi Simon! The problem is that in one line, I can have several variables. I would have to allow selecting one first or editing all of them. |
If there are multiple ones in that line you may just an extra select, but if definitely suggest to first check if one of those is under the cursor.
|
Hello Simon! I included the option to edit the variable in one line. Just click on H and then on "E". |
Hello Prof. Eugenio! Could you forward me the modified 'odt' file? |
Hi Marcos, |
Hello Prof. Eugenio! Thank you very much. I will update the project. |
Again @eugeniodilo I suggest to move the docs to a subfolder "doc" and use markdown. This way it is easily visible online, can be easily adjusted - also partially - using pull requests and keeps images separated from text. |
Hi Simon, I went to check out your suggestion for which I thank you. |
OK. Note: there are wysiwyg editors for markdown, too, but if you both can go on with documenting this project I won't complain :-)
Note: this issue is fixed, it should be closed.
|
Isuue solved. |
The easiest implementation just works for simple
PIC 9
/PIC X
, and/or has no validation of the data. A more complex one could do the validation and adjusting as needed (the "upstream" extension should have code in for that).The E command would be also useful to be handled within the current Variable displays (just popping up as another window).
The text was updated successfully, but these errors were encountered: