Skip to content
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

Branch Search Hidden Input #4983

Closed
navtoj opened this issue Sep 27, 2024 · 0 comments · Fixed by #4984
Closed

Branch Search Hidden Input #4983

navtoj opened this issue Sep 27, 2024 · 0 comments · Fixed by #4984
Labels
bug Something isn't working 🎉reproduced🎉 The issue could be reproduced by following the instructions UI Any broken visuals like layouts, colors etc.

Comments

@navtoj
Copy link
Contributor

navtoj commented Sep 27, 2024

Version

0.12.26 (20240924.135933)

Operating System

Mac OS X

Distribution Method

dmg (Apple Silicon)

Describe the issue

It is possible to search through the branches even when the search input is not visible.

Branch.Search.Hidden.Input.mov

How to reproduce

Click within the area marked in the screenshot below and start typing.

Branch Search Hidden Input Area

Expected behavior

The user should not be able to access the search input when it's in a hidden state.

Relevant source code

let searching = $state(false);

<div class="search-container" class:show-search={searching}>
<button tabindex={searching ? -1 : 0} class="search-button" onclick={toggleSearch}>
<Icon name={searching ? 'cross' : 'search'} />
</button>
<input
bind:this={searchEl}
bind:value={$searchTerm}
class="search-input text-13"
type="text"
placeholder="Search branches"
onkeydown={handleSearchKeyDown}
/>
</div>

.show-search {
width: 100%;
& .search-button::after {
transform: scale(0.7);
}
& .search-input {
opacity: 1;
}
}

@navtoj navtoj added the bug Something isn't working label Sep 27, 2024
@Byron Byron added UI Any broken visuals like layouts, colors etc. 🎉reproduced🎉 The issue could be reproduced by following the instructions labels Sep 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working 🎉reproduced🎉 The issue could be reproduced by following the instructions UI Any broken visuals like layouts, colors etc.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants