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

Add setting for displaying all borders on active pane #4666

Closed
deoradh opened this issue Feb 20, 2020 · 5 comments
Closed

Add setting for displaying all borders on active pane #4666

deoradh opened this issue Feb 20, 2020 · 5 comments
Labels
Area-Settings Issues related to settings and customizability, for console or terminal Area-User Interface Issues pertaining to the user interface of the Console or Terminal Issue-Task It's a feature request, but it doesn't really need a major design. Product-Terminal The new Windows Terminal. Resolution-Duplicate There's another issue on the tracker that's pretty much the same thing.

Comments

@deoradh
Copy link

deoradh commented Feb 20, 2020

Environment

Windows build number: 10.0.18363.0
Windows Terminal version (if applicable): 0.9.433.0

Any other software? No

Steps to reproduce

  1. Open tab
  2. Divide tab into two, either horizontally or vertically. Don't divide further.

Expected behavior

The UI clearly shows which pane has focus.

Actual behavior

The current design (#994) just highlights the internal borders between panes: it doesn't fully encircle the pane or provide other UI cues to the user. The shift in line placement in this case is about one line width, making the difference relatively imperceptible and largely useless at that point.

Straw man suggestion

The existing design relies on having multiple cells to allow for disambiguation. I'd suggest just showing all borders of the pane that has focus, making it clear in all cases.

@ghost ghost added Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting Needs-Tag-Fix Doesn't match tag requirements labels Feb 20, 2020
@zadjii-msft zadjii-msft added Area-Settings Issues related to settings and customizability, for console or terminal Area-User Interface Issues pertaining to the user interface of the Console or Terminal Issue-Task It's a feature request, but it doesn't really need a major design. Product-Terminal The new Windows Terminal. labels Feb 20, 2020
@ghost ghost removed the Needs-Tag-Fix Doesn't match tag requirements label Feb 20, 2020
@zadjii-msft zadjii-msft changed the title Pane accessibility: highlighting is ambiguous in simple case Add setting for displaying all borders on active pane Feb 20, 2020
@zadjii-msft
Copy link
Member

Questions:

  • Should all borders be visible around all panes always when this setting is enabled?
  • If no, when the borders are added to the active pane, should they:
    • cover content from the pane?
    • change the width/height of the pane content, to make room for the borders around the edges?

@zadjii-msft zadjii-msft added this to the Terminal Backlog milestone Feb 20, 2020
@deoradh
Copy link
Author

deoradh commented Feb 20, 2020

Should all borders be visible around all panes always when this setting is enabled?

If there are multiple panes, all borders should be visible on the pane that has focus. If there is only a single pane, the border is unnecessary.

If no, when the borders are added to the active pane, should they:
cover content from the pane?

No. Covering content in neighboring panes should be okay, but can be avoided if inter-pane spacing is appropriately sized (if all borders are drawn, that spacing can be shared in ways it cannot be, today). The question of what to do with the edges that correspond to the tab window itself is a slightly tricky one, of course.

change the width/height of the pane content, to make room for the borders around the edges?

No. You risk weird UI behaviors doing so (eg, scroll bars pop into existence for no apparent reason).

@DHowett-MSFT
Copy link
Contributor

I'm going to hook this up to #1000 and put it on the backlog, marked help-wanted.

I think there's a good UI we can come to that doesn't require us to add another setting.

Perhaps #3062 would help?

@DHowett-MSFT
Copy link
Contributor

Actually, yeah, I'd like to roll this into /dup #3062. Providing better visual differentiation for focused versus unfocused terminals is going to do a lot more to solve this than adding another border option.

@ghost
Copy link

ghost commented Feb 21, 2020

Hi! We've identified this issue as a duplicate of another one that already exists on this Issue Tracker. This specific instance is being closed in favor of tracking the concern over on the referenced thread. Thanks for your report!

@ghost ghost closed this as completed Feb 21, 2020
@ghost ghost added Resolution-Duplicate There's another issue on the tracker that's pretty much the same thing. and removed Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting labels Feb 21, 2020
@zadjii-msft zadjii-msft removed this from the Terminal Backlog milestone Mar 2, 2020
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Settings Issues related to settings and customizability, for console or terminal Area-User Interface Issues pertaining to the user interface of the Console or Terminal Issue-Task It's a feature request, but it doesn't really need a major design. Product-Terminal The new Windows Terminal. Resolution-Duplicate There's another issue on the tracker that's pretty much the same thing.
Projects
None yet
Development

No branches or pull requests

3 participants