Skip to content

"Launch Geometry Dash" Run configuration + Minor tweaks#1

Merged
prevter merged 5 commits intogeode-sdk:mainfrom
dom3:port-from-vscode
Mar 18, 2026
Merged

"Launch Geometry Dash" Run configuration + Minor tweaks#1
prevter merged 5 commits intogeode-sdk:mainfrom
dom3:port-from-vscode

Conversation

@dom3
Copy link
Copy Markdown
Contributor

@dom3 dom3 commented Mar 17, 2026

Add a "Launch Geometry Dash" run configuration like how VSCode does it.
Maybe in the future custom arguments can be added or a custom working directory but for now I have it like this

image image

I also changed the way icons are organized like how the docs recommend it to be
This means that the icons are organized like this into one single file

object GeodeIcons {
    @JvmField
    val Geode: ()->Icon = {
        val icon = IconLoader.getIcon("/icons/geode.png", javaClass)
        val scale = min(DEFAULT_ICON_SIZE.toFloat() / icon.iconWidth, DEFAULT_ICON_SIZE.toFloat() / icon.iconHeight)
        IconUtil.scale(icon, null, scale)
    }

    @JvmField
    val GeometryDash: ()->Icon = {
        val icon = IconLoader.getIcon("/icons/geometry_dash.png", javaClass)
        val scale = min(DEFAULT_ICON_SIZE.toFloat() / icon.iconWidth, DEFAULT_ICON_SIZE.toFloat() / icon.iconHeight)

        IconUtil.scale(icon, null, scale)
    }
}

And changed other uses to utilize this
image

And finally I moved some of the Geode cli functions (eg. runGeodeCli(), or detectGeodeSdkVersion()) into their own little class src/main/kotlin/org/geodesdk/utils/GeodeUtils.kt so they are more reusable

Comment thread src/main/kotlin/org/geodesdk/newproject/GeodeNewProjectPanel.kt Outdated
Comment on lines +16 to +23
var sdkVersion = GeodeUtils.detectGeodeSdkVersion();

private val graph = PropertyGraph()
private val templateProp = graph.property(GeodeTemplate.DEFAULT)
private val repoInputProp = graph.property("")
private val nameProp = graph.property("")
private val versionProp = graph.property("v1.0.0")
private val developerProp = graph.property(detectGeodeDefaultDeveloper() ?: "")
private val developerProp = graph.property(sdkVersion ?: "")
Copy link
Copy Markdown

@SorkoPiko SorkoPiko Mar 17, 2026

Choose a reason for hiding this comment

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

Suggested change
var sdkVersion = GeodeUtils.detectGeodeSdkVersion();
private val graph = PropertyGraph()
private val templateProp = graph.property(GeodeTemplate.DEFAULT)
private val repoInputProp = graph.property("")
private val nameProp = graph.property("")
private val versionProp = graph.property("v1.0.0")
private val developerProp = graph.property(detectGeodeDefaultDeveloper() ?: "")
private val developerProp = graph.property(sdkVersion ?: "")
var sdkVersion = GeodeUtils.detectGeodeSdkVersion();
var defaultDeveloper = GeodeUtils.detectGeodeDefaultDeveloper();
private val graph = PropertyGraph()
private val templateProp = graph.property(GeodeTemplate.DEFAULT)
private val repoInputProp = graph.property("")
private val nameProp = graph.property("")
private val versionProp = graph.property("v1.0.0")
private val developerProp = graph.property(defaultDeveloper ?: "")

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

oh prevter beat me 😭 mb

@dom3
Copy link
Copy Markdown
Contributor Author

dom3 commented Mar 17, 2026

You were correct I missed that part.

@dom3
Copy link
Copy Markdown
Contributor Author

dom3 commented Mar 18, 2026

I added a snippet to the README.md just because the vscode one has the same thing

@dom3 dom3 requested a review from prevter March 18, 2026 02:45
@dom3
Copy link
Copy Markdown
Contributor Author

dom3 commented Mar 18, 2026

Im genuinely so sorry for how terribly I have managed this pull request. I think I have fixed everything and will do better in the future

@SorkoPiko
Copy link
Copy Markdown

dude it's chill 😅 it was just a simple issue, this is a cool PR. don't feel discouraged by a single mistake.

@prevter prevter merged commit 09bb6b4 into geode-sdk:main Mar 18, 2026
1 check passed
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.

3 participants