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

Improve Panels behaviors in IDE #2463

Closed
2 tasks
slemeur opened this issue Sep 16, 2016 · 8 comments
Closed
2 tasks

Improve Panels behaviors in IDE #2463

slemeur opened this issue Sep 16, 2016 · 8 comments
Assignees
Labels
kind/enhancement A feature request - must adhere to the feature request template.

Comments

@slemeur
Copy link
Contributor

slemeur commented Sep 16, 2016

The goal of this issue is to improve the behaviors of panel.
A user should be able to maximize the size of any panel in the UI
A panel could have display options and various secondary actions which does not need to be displayed all the time to the user.

There are two enhancements:

  • Provide mecanism for panel's options
  • Allow any panel to be maximized/minimized

Here is a first proposal for the UI.
Minimized version of the Console Panel
ide - console panel - small

Maximized version of the Console panel
ide - console panel - fullscreen

@slemeur slemeur added kind/enhancement A feature request - must adhere to the feature request template. team/ide labels Sep 16, 2016
@TylerJewell
Copy link

It takes two clicks for someone to hide the console tree. That doesn't seem very user friendly.

@TylerJewell
Copy link

Developers will expect key bindings for each behavior as well.

@slemeur
Copy link
Contributor Author

slemeur commented Sep 23, 2016

Discussed and agreed on this solution.
Will soften the icons.

@slemeur
Copy link
Contributor Author

slemeur commented Sep 23, 2016

Updated mockups

@ashumilova ashumilova added team/plugin status/open-for-dev An issue has had its specification reviewed and confirmed. Waiting for an engineer to take it. and removed team/ide labels Sep 27, 2016
@TylerJewell
Copy link

"Fullscreen Pane" is not good english. Change it to "Maximize". Also, the buttons seem to be too big vs. other menu drop downs that we have. And I am not clear on the differences between the button to expand vs. the drop down. So I am not sure this issue is design complete.

@slemeur
Copy link
Contributor Author

slemeur commented Oct 19, 2016

Updated icon shape + styles.

ide - console panel - small

ide - console panel - fullscreen

@slemeur
Copy link
Contributor Author

slemeur commented Oct 21, 2016

Two new options proposed.

Option1: Panels with always 3 buttons

  • button1: expand/collapse
  • button2: maximize
  • button3: more options

The button3 "more options" will display a dropdown which show the state of the maximize mode with a checkmark. The other panel options are displayed in this dropdown menu.

ide - console panel - normal
ide - console panel - small
ide - console panel - fullscreen

Option2: Panels with 1 button always displayed and other revealed on hover in the title bar

The idea of this option is to limit the number of buttons displayed in the toolbar. Only primary actions should be displayed to a user - and secondary actions should be displayed in a discoverable and learnable way.

All panels would display the "option button":
ide - console panel - no hover

On hover, we then display the other actions:
ide - console panel - hover

On clicked, we show the dropdown:
ide - console panel - clicked

Maximized:
ide - console panel - fullscreen

Option2 provide a more elegant UX, but for simplicity and consistency with other existing IDE, I prefer the Option1.

I let others voting.

@vitaliy-guliy
Copy link
Contributor

Which option is preferred?

@ashumilova ashumilova added status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. and removed status/open-for-dev An issue has had its specification reviewed and confirmed. Waiting for an engineer to take it. labels Nov 23, 2016
@vitaliy-guliy vitaliy-guliy added status/pending-merge and removed status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. status/pending-merge labels Dec 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement A feature request - must adhere to the feature request template.
Projects
None yet
Development

No branches or pull requests

5 participants