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

Windows: the X to close an editor should be on the right #7058

Closed
bpasero opened this issue May 31, 2016 · 7 comments
Closed

Windows: the X to close an editor should be on the right #7058

bpasero opened this issue May 31, 2016 · 7 comments
Assignees
Labels
on-testplan ux User experience issues workbench-tabs VS Code editor tab issues
Milestone

Comments

@bpasero
Copy link
Member

bpasero commented May 31, 2016

It is natural on Windows to have the X affordance to the right of the document and not to the left. This applies for both tabs enabled or disabled.

However, we should not just put the X to the right of the file label if you have tabs disabled because that means having an unstable location of the close button depending on the size of the file path. Instead, we should move this action to the far right, but clearly inside the tab and not outside. See #7060 for a related issue.

/cc @alexandrudima

@bpasero bpasero added ux User experience issues workbench-tabs VS Code editor tab issues labels May 31, 2016
@bpasero bpasero added this to the June 2016 milestone May 31, 2016
@bpasero bpasero self-assigned this May 31, 2016
@bpasero bpasero modified the milestones: Backlog, June 2016 May 31, 2016
@bpasero bpasero assigned stevencl and bgashler1 and unassigned bpasero May 31, 2016
@stevencl
Copy link
Member

stevencl commented Jun 1, 2016

We discussed at the UX meeting today and agreed that we would refine the button layout like so:

image

Brad will tidy up the mockups I created and copy them up to the deck with our visual designs.

@bgashler1
Copy link
Contributor

bgashler1 commented Jun 2, 2016

With Tabs

tabs-02a-light
tabs-02a

Tabs disabled

tabs-disabled-02a

@joaomoreno
Copy link
Member

  • How does it look when just one or two tabs are open? Do they stay that size, or do they grow?
  • How does it look when the view is split?
  • The second screenshot shows a very tiny src below the active tab, is that a glitch? How will we should the path of the file?
  • In the no-tabs design, the actions to the right look broken, almost to the point of screen cheese. If the user explicitly disables tabs, why not simply remove that notion and just put the x button all the way to the far end of the actions and remove the lighter (darker, in light theme) grey background colour?

@bpasero
Copy link
Member Author

bpasero commented Jun 2, 2016

How does it look when just one or two tabs are open? Do they stay that size, or do they grow?

We think that we want to keep all tabs the same width up to a maximum width but we would not let them span the entire width even if it is free. For example, if the max width is 200px and all tabs are smaller than 200px, we just use that. If one tab is larger, it gets the extra size and takes it away from the other tabs if possible. We would still try to keep all other tabs equal width if possible. We think that we do not want to crop the tab title (name of the file) at all, so once a tab title is cropped, it pushes other tabs out of the view. That way you do not end up with tiny cropped tabs that are hard to use and in most cases all tabs should show up with equal width.

How does it look when the view is split?

Every group has its own concept of tabs and actions, so a split duplicates the little action area where you see the "...", split action and stacks action.

The second screenshot shows a very tiny src below the active tab, is that a glitch? How will we should the path of the file?

I would not put the path of the file below the tab because there is likely not enough space if we keep the title height of the editor part.

In the no-tabs design, the actions to the right look broken, almost to the point of screen cheese. If the user explicitly disables tabs, why not simply remove that notion and just put the x button all the way to the far end of the actions and remove the lighter (darker, in light theme) grey background colour?

We want to try this design to emphasize the fact that there are actions targeting the currently active editor and actions that target the group. Having all of the actions mix together is not very intuitive. So we would like to try a design where this becomes clearer. I would try it out to see how bad the experience is and then continue the discussion. If the background color looks like screen cheese we can also just try to put a separator between editor actions and the group actions.

@alexdima
Copy link
Member

alexdima commented Jun 2, 2016

Please excuse my horrible Paint skills. IMHO the icon with the 3 files (that is currently the seargent icon image) should not be an icon, but should be painted custom. I also think no other actions should be in that area except the "seargent" icon. e.g.:

The idea is that the area to the right of the single tab or multi-tab is a fully clickable area, it is not an icon, but it is somehow painted to show the idea that there are other tabs under/not visible. I certainly fail at drawing this, shadows and co would help, but does it make sense?

When you would hover over that area it would light up somehow to indicate that it is clickable and that it allows you to go to the other tabs behind/not visible.

As for the actions, IMHO they should either go away entirely (I haven't seen a tab UI that includes actions in the tabs!) or go to the left of the 'x'. But if we do tabs, IMHO the area to the right contains just the tabs and not a mix of some actions + some tab management actions (and yes split editors should not be there! - maybe that is solved by drag & drop a tab towards the right of the screen creates a new side or something - goes from 1 to 2 vertical things)

image

@joaomoreno
Copy link
Member

Every group has its own concept of tabs and actions, so a split duplicates the little action area where you see the "...", split action and stacks action.

Space's going to be pretty tight once we start duplicating that three-action box.

The second screenshot shows a very tiny src below the active tab, is that a glitch? How will we should the path of the file?

I would not put the path of the file below the tab because there is likely not enough space if we keep the title height of the editor part.

Meaning: it will not be put anywhere?

I would try it out to see how bad the experience is and then continue the discussion. If the background color looks like screen cheese we can also just try to put a separator between editor actions and the group actions.

The experience isn't going to be bad: the actions are useful and visible; it just doesn't look good. A separator would fix it perfectly.

@bpasero
Copy link
Member Author

bpasero commented Jun 2, 2016

@alexandrudima I am not sure why you are so opposed to putting actions into the area to the right of the tab-group. I think we can add actions there as long as they are for group management. The only actions in there are moving groups around, closing all in the group and we argue that the split action is important enough that it should be there as well. I think we can copy from other tab UIs but we can also evolve the UI to be better.

To give an example, Eclipse uses tabs and also decided to use the space to the right for actions:

image

I find this a very good use of space in that otherwise empty area.

@joaomoreno we will not show the path when you enable tabs but you can see the path on hover. When disabling tabs we show the path as today.

@bpasero bpasero closed this as completed Jun 6, 2016
@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
on-testplan ux User experience issues workbench-tabs VS Code editor tab issues
Projects
None yet
Development

No branches or pull requests

6 participants