Comments (7)
sorry, was on vacation most of last week. I'll get a PR open early this week.
from typeahead.
Great discussion. I agree very much with getting your changes merged in @pwelter34.
I think @vertonghenb comments above make a lot of sense. I also have a suggestion or two but perhaps it would be best to get the PR raised then we can sort things out on there?
My main concern though is keeping the public API intact.
What do you both think?
from typeahead.
@pwelter34 Paul, looks great! If I knew I wouldn't have started the last PR. Maybe we can combine the 2? Personally, I don't like the search bar, but you'll probably need that for multi values. The other stuff looks great especially the 2 controls merged into 1. I'm willing to abandon the last PR but then we need to make some design decisions.
What I like
- Arrow key functionality
- the overall styling, especially the grey indicator on the selected value(s)
- The initial list of items
- 1 control for Form and Standalone (kudos for that one)
- The Convert Method
What needs to be improved:
- Escape / Tab functionality
- Searchbar, can't we have this in the same input control as the selected value(s)?
- Arrow Keys in big lists so that the scrollbar navigates with it (might be easier to focus the next element, then the scrollbar is handled by the browser)
- Clear button, I think it's currently not working for multi-select.
Just my 2 cents, @chrissainty what do you think?
from typeahead.
@vertonghenb thanks for feedback.
- I'm neutral in the look and feel. I personal don't like the search covering the current selected value, but most typeahead do it that way so i'm willing to change it.
- My version supports keyboard selection with up and down arrows as well as enter to select. I can add escape support.
- I fixed the clear button on multiselect. It was an issue in the demo code, not the control.
If @chrissainty agrees, i can merge in my stuff. I'll refactor to match public property names as much as possible. I think having one good version is best for community.
~ Paul
from typeahead.
@pwelter34
Agreed, one good version should be best for the community.
Therefore I believe it's best to merge in your changes with the following improvements:
- 1 control to search/show the values, I think it's the way to go and makes the design cleaner, you can check out the Sync Fusion Control, which does this.
- Support Escape, you can check out the escape fix I made using the event system and Interop class.
- Support Tab, currently when you tab into the search, you proceed to the next form control. I think it would be better to go to the first option in the list, basically adding a
tabindex=0
should suffice. However this is subjective so you can do whatever you like with this.
from typeahead.
@pwelter34 any update on this? You can reach out to me on Gitter if you need something.
from typeahead.
@pwelter34, how is the PR coming along? Do you need help with anything?
from typeahead.
Related Issues (20)
- [Question] Is there a way to programmatically select multiple items? HOT 1
- [Bug] When used in a child component in multiselect mode items not added when selected HOT 1
- Have a OnClose Option for the Dropdown list
- [Question] Are you able to just use strings for Value and date related bindings on the control?
- Two way binding
- Typeahead search text dissapearing from search box after displaying results in v 4.7.0 HOT 14
- [Feature Request] Option to disable deleting items with "Backspace" when using MultiSelect
- Method to Force results dropdown to close?
- how to Enable mutli line
- Laptop touchpad not selecting item in results
- Error in "Using complex types but only binding to a single property" HOT 1
- [Bug] - blazored-typeahead results hides behind the modal-footer
- [Bug] Typeahead inside Blazored.Modal search text erase with .NET 7.0 HOT 3
- [Question] How to Justify Dropdown Content
- [Bug] ArgumentNullException after pressing 'backspace' key
- Clicking typeahead inside MudTable in <MudTd> by MudBlazor not set to Focus
- [Bug] Multiple Pasting of text into the control causes app pool to crash
- WORK PAUSED - PLEASE READ BEFORE POSTING AN ISSUE HOT 1
- how to set category id for Searchmethod HOT 1
- Scroll in dropdown menu
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from typeahead.