Skip to content

just passed props down and small formatting#279

Closed
arausly wants to merge 5 commits intodevelopfrom
feature/supportProjectVariableAutocompletion-CMEM-5572
Closed

just passed props down and small formatting#279
arausly wants to merge 5 commits intodevelopfrom
feature/supportProjectVariableAutocompletion-CMEM-5572

Conversation

@arausly
Copy link
Copy Markdown
Contributor

@arausly arausly commented May 28, 2025

No description provided.

/** Optional height of the component */
height?: number | string;
/** Set read-only mode. Default: false */
readOnly?: boolean;
Copy link
Copy Markdown
Member

@haschek haschek Jun 4, 2025

Choose a reason for hiding this comment

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

readOnly can be set but then it is not used, is it?

/** Optional height of the component */
height?: number | string;
/** Set read-only mode. Default: false */
readOnly?: boolean;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

same here.

showScrollBar={showScrollBar}
multiline={multiline}
onMouseDown={handleInputMouseDown}
height={height}
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

height is set via <CodeAutocompleteField /> in Silk?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes, for one of the editors. also readOnly in certain cases. I noticed that <CodeAutoCompleteField /> didn't accept them when it could be configured via<CodeEditor />

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

But how readOnly forwarded if it is not used in the code (hight is used)), not directly, not via something like ...otherProps, have I overseen something?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I have made the modifications, thanks a lot

@arausly arausly closed this Jun 6, 2025
@arausly arausly deleted the feature/supportProjectVariableAutocompletion-CMEM-5572 branch June 6, 2025 10:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants