Add toggle-reverse described in issue 16

[Kotsuha]

Describe the pull request

Implement the feature described in issue #16.

Screenshots

If applicable, add screenshots to help explain what is being modified.

This is draft. For example, I see there are test files. I did not write test for this new feature. Feel free to modify anything.

[HiDeoo]

Thanks for the contribution, super appreciated 🙌

This looks pretty straightforward and working as expected, I just added the following changes:

• Refactor to use an explicit direction (forward or backward) instead of a boolean reverse flag.
• Add a test to ensure it's working and properly loop back to the end of the list.
• Update the CHANGELOG file.

Thanks again.

[Kotsuha]

@HiDeoo
Not sure if you'll see this but hey :D
I'd like to know your thought on this refactor commit. I just found myself writing similar code again in my project, and it reminds me of this commit made by you.

This is my code:

private void OnEditButtonClick()
{
    RequestOpenSession(true).Forget(); // <------ THIS LINE
}

private async UniTask RequestOpenSession(bool askPassword) // <------ THIS LINE
{
    // ...
}

askPassword sounds like a true or false, so I just use a boolean. But then the readability is not good. It's similar to this commit. I'd like to know if it's you who is writing the code, will you write it the other way?

Yes, I intentionally ask for a code review, sorry. You can ignore this message if you don't feel like it.

[HiDeoo]

Sorry for the late reply, did not see your message earlier.

I think there is a difference between your case and the one from this PR you linked. The direction explicitely had 2 options that be easily described (forward and backward) whereas in your case, it may not be that explicit with the use of a password or not.

It's difficult to say without knowing or writing the code, but my first guess would be, if this happens more than a few times, to not use RequestOpenSession(true) and maybe have an extra wrapper function, e.g. RequestOpenSessionWithPassword() which would be way more explicit for the reader of the code that would call the underlying function with the correct parameter.