Skip to content

feat: add default value support to alertInput and related components - #1211

Merged
kwaroran merged 1 commit into
kwaroran:mainfrom
tammy3211:add-alertInput-defaultValue
Jan 30, 2026
Merged

kwaroran merged 1 commit into
kwaroran:mainfrom
tammy3211:add-alertInput-defaultValue

Conversation

@tammy3211

Copy link
Copy Markdown
Contributor

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, if true, 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 understanded what the code does?
    • Have you cleaned up any unnecessary or redundant code?
    • Is is not a huge change?
      • We currently do not accept highly ai generated PRs that are large changes.

Summary

The defaultValue property was added to alertInput to allow pre-entering of a word.

Related Issues

During testing, pressing the Esc key occasionally resulted in 0 being entered. This is a bug caused by interaction with the Esc button, and will need to be fixed later, but is not covered in this PR.

Changes

A defaultValue property was added to alertInput, and some elements that use alertInput were modified to use defaultValue. Key and password fields were not touched.

Impact

For example, when renaming bookmarks, folders, etc., you can edit the desired field instead of renaming them from beginning. This feature can also be used in other development projects.
image

Additional Notes

This feature needs to be modified to work with v2trigger and lua, but since another PR #998 is currently in progress, I'll rewrite this later.

Footnotes

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

  2. Almost over 80% of the code is ai generated. ↩

@kwaroran
kwaroran merged commit 0b9e91e into kwaroran:main Jan 30, 2026
5 checks passed
@tammy3211
tammy3211 deleted the add-alertInput-defaultValue branch January 30, 2026 11:43
yor42 pushed a commit to yor42/RisuTanium that referenced this pull request Sep 21, 2026
…waroran#1211)

## PR Checklist

- Required Checks
    - [x] Have you added type definitions?
    - [x] Have you tested your changes?
    - [x] Have you checked that it won't break any existing features?
- [ ] If your PR uses models[^1], if true, 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 generated[^2], check the following:
    - [ ] Have you understanded what the code does?
    - [ ] Have you cleaned up any unnecessary or redundant code?
    - [ ] Is is not a huge change?
- We currently do not accept highly ai generated PRs that are large
changes.


[^1]: Modifies the behavior of prompting, requesting or handling
responses from ai models.
[^2]: Almost over 80% of the code is ai generated.

## Summary

The defaultValue property was added to alertInput to allow pre-entering
of a word.

## Related Issues

During testing, pressing the Esc key occasionally resulted in 0 being
entered. This is a bug caused by interaction with the Esc button, and
will need to be fixed later, but is not covered in this PR.

## Changes

A defaultValue property was added to alertInput, and some elements that
use alertInput were modified to use defaultValue. Key and password
fields were not touched.

## Impact

For example, when renaming bookmarks, folders, etc., you can edit the
desired field instead of renaming them from beginning. This feature can
also be used in other development projects.
<img width="248" height="197" alt="image"
src="https://github.com/user-attachments/assets/3cff31b6-7b2b-4b2e-92c8-0622377dcca2"
/>

## Additional Notes

This feature needs to be modified to work with v2trigger and lua, but
since another PR kwaroran#998 is currently in progress, I'll rewrite this later.
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