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

Add focusOptions for sibling functions #23

Closed
wants to merge 11 commits into from
Closed

Conversation

Kulevw
Copy link

@Kulevw Kulevw commented Mar 17, 2021

I use focus-lock for focus management in modal dialogs, but focus-lock have helpers functions for focus management and I use sibling functions for custom select input

@codecov
Copy link

codecov bot commented Mar 17, 2021

Codecov Report

Merging #23 (36eaa9c) into master (125a50b) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #23   +/-   ##
=======================================
  Coverage   82.69%   82.69%           
=======================================
  Files          20       20           
  Lines         312      312           
  Branches       68       68           
=======================================
  Hits          258      258           
  Misses         49       49           
  Partials        5        5           
Impacted Files Coverage Δ
src/sibling.ts 91.30% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 125a50b...36eaa9c. Read the comment docs.

.DS_Store
Copy link
Owner

Choose a reason for hiding this comment

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

please gitignore the dist back

"name": "focus-lock",
"version": "0.8.1",
"name": "@kulevw/focus-lock",
"version": "0.8.2",
Copy link
Owner

Choose a reason for hiding this comment

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

please do not change version manually. (as well as change the name of this package)

/**
* options for focus call
*/
focusOptions?: FocusOptions;
Copy link
Owner

Choose a reason for hiding this comment

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

Why I remember doing this change 😅

Copy link
Owner

Choose a reason for hiding this comment

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

@@ -29,6 +29,10 @@ interface FocusNextOptions {
* @default true
*/
cycle?: boolean;
/**
Copy link
Owner

Choose a reason for hiding this comment

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

please make this change the single change in this PR

@@ -29,6 +29,10 @@ interface FocusNextOptions {
* @default true
*/
cycle?: boolean;
/**
* options for focus call
Copy link
Owner

@theKashey theKashey Mar 18, 2021

Choose a reason for hiding this comment

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

Suggested change
* options for focus call
* Optionally, can pass focus options to control the focus more precisely (ie. `{ preventScroll: true }`)

@theKashey
Copy link
Owner

Implemented in v0.9.0

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.

None yet

2 participants