Skip to content

Comments

Editor action "move all others to other view"#2714

Merged
sunderme merged 3 commits intotexstudio-org:masterfrom
octaeder:moveOthers
Nov 27, 2022
Merged

Editor action "move all others to other view"#2714
sunderme merged 3 commits intotexstudio-org:masterfrom
octaeder:moveOthers

Conversation

@octaeder
Copy link
Contributor

@octaeder octaeder commented Nov 27, 2022

This PR closes #2713. Added action can be seen in editor tabs context menu:

grafik

@sunderme
Copy link
Member

hmmm, I am not a fan of filling menus/context menus with fringe functionality.
"move all others" is bascially "move all" and then "one back".
What can make some sense is "move all others" on the left view and "move all" on the right view.

@octaeder
Copy link
Contributor Author

octaeder commented Nov 27, 2022

Open a dozen documents (which may even have lengthy names or shrink window size) in the left view. Open context menu on some tab from the left and choose "move all to other view". How do you bring back this tab? This is likely not possible with two clicks. Reason is that after moving all tabs to the right view the last tab moved will be active. Thus some of the first tabs will not be visible (tabs move out to the left). You will have to scroll to the left or open a document list. And for this task you have to remember the title of the document (sorry if some of the documents are untitled) or its position. Thus you impose some restrictions to the user, consume user's memory and have a varying menu entry (same list position but changing name and function) which is irritating. What do you prefer? Let me know so I can do the changes if necessary.

@sunderme
Copy link
Member

I stick with my proposal.
move all on the left view is basically never useful, on the right "move all" serves to close the dual view set-up.

@octaeder
Copy link
Contributor Author

Further detail: Only change the menu of the left side (view) to show "Move all others to other view" replacing "Move all to other view", while right side continues to show "Move all to other view":

left view:
grafik

right view:
grafik

@sunderme
Copy link
Member

thanks

@sunderme sunderme merged commit 9f106bb into texstudio-org:master Nov 27, 2022
@octaeder octaeder deleted the moveOthers branch November 27, 2022 18:36
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.

Editor: add action "move all others to other view

2 participants