Skip to content
This repository has been archived by the owner on Nov 17, 2021. It is now read-only.

Enhance visibility of the expand/hide buttons on the left control panel #173

Open
jterstriep opened this issue Apr 27, 2018 · 9 comments
Open
Assignees
Labels

Comments

@jterstriep
Copy link
Member

No description provided.

@Zodiase Zodiase self-assigned this May 2, 2018
@kintigh
Copy link

kintigh commented May 2, 2018

Please change opacity default to 50% at the same time

Zodiase added a commit that referenced this issue May 7, 2018
@Zodiase Zodiase closed this as completed in fe87c46 May 7, 2018
@Zodiase
Copy link
Contributor

Zodiase commented May 7, 2018

image

Now the toggle buttons have clear labels, shadows and background colors depending on their state.

@tmcphillips
Copy link
Member

Currently the Style buttons do not have little arrows on them indicating that they are used to expand or collapse the style panel for each variable. As a consequence, it still is not clear that one can collapse the styles for a variable once it is open (see image in preceding comment).

Is there a reason why we are not using arrows on the style expand/collapse buttons?

@tmcphillips tmcphillips reopened this Jun 26, 2018
@Zodiase
Copy link
Contributor

Zodiase commented Jun 27, 2018

@tmcphillips This issue was created to address the concern about the panel expand/collapse buttons, like the one shown in the screenshot below. It does not cover any requirement on the overlay style config panel buttons.

image

According to Keith, by the arrows alone, it was not clear to users, what the up/down icons mean or the fact that the panels are collapsible. There was no requirement, as far as this issue is concerned, for any other buttons to have the arrows. Since other buttons didn't have any arrows in the first place.

So I made the minimum viable changes to address this issue.

As to why there is no arrows for the toggle button for the overlay style config panels, it's because they are different type of UI elements, in my mind, and should be distinguished to avoid confusion.

image

See this screenshot of Photoshop (not the best software known for its intuitive UI, but has a similar level of UI complexity in this case) for example. The layout panel (which contains the "Layers", "Chanels" and "Paths" tabs) itself is collapsible, even though it may not be clearly visible. The panel does not have arrows to indicate it's collapsible, either. The layers list, however, has arrows on the left of the folders, indicating a nested structure. The individual layers/folders, may or may not have another kind of arrow to the right for showing/hiding the layer effects list.
You can see a pattern here. For a different kind of UI elements, a new design is needed to avoid confusion. Also toggle controls don't necessarily need arrows. Imagine all of those show/hide controls to have the identical type of arrows, would you be able to know what to expect when you expand something?

So, I didn't add arrows to the "Style" buttons so they won't look too similar to the panel toggle buttons.

We can talk about changing the overall design of toggle controls but I think that should be a separate issue.

@kintigh
Copy link

kintigh commented Jun 27, 2018 via email

@Zodiase
Copy link
Contributor

Zodiase commented Jun 27, 2018

Todo:

  • Move expand/collapse arrow icon to the left of the panel title.
  • Change the style panel button to use a gear icon only.

@kintigh
Copy link

kintigh commented Jun 27, 2018

The agreement was to change the Tent and Style button to a gear button,

Zodiase added a commit that referenced this issue Jun 28, 2018
Zodiase added a commit that referenced this issue Jun 28, 2018
@Zodiase
Copy link
Contributor

Zodiase commented Jun 28, 2018

image

@Zodiase Zodiase added the pending deployment The solution is implemented but not yet deployed. label Jun 28, 2018
@Zodiase Zodiase removed the pending deployment The solution is implemented but not yet deployed. label Jul 9, 2018
@Zodiase
Copy link
Contributor

Zodiase commented Jul 9, 2018

Feature is deployed in v0.1.27.

@Zodiase Zodiase added the v0.1.27 label Jul 9, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

4 participants