Closed tiwarishub closed 3 years ago
Cool description @tiwarishub ❤️
Few suggestion based on description:
.github
folder in temp as well and keep same folder layout? This way it might easier for user to co-relate the structure of the files.Few suggestion based on description:
- Should we add
.github
folder in temp as well and keep same folder layout? This way it might easier for user to co-relate the structure of the files.- Suggestion to try: Does vs code allow closing a editor by its handle? We would get the handle when we open the editor. Just a thought.
- The two notifications are looking pretty same and hence confusing. Can the workflow file related message be made part of same notification? We can add more text to convey this and may be make the file url as a hyperlink text.
export let activeTextEditor: TextEditor | undefined;
, which is basically which is active editor. But in TextEditor type there is no way to close that editor (they had hide() function but now they have deprecated it because of its unusual behaviour ). For the notification message, I feel it is very obscure and not intuitive for user to understand that it will take to workflow file. We can add explicit text to tell user that a workflow file (with hypelink file name) has been pushed to their repo. Take @pulkitaggarwl 's help to get the string.
Hi @pulkitaggarwl , Need your help in finalizing the information box to be shown to user for tmp directory changes , which we had discussed with you. Below are some of options , Please let us know if any one of them is fine or if have any suggestion for different one. Commit files url we have made hyperlink (in 2 and 3rd screenshot) , so that user can browse commited files as well
This format. change "commit url link" to "commit url" and hyperlink "commit url"
Changed commit URL
This change is done to show draft workflow files in Temp or tmp directory instead of adding them to local repo . Below is screenshot how workflow will be shown.
Once user clicks on "Modify and Commit" button, we will remove the workflow files from tmp directory and add
(deleted)
for the active editor tabs, for example like this In the above*-1365.yml
files are deleted one.Note :
A good user interface for user will be , we should close/hide the text editors for tmp files as those files are deleted but vscode does not expose any api for for open text editor which can be used to close a particular text editor. They only thing they provide is
commands.executeCommand(workbench.action.closeActiveEditor)
orcommands.executeCommand('workbench.action.closeAllEditors')
, which will not fulfil our scenarios. They have long pending feature request (almost 3 years), but it is still not delivered. https://github.com/microsoft/vscode/issues/21617 https://github.com/microsoft/vscode/issues/15178If user discards the workflow, then we will move the workflow files from tmp directory to git repo directory. then will again will open those workflows,
Please note from above pic is, now each created workflow file is there in two tabs of vscode, One with
(deleted)
and other one is which is moved.Last , if user want to browse the committed workflow files, then we have added one information messages for that as well.