Skip to content

Gmic improvements#1254

Merged
ddennedy merged 1 commit into
masterfrom
gmic_improvements
Jun 10, 2026
Merged

Gmic improvements#1254
ddennedy merged 1 commit into
masterfrom
gmic_improvements

Conversation

@bmatherly

Copy link
Copy Markdown
Member

No description provided.

Comment thread src/modules/openfx/openfx/include/nuke/fnPublicOfxExtensions.h
Comment thread src/framework/mlt_property.c Outdated
@bmatherly bmatherly requested a review from ddennedy June 8, 2026 02:15
@bmatherly bmatherly marked this pull request as draft June 8, 2026 02:24
@ddennedy

ddennedy commented Jun 8, 2026

Copy link
Copy Markdown
Member

Besides a UI improvement is there some difference in image effect with this?

Read OFX layout hints and propagate them into generated parameter metadata
so UIs can better preserve plugin-authored control layout.

- map OFX layout hint values to metadata:
  - no_new_line
  - divider
- mark label-style string params as hide-label in addition to readonly
- extend metadata schema with:
  - layout-hint enum
  - hide-label boolean (default: false)
- include the Nuke OFX public extensions header for layout-hint constants

Downloaded from
https://github.com/NatronGitHub/openfx/blob/master/include/nuke/fnPublicOfxExtensions.h
@bmatherly bmatherly force-pushed the gmic_improvements branch from de2c0dd to cd75009 Compare June 8, 2026 03:23
@bmatherly

Copy link
Copy Markdown
Member Author

I removed the animation related changes.

@ddennedy ddennedy added this to the v7.40.0 milestone Jun 9, 2026
@bmatherly

Copy link
Copy Markdown
Member Author

Besides a UI improvement is there some difference in image effect with this?

No. It just improves the UI controls/widgets.

@ddennedy ddennedy marked this pull request as ready for review June 10, 2026 05:27
@ddennedy ddennedy merged commit a9c4772 into master Jun 10, 2026
23 of 25 checks passed
@ddennedy ddennedy deleted the gmic_improvements branch June 10, 2026 05:27
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.

2 participants