Navigation Menu

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

Consoles get renamed when moved in the IPython console #6098

Closed
ccordoba12 opened this issue Dec 30, 2017 · 3 comments
Closed

Consoles get renamed when moved in the IPython console #6098

ccordoba12 opened this issue Dec 30, 2017 · 3 comments

Comments

@ccordoba12
Copy link
Member

If I rename a console foo and move it next to another one, their names get swapped.

@csabella
Copy link
Contributor

They don't need to be renamed for this to happen. Just swapping two tabs causes them to switch names.

@csabella
Copy link
Contributor

@ccordoba12 It appears this is a result of PR #5902. In tabs.py, PR 5902 commented out the code in mouseMoveEvent that created the QDrag object. Without the QDrag object, dragEnterEvent and dropEvent aren't called, which means the signals aren't emitted to move_tabs which does the renaming. editor.py uses the tabMoved() signal (which was added in Qt 4.5).

As you said in #5902, I think tabs.py should be updated to match editor.py, which would mean using tabMoved(). However there is also code in there to move tabs from one widget to another, so that would be lost without the custom drag and drop. Although, it's not working anyway, so maybe it wouldn't be an issue to lose that functionality?

@ccordoba12
Copy link
Member Author

Thanks Cheryl! I used your idea of switching to tabMoved to handle this.

However there is also code in there to move tabs from one widget to another, so that would be lost without the custom drag and drop

You're right. I think we should remove all that code for Spyder 4.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants