Skip to content

Modal toside#613

Open
Ladzsz wants to merge 3 commits intoEXXETA:mainfrom
Ladzsz:modalToside
Open

Modal toside#613
Ladzsz wants to merge 3 commits intoEXXETA:mainfrom
Ladzsz:modalToside

Conversation

@Ladzsz
Copy link
Copy Markdown
Contributor

@Ladzsz Ladzsz commented Nov 6, 2025

Changes

moved modal logic to a component that is now rendered inline. changes for issue #543

Describe the changes of your PR here
now when rename is clicked (currently only for requets an input pops up instead of the modal.

(NOTE ENSURE TO READ i did not do it for the folders yet as i want to ensure the current version of it is too the hosts liking first before doing that. please let me know if you want changes to the current system before i ensure it works for folders as well.)

Testing

upon turning off program and turning it on changes stay
If relevant, mention your manual testing here. If possible, include screenshots.

Checklist

  • [ yes] Issue has been linked to this PR
  • [yes ] Code has been reviewed by person creating the PR
  • Automated tests have been written, if possible
  • [ yes] Manual testing has been performed
  • Documentation has been updated, if necessary
  • Changes have been reviewed by second person

@SoulKa SoulKa requested a review from giemic8 November 10, 2025 08:08
@Ladzsz
Copy link
Copy Markdown
Contributor Author

Ladzsz commented Dec 12, 2025

became really busy with alot of things will be going on a hiatus.

@Ladzsz
Copy link
Copy Markdown
Contributor Author

Ladzsz commented Jan 20, 2026

I AM back!!!

does this issue still need reviewing? should i wait for review or should i work on another issue?

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