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

Better on mobile and better about which option is active #103

Merged
merged 11 commits into from
Sep 3, 2022

Conversation

janosh
Copy link
Owner

@janosh janosh commented Sep 2, 2022

  • determine active option not by label but by index in array of matchingOptions. Before identical options would all become active simultaneously when hovering any one of them.
  • replace setOptionsVisible(bool) with open_dropdown() and close_dropdown()
  • trigger close_dropdown() on:touchstart outside outerDiv (as well as on:click outside), Multiselect wasn't closing on mobile without that
  • rename prop showOptions to open
  • add props focusInputOnSelect: boolean | 'desktop' = 'desktop' and breakpoint = 800

…gOptions

replace setOptionsVisible(bool) with open_dropdown() and close_dropdown()
trigger close_dropdown() on touchstart outside (as well as click outside), select wasn't closing on mobile without that
rename prop showOptions to open
add props focusInputOnSelect: boolean | 'desktop' and breakpoint = 800
until sveltejs/kit#6462 resolves
static

> Using @sveltejs/adapter-static fails with error
  @sveltejs/adapter-static: all routes must be fully prerenderable (unless using the 'fallback' option — see https://github.com/sveltejs/kit/tree/master/packages/adapter-static#spa-mode). Try adding `export const prerender = true` to your root layout — see https://kit.svelte.dev/docs/page-options#prerender for more details

run CI tests on both linux and windows using matrix strategy
@janosh janosh merged commit 86a1d4b into main Sep 3, 2022
@janosh janosh deleted the better-on-mobile branch September 3, 2022 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant