Skip to content

Fix #244#247

Merged
tomlin7 merged 3 commits intotomlin7:mainfrom
vyshnav-vinod:fix/multiple-tabs-of-same-file
Mar 18, 2024
Merged

Fix #244#247
tomlin7 merged 3 commits intotomlin7:mainfrom
vyshnav-vinod:fix/multiple-tabs-of-same-file

Conversation

@vyshnav-vinod
Copy link
Copy Markdown
Contributor

@vyshnav-vinod vyshnav-vinod commented Mar 18, 2024

fix #244

Fixed the bug of a new tab being opened while clicking a file who already has a tab opened in the editor

vyshnav-vinod and others added 2 commits March 18, 2024 14:51
Fixed issue causing a new tab to be opened when selecting a file who already has a tab opened tomlin7#244
@vyshnav-vinod
Copy link
Copy Markdown
Contributor Author

Hey @billyeatcookies it would be also better to maybe make a requirements_dev.txt for people who wish to contribute because the current requirements.txt doesnt include pytest and python-dotenv. I tried to create a new requirements.txt myself but i guess because of tkextrafont i got some extra packages and dependencies. Hence it would be better for you to create it

Copy link
Copy Markdown
Owner

@tomlin7 tomlin7 left a comment

Choose a reason for hiding this comment

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

@vyshnav-vinod the requirements-dev.txt idea is nice but since biscuit is not planned to be published on pypi I just didn't give it much of a thought, currently the requirements.txt serves purpose of both type of (dev + build) environments.

Changes to the code:

  • since open_editor is the main interface to opening existing files from directory tree, that can be modified, so let's leave the add_editor function as it is rn.

@tomlin7 tomlin7 merged commit 176e27d into tomlin7:main Mar 18, 2024
@vyshnav-vinod
Copy link
Copy Markdown
Contributor Author

The current requirements.txt doesn't include pytest and python-dotenv

@vyshnav-vinod vyshnav-vinod deleted the fix/multiple-tabs-of-same-file branch March 19, 2024 04:43
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.

Multiple tabs of the same file is opened when clicked on the file again from the explorer

2 participants