Skip to content

[Bug]: Improve the settings UI for the Language Chat Transform Plugi…#4689

Merged
hansva merged 1 commit into
apache:mainfrom
ep9io:pr/4688-langchat-ui-fixes
Dec 10, 2024
Merged

[Bug]: Improve the settings UI for the Language Chat Transform Plugi…#4689
hansva merged 1 commit into
apache:mainfrom
ep9io:pr/4688-langchat-ui-fixes

Conversation

@ep9io
Copy link
Copy Markdown
Contributor

@ep9io ep9io commented Dec 10, 2024

…n. #4688

Fixed the unnecessary grey areas. Some were caused by inputs in Anthropic section that haven't yet been implemented. They are commented out at the moment as TODO items as they are placeholders for two features that need to be re-evaluated again (seed and stop-sequences).

Below is a screenshot of how the UI looks now.

image

Also added the icon to the right of the transform name to make it more eye-candy.

image


Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

  • Run mvn clean install apache-rat:check to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
  • If you have a group of commits related to the same change, please squash your commits into one and force push your branch using git rebase -i.
  • Mention the appropriate issue in your description (for example: addresses #123), if applicable.

To make clear that you license your contribution under the Apache License Version 2.0, January 2004
you have to acknowledge this by using the following check-box.

@hansva hansva force-pushed the pr/4688-langchat-ui-fixes branch from 00a3249 to 6d9c4ed Compare December 10, 2024 14:55
@hansva hansva merged commit 2a0fc29 into apache:main Dec 10, 2024
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