Skip to content

improve: script editor search results can be resized#7626

Merged
vadi2 merged 8 commits intoMudlet:developmentfrom
ZookaOnGit:splitter-2905
Jan 5, 2025
Merged

improve: script editor search results can be resized#7626
vadi2 merged 8 commits intoMudlet:developmentfrom
ZookaOnGit:splitter-2905

Conversation

@ZookaOnGit
Copy link
Copy Markdown
Contributor

Brief overview of PR changes/additions

Allow the script editor search results to be resized. It can be resized all the way to hidden.

Motivation for adding to Mudlet

Improves user experience.

Other info (issues closed, discussion etc)

Wondering if the toggle show/hide results button is really necessary now?

Peek.2025-01-01.16-59.webm

closes #2905
/claim #2905

@ZookaOnGit ZookaOnGit requested a review from a team as a code owner January 1, 2025 13:03
@add-deployment-links
Copy link
Copy Markdown

add-deployment-links bot commented Jan 1, 2025

Hey there! Thanks for helping Mudlet improve. 🌟

Test versions

You can directly test the changes here:

No need to install anything - just unzip and run.
Let us know if it works well, and if it doesn't, please give details.

Copy link
Copy Markdown
Member

@vadi2 vadi2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree, the show/hide button can probably go now.

Does the splitter remember where it was across Mudlet restarts?

@ZookaOnGit
Copy link
Copy Markdown
Contributor Author

Does the splitter remember where it was across Mudlet restarts?

no, it will always start at just showing the search term input bar

would you like that feature?

@vadi2
Copy link
Copy Markdown
Member

vadi2 commented Jan 1, 2025

Would be nice, what do you think?

The default size on Appimage is a bit small btw:
image

@ZookaOnGit ZookaOnGit marked this pull request as draft January 1, 2025 16:57
@ZookaOnGit ZookaOnGit marked this pull request as ready for review January 1, 2025 17:34
@ZookaOnGit
Copy link
Copy Markdown
Contributor Author

added suggestions

@vadi2 vadi2 merged commit 4e465d3 into Mudlet:development Jan 5, 2025
@ZookaOnGit ZookaOnGit deleted the splitter-2905 branch January 15, 2025 01:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add splitter to increase vertical size of editor search area

2 participants