Skip to content

Conversation

@ggbhat
Copy link
Contributor

@ggbhat ggbhat commented Feb 26, 2020

Fixes - #56
Jupyter Lab Issue - 6617

Added type search to input to show clear search icon. It will solve usability issue by user should be able to use mouse click to clear search text.

searchwithclear1

@blink1073
Copy link
Contributor

cc @tgeorgeux

@vidartf
Copy link
Member

vidartf commented Feb 28, 2020

What happened to our binder bot? :(

I cannot see such a cross when testing here: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input/search (Win 10, Firefox).

@ggbhat
Copy link
Contributor Author

ggbhat commented Mar 1, 2020

@vidartf Yeah, clear search is not available for firefox. Need to add custom clear button then.

@ggbhat
Copy link
Contributor Author

ggbhat commented May 15, 2020

Added custom clear search button since input type search is not works on Firefox.

@jasongrout
Copy link
Contributor

Added custom clear search button since input type search is not works on Firefox.

input type search definitely works in Firefox, since Firefox 4, IIRC. However, having a clear button is one of the differences between browsers: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input/search#Differences_between_search_and_text_types

@jasongrout
Copy link
Contributor

See also https://bugzilla.mozilla.org/show_bug.cgi?id=558594 (which has seen recent work) and https://stackoverflow.com/a/46687047

@tgeorgeux
Copy link

tgeorgeux commented May 17, 2020

I am overtly +1 on this, provided it's technically sound, (which I am far from qualified to assess).

Copy link
Contributor

@blink1073 blink1073 left a comment

Choose a reason for hiding this comment

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

Thanks!

@blink1073 blink1073 merged commit 7831eb2 into jupyterlab:master May 18, 2020
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants