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

Config option to insert new item at start of list #72

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

siahgoodwork
Copy link

In our use case, on certain content displays it is useful to have new items appear at the start of the list by default, instead of the end as it is currently.

I also added a button to push item to the start on the "Toggle Increment" buttons.

@SimeonGriggs
Copy link
Collaborator

Hey @siahgoodwork good idea.

  1. Could you update the README to include instructions on how to use this
  2. Consider instead of newItemAtStart, a more generic newItemPosition which defaults to after and can be set to before | after

@siahgoodwork
Copy link
Author

siahgoodwork commented Feb 16, 2024

Hi @SimeonGriggs I notice the logic is already modified to your suggestion after doing an edit on my end 🤣
Have added a bit on the README to document this option and kept the logic that's already on main which is more elegant.

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.

2 participants