Skip to content

[RFC] Bikeshed the tap and top icons#1637

Merged
adleong merged 4 commits intomasterfrom
alex/tap-icon
Sep 14, 2018
Merged

[RFC] Bikeshed the tap and top icons#1637
adleong merged 4 commits intomasterfrom
alex/tap-icon

Conversation

@adleong
Copy link
Member

@adleong adleong commented Sep 12, 2018

I find the tap and top icons a bit strange. Using the filter icon for tap is weird because we already use the filter icon for filtering columns. The caret-up icon looks weird to me for top because it looks like something that is click to expand.

Unfortunately, the alternatives aren't super great either. Here are some possibilities I could think of:

For tap:

  • eye (although this sometimes indicates show/hide... and it also just looks weird)
  • phone
  • shower (too whimsical?)

For top:

  • table (looks like a calendar)
  • list-ol
  • arrow-up

Thoughts?

Signed-off-by: Alex Leong [email protected]

@adleong adleong self-assigned this Sep 12, 2018
@grampelberg
Copy link
Contributor

I'm good with these, but I went and looked for maybe a couple other options. Some ideas:

  • compass
  • lightbulb
  • stream

Some slightly more out there ideas:

  • route
  • analytics
  • searchengin

@adleong
Copy link
Member Author

adleong commented Sep 12, 2018

I like stream for top! Another idea: microscope for tap.

@grampelberg
Copy link
Contributor

grampelberg commented Sep 13, 2018 via email

@rmars
Copy link

rmars commented Sep 13, 2018

Yayyy, thanks for fixing these. I couldn't really think of good ones when I added this so I chose super arbitrarily.
Where are stream and microscope? Are they at https://ant.design/components/icon/ and I'm just blind?

@adleong
Copy link
Member Author

adleong commented Sep 13, 2018

@rmars
Copy link

rmars commented Sep 13, 2018

I don't think we actually have them anywhere right now, but we used to? They shouldn't be too bad to add.

@klingerf
Copy link
Contributor

Looks like we're still importing the font-awesome css, here:

https://github.com/linkerd/linkerd2/blob/master/web/app/css/styles.css#L1

But we'd need to upgrade to the latest version to get the microscope icon. I'm planning on doing this as part of #1461.

@rmars
Copy link

rmars commented Sep 14, 2018

I like microscope a lot! Stream is good too!

Copy link
Contributor

@klingerf klingerf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⭐️ 😭 It's beautiful.

image

Copy link

@rmars rmars left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🍔 🍔 🏗

@adleong adleong merged commit 6c6310e into master Sep 14, 2018
@adleong adleong deleted the alex/tap-icon branch September 14, 2018 22:30
zachalbert added a commit to zachalbert/linkerd2 that referenced this pull request Sep 15, 2018
* master:
  Move more info from the tap table into the expanded row (linkerd#1641)
  `linkerd check` sends params on version check (linkerd#1642)
  Bikeshed the tap and top icons (linkerd#1637)
  Add link to tap each row in top table (linkerd#1643)
  Bump default check retry time to 5 minutes (linkerd#1645)
  Make wait=true a default option for check and dashboard (linkerd#1640)
  Add version check to Grafana dashboard (linkerd#1638)
  Add data plane check for metrics Prometheus (linkerd#1635)
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

Successfully merging this pull request may close these issues.

4 participants