Skip to content

feat: Add Input field next to SliderInput#1295

Open
sub-hub wants to merge 3 commits intokwaroran:mainfrom
sub-hub:sliderInputField
Open

feat: Add Input field next to SliderInput#1295
sub-hub wants to merge 3 commits intokwaroran:mainfrom
sub-hub:sliderInputField

Conversation

@sub-hub
Copy link
Copy Markdown
Collaborator

@sub-hub sub-hub commented Mar 1, 2026

PR Checklist

  • Required Checks
    • Have you added type definitions?
    • Have you tested your changes?
    • Have you checked that it won't break any existing features?
  • If your PR uses models1, check the following:
    • Have you checked if it works normally in all models?
    • Have you checked if it works normally in all web, local, and node-hosted versions? If it doesn't, have you blocked it in those versions?
  • If your PR is highly AI generated2, check the following:
    • Have you understood what the code does?
    • Have you cleaned up any unnecessary or redundant code?
    • Is it not a huge change?
      • We currently do not accept highly AI generated PRs that are large changes.

Summary

chrome_fVwOL53XjB.mp4

This PR changes src/lib/SettingRender.svelde to create an input next to the slider when you render the slider.

Related Issues

There are many people who have difficulty adjusting the number of sliders in detail at the moment.

Impact

Now that src/lib/Setting/SettingRender.svelde has been modified, the input will be added next to all sliders shown with this renderer in the future.

Footnotes

  1. Modifies the behavior of prompting, requesting, or handling responses from AI models.

  2. Over 80% of the code is AI generated.

@sub-hub sub-hub marked this pull request as ready for review March 2, 2026 06:16
@sub-hub
Copy link
Copy Markdown
Collaborator Author

sub-hub commented Mar 27, 2026

conflict resolved.
Previously, I modified SettingRender, but as the components were separated from this file, I now only modified SettingSlider file.

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.

1 participant