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

DevTools Pane/Panel arrow icon #43

Open
DPX-designer opened this issue Nov 23, 2018 · 10 comments
Open

DevTools Pane/Panel arrow icon #43

DPX-designer opened this issue Nov 23, 2018 · 10 comments

Comments

@DPX-designer
Copy link

This arrow svg will become the new DevTools expand/collapse arrow for all panes.
I am currently finalizing the documentation for panes and wondered whether this svg could live with the other Photon icons so that I may link to it from the docs?

Pane-arrow.svg.zip

@aminalhazwani
Copy link
Collaborator

@mcroud we currently have this icon https://github.com/FirefoxUX/photon-icons/blob/master/icons/desktop/arrowhead-down-12.svg (which unfortunately is not accessible on the website) that is also use on about:newtab for collapsable sections. It's a smaller 12px version of our standard 16px icon.

image

Can DevTools panes use the icon up here or is there a larger context for introducing a new one?

@DPX-designer
Copy link
Author

Interesting! Well this SVG is 'new' from a scale point of view only. It is very similar to what we have currently had, only the scale and format (PNG -> SVG) has been tweaked to coincide with padding measurements we are trying to standardize.
So the filled-in, triangle style is what is there already.

@aminalhazwani
Copy link
Collaborator

@mcroud gotcha, thanks for the further explanation. Do you think there is space for using the same icon then?

I made a small test and replaced the icons with the Photon one, what do you think?

image

image

@DPX-designer
Copy link
Author

The chevron style does sit well with me for panel headers, your visuals work equally well as the filled in triangle.

The snag is that the filled in triangle lives in lots and lots of other places, not just panel headers, one of which is the Inspector code view:

screen shot 2018-11-26 at 15 46 50

The code view is a jungle of < and > symbols. There is the potential that having a chevron style icon may introduce a margin of confusion here.
The alternative is to perhaps only update the panel headers, the argument against may be we're introducing a new icon to DevTools that only applies to some parts of the UI but behaviorally they have the same function? Tricky stuff!

@aminalhazwani
Copy link
Collaborator

Ahhhh so many arrow drops 🙂 Yeah, I agree with you that it makes sense to keep this as it is as it might get confusing with the markup.

Regarding the panel headers, I would leave the decision to you if you want to use the arrow head icon or the caret icon.

@DPX-designer
Copy link
Author

@aminalhazwani Super, does the SVG meet the criteria for inclusion in the Photon Icon Viewer?

@aminalhazwani
Copy link
Collaborator

@mcroud it's looking good, only two minor things. We would need 4 separate versions of the icon (up, down, right, and left). And the icon should we wrapped in a 12x12px container.

If you need a .ai template you can find it here https://design.firefox.com/photon/resources/icons.html#icons-template 🙂

@DPX-designer
Copy link
Author

@brassy- I have followed Amin's guidance above, please find below the icon with 4 directional variants, each within a 12px grid.

DevTools Arrow.zip

Let me know if there is anything else I need to do!

@brassy-
Copy link
Contributor

brassy- commented Dec 14, 2018 via email

@ariasuni
Copy link

Any news about this?

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

No branches or pull requests

4 participants