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

Allow inserting hologram lines at the end of the list #268

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

Conversation

NonSwag
Copy link

@NonSwag NonSwag commented Dec 14, 2024

Closes #267
Previously, inserting at the end of the hologram required workarounds as the index check did not permit it. This change updates the logic to allow an index equal to size(), enabling straightforward insertion at the end. It improves functionality and aligns with expected behavior.

Previously, inserting at the end of the hologram required workarounds as the index check did not permit it. This change updates the logic to allow an index equal to `size()`, enabling straightforward insertion at the end. It improves functionality and aligns with expected behavior.
Replaced redundant code in the addLine method with a call to insertLine to improve code clarity and maintainability. This ensures consistent behavior and reduces duplication, streamlining future updates.
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.

Can't insert hologram lines at the end using the index
1 participant