Skip to content

Conversation

@jamesobutler
Copy link
Contributor

@jamesobutler jamesobutler commented Sep 17, 2025

This adds templating support for defining the "APPLICATION_DISPLAY_NAME" which is compatible with the changes in Slicer/Slicer#8731.

Unlike "APPLICATION_NAME", "APPLICATION_DISPLAY_NAME" can contain space characters.

Compatible with Slicer/Slicer@8e30ae8.

Unlike "APPLICATION_NAME", "APPLICATION_DISPLAY_NAME" can contain space characters.

Compatible with Slicer/Slicer@8e30ae8.
@jamesobutler jamesobutler force-pushed the application-display-name branch from ae7a8c3 to 6249b86 Compare September 18, 2025 22:01
@jcfr jcfr linked an issue Sep 18, 2025 that may be closed by this pull request
@jcfr
Copy link
Collaborator

jcfr commented Sep 18, 2025

@jamesobutler Is this ready for integration ?

@jamesobutler
Copy link
Contributor Author

Yes. I had just updated the last commit with some files specific to custom apps so that they would also use the applicationDisplayName property. I confirmed these changes using my own Slicer custom app. Should be good to go 👍🏻

@jcfr jcfr merged commit a84a5fc into KitwareMedical:main Sep 18, 2025
4 checks passed
@jamesobutler jamesobutler deleted the application-display-name branch September 18, 2025 22:22
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.

Unable to configure project with name that contains a space

2 participants