-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Feat(#1093) Added ability to open tab context menu #1105
Conversation
…new-context-menu-for-tabs
Thanks for working on this. Could you add "Close all" and "Close all to the right"? Currently there's no easy way to close all open tabs. |
…new-context-menu-for-tabs
Hello @helloanoop. For information, I updated the branch and carried out the tests. The feature still works properly and is still ready to merge. Once merged, it would add many features to this tab context menu. |
Closes: #715 |
Closing this PR as the Tab Context Menu was implemented in #2183 |
Hello,
I am submitting this PR to initiate context menus for the tabs.
Description
As of today, I have added two simple features: New Request and Tab Clone.
It took me some time to find the right approach and process. In my opinion, it is quite comprehensible, relies on the state of the tabs, and respects the "separation of concerns" principle, particularly in event management.
If this PR is accepted, a significant milestone, which I believe is highly expected, will be managing tab closures (individually, all except the current one, etc.). The challenge will lie in controlling the "draft mode" with control modals.
Please let me know your feedback if you think of a different approach.
Contribution Checklist:
Issue : #1093