Skip to content

feat: render settings page served from language server [IDE-1457]#345

Merged
andrewrobinsonhodges-snyk merged 28 commits intomainfrom
feat/IDE-1457_implement-settings-panel
Dec 16, 2025
Merged

feat: render settings page served from language server [IDE-1457]#345
andrewrobinsonhodges-snyk merged 28 commits intomainfrom
feat/IDE-1457_implement-settings-panel

Conversation

@andrewrobinsonhodges-snyk
Copy link
Copy Markdown
Contributor

@andrewrobinsonhodges-snyk andrewrobinsonhodges-snyk commented Dec 10, 2025

Description

Snyk plugin Settings page now uses HTML served from Language Server by default.

This can be overridden with an environment variable; we use factories to create the correct page.

Note that Eclipse splits out the per-project properties. We have replaced that page with a link to the main settings page.

I have also removed some unused code in FolderConfig.

Checklist

Screenshots / GIFs

Visuals that may help the reviewer. Please add screenshots for any UI change. GIFs are most welcome!

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Dec 10, 2025

CLA assistant check
All committers have signed the CLA.

@snyk-io
Copy link
Copy Markdown

snyk-io bot commented Dec 10, 2025

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@andrewrobinsonhodges-snyk andrewrobinsonhodges-snyk marked this pull request as ready for review December 15, 2025 15:13
@rrama rrama changed the title [WIP] feat: render settings page served from language server [IDE-1457] feat: render settings page served from language server [IDE-1457] Dec 15, 2025
Copy link
Copy Markdown
Contributor

@rrama rrama left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just have a bit of old code in there it seems.

Comment thread plugin/src/main/java/io/snyk/eclipse/plugin/html/BaseHtmlProvider.java Outdated
Comment thread plugin/src/main/java/io/snyk/eclipse/plugin/html/BaseHtmlProvider.java Outdated
Comment thread plugin/src/main/java/io/snyk/eclipse/plugin/preferences/Preferences.java Outdated
Comment thread plugin/src/main/java/io/snyk/eclipse/plugin/preferences/Preferences.java Outdated
Comment thread plugin/src/main/java/io/snyk/eclipse/plugin/html/BaseHtmlProvider.java Outdated
@andrewrobinsonhodges-snyk andrewrobinsonhodges-snyk merged commit b959eef into main Dec 16, 2025
10 checks passed
@andrewrobinsonhodges-snyk andrewrobinsonhodges-snyk deleted the feat/IDE-1457_implement-settings-panel branch December 16, 2025 14:17
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

Successfully merging this pull request may close these issues.

4 participants