-
Notifications
You must be signed in to change notification settings - Fork 13
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
feat: Add improved platform support #196
Conversation
@jbrundage please review |
e51586b
to
f2ed30c
Compare
@jbrundage please review |
0bd3921
to
bd39d9f
Compare
README.md
Outdated
// Open Workunits in external browser. | ||
"ecl.WUOpenExternal": true | ||
// Write eclcc logfile to specified file. | ||
"ecl.eclccLogfile": "" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dup
README.md
Outdated
|![Submit](resources/light-png/play.png) Submit | F5 | Submit ECL | | ||
|![Compile](resources/light-png/file-binary.png) Compile | | Compile ECL | | ||
| Syntax Check | F7 | Save and check syntax of current file | | ||
| Language Reference Lookup | shift + F1 | Toggle between "My" and "All" Workunits | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wrong description
README.md
Outdated
|---------------------------|:--------:|------------------------------------------------------------------| | ||
|![My Workunits](resources/light-png/person.png) My workunits | | Toggle between "My" and "All" Workunits | | ||
|![All Workunits](resources/light-png/organization.png) All workunits | | Toggle between "My" and "All" Workunits | | ||
|![Refresh](resources/light-png/refresh.png) ECL Watch | | Refresh Tree | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wrong command string
src/ecl/check.ts
Outdated
@@ -74,7 +74,7 @@ function checkUri(uri: vscode.Uri, eclConfig: vscode.WorkspaceConfiguration): Pr | |||
if (uri) { | |||
const wsf = vscode.workspace.getWorkspaceFolder(uri); | |||
if (wsf) { | |||
vscode.window.setStatusBarMessage(`Syntax Check: ${path.relative(wsf.uri.fsPath, uri.fsPath)}`); | |||
// vscode.window.setStatusBarMessage(`Syntax Check: ${path.relative(wsf.uri.fsPath, uri.fsPath)}`); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Commented code.
src/ecl/eclWatchDocument.ts
Outdated
|
||
public static register(context: vscode.ExtensionContext): vscode.Disposable { | ||
|
||
// vscode.workspace.registerTextDocumentContentProvider("cowsay", new MyProvider()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cowsay!
src/ecl/eclWatchDocument.ts
Outdated
resolveCustomEditor(document: vscode.CustomDocument, webviewPanel: vscode.WebviewPanel, token: vscode.CancellationToken): void | Thenable<void> { | ||
throw new Error("YYYMethod not implemented."); | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove entire file
src/eclwatch.tsx
Outdated
const bodyStyles = window.getComputedStyle(document.body); | ||
|
||
const backColor = bodyStyles.getPropertyValue("--vscode-editor-background"); | ||
const foreColour = bodyStyles.getPropertyValue("--vscode-input-foreground");//Palette.textColor(backColor); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
commented code
src/eclwatch.tsx
Outdated
|
||
// const oldState = vscode.getState() || { url: "" }; | ||
// const iframe: HTMLIFrameElement = document.getElementById("myFrame") as any; | ||
// iframe.src = oldState.url; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Commented code
src/eclwatch.tsx
Outdated
|
||
// Local debugging without VS Code | ||
if (document.location.protocol === "file:") { | ||
//render("http", "10.173.14.207", "8010", "gosmith", "28Ee1$BHoJvNOTk$KL6F", "W20200910-090123"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Commented code
bd39d9f
to
1a3f243
Compare
src/eclwatch/WUResultFluent.tsx
Outdated
const startIndex = loremIndex + wordCount > LOREM_IPSUM.length ? 0 : loremIndex; | ||
loremIndex = startIndex + wordCount; | ||
return LOREM_IPSUM.slice(startIndex, loremIndex).join(" "); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove file
src/eclwatch/WUResultStore.ts
Outdated
totalLength: retVal.then(response => response.totalLength) | ||
}); | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Check if this custom version is needed.
test.html
Outdated
<iframe id="myFrame" src="http://10.173.14.207:8010/esp/files/stub.htm" | ||
style="position:fixed; top:0; left:0; bottom:0; right:0; width:100%; height:100%; border:none; margin:0; padding:0; overflow:hidden; z-index:999999;"> | ||
Your browser doesn't support iframes | ||
</iframe> | ||
</body> | ||
|
||
</html> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove
webview-test.html
Outdated
|
||
</body> | ||
|
||
</html> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove
webview-styles.txt
Outdated
--vscode-gitDecoration-untrackedResourceForeground:#73c991; | ||
--vscode-gitDecoration-ignoredResourceForeground:#8c8c8c; | ||
--vscode-gitDecoration-conflictingResourceForeground:#6c6cc4; | ||
--vscode-gitDecoration-submoduleResourceForeground:#8db9e2; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove
Added Workunit Tree Added Submit / Compile buttons Added pinned launch configuration Added eclcc log file override Fixes hpcc-systems#191 Fixes hpcc-systems#188 Fixes hpcc-systems#187 Fixes hpcc-systems#190 Fixes hpcc-systems#143 Fixes hpcc-systems#184 Signed-off-by: Gordon Smith <GordonJSmith@gmail.com>
1a3f243
to
22e3cd8
Compare
Added Workunit Tree
Added Submit / Compile buttons
Added pinned launch configuration
Added eclcc log file override
Fixes #191
Fixes #188
Fixes #187
Fixes #190
Fixes #143
Fixes #184
Signed-off-by: Gordon Smith GordonJSmith@gmail.com