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

fix(commands.lua): fix function close_others #882

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

minhtri2710
Copy link

image
I encountered an error because item.id in state.components is no longer suitable for tabclose to use. I changed the code to be able to close the tabs before and after the current tab. I use Windows OS.

if i ~= index then
delete_element(id)
else
id = 2
Copy link
Owner

Choose a reason for hiding this comment

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

I don't understand what this line of code is trying to do

Copy link
Owner

@akinsho akinsho left a comment

Choose a reason for hiding this comment

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

@minhtri2710 thanks for the PR, can you explain in more detail exactly what you are trying to fix i.e. a step by step of when this issue occurs and why you chose a specific solution so it's clear what is happening

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