[GitHub] [superset] stephenLYZ commented on pull request #17976: feat(Select): [cosmetic] support allowTopOptions for select component

[GitBox <gi...@apache.org>]

stephenLYZ commented on pull request #17976:
URL: https://github.com/apache/superset/pull/17976#issuecomment-1008951962


   @michael-s-molina @geido Thanks for the explanations! I also
   agree keeping the behavior consistent. btw, I am curious  why single-select component would keep selected option on top. It is a bit different from other site. Maybe scroll to selected option could be better? Since after each selection of value will lead to the reorder of the list, if the list is very large there maybe some performance issues.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org