Skip to content

add manual input to graphical select widget#285

Merged
Vegita2 merged 3 commits into
masterfrom
278-manual-input
Aug 25, 2023
Merged

add manual input to graphical select widget#285
Vegita2 merged 3 commits into
masterfrom
278-manual-input

Conversation

@Vegita2
Copy link
Copy Markdown
Member

@Vegita2 Vegita2 commented Aug 6, 2023

fixes #278

Added input so types from mods or missing ones can still be used.

image

Not added for ScalableProp and ItemDestruct because they have additional logic for setting a value

@Vegita2 Vegita2 requested a review from 2767mr August 6, 2023 15:07
Copy link
Copy Markdown
Member

@2767mr 2767mr left a comment

Choose a reason for hiding this comment

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

Looks good, although I would have expected it to be above the filter or even in the same textbox

Copy link
Copy Markdown
Member

@lexisother lexisother left a comment

Choose a reason for hiding this comment

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

I'd prefer if this PR also added support for ScalableProp and ItemDestruct before it is merged.

moved manual input outside of image select widget
@Vegita2
Copy link
Copy Markdown
Member Author

Vegita2 commented Aug 23, 2023

I'd prefer if this PR also added support for ScalableProp and ItemDestruct before it is merged.

ItemDestruct is supported.
ScalableProp uses a complex object which can't be easily edited with a single input field. You can add a new issue for it if you want it.

@Vegita2 Vegita2 mentioned this pull request Aug 24, 2023
@Vegita2 Vegita2 merged commit ee74048 into master Aug 25, 2023
@Vegita2 Vegita2 deleted the 278-manual-input branch August 25, 2023 15:59
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.

Graphical Selection: Add support for manual entry

3 participants