-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add note in settings to prevent premature uninstalls #312
base: dev
Are you sure you want to change the base?
Conversation
jaretpr
commented
Aug 12, 2024
- Created a section within the settings to indicate to users that spawn times may not be immediate.
v1.14.0 update
v1.14.1 update
v1.14.2 update
v1.14.3 update
v1.14.4 update
v1.14.5 update
v1.15.0 update
v1.15.1 update
- Created a section within the settings to indicate to users that spawn times may not be immediate.
Hello @jaretpr, thank you for your contribution. It seems a bit out of place to place a note at the bottom of the settings page. Especially since you've placed it in the developer form component, this is not related to that component. I'm also not sure if the bottom of the settings page is the best place to mention this. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wouldn't leave these changes in this file.
<div data-orientation="horizontal" role="none" className="bg-border h-[1px] w-full"> | ||
<span className="select-none invisible"></span> | ||
</div> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use the installed separator component for this.