Skip to content
This repository has been archived by the owner on Feb 8, 2018. It is now read-only.

add status icons to Team explorer #3739

Closed
chadwhitacre opened this issue Sep 1, 2015 · 11 comments
Closed

add status icons to Team explorer #3739

chadwhitacre opened this issue Sep 1, 2015 · 11 comments
Milestone

Comments

@chadwhitacre
Copy link
Contributor

red, yellow, green for rejected, under review, and approved

See GitHub Issues open/closed icons for inspiration.

Reticketed from #3704.

@chadwhitacre
Copy link
Contributor Author

Done in #3749.

@mattbk
Copy link
Contributor

mattbk commented Sep 8, 2015

Icons overlap text in Chrome for Windows.
image

@chadwhitacre
Copy link
Contributor Author

Interesting! I don't see that behavior in Chrome on Mac OS:

screen shot 2015-09-08 at 7 51 35 pm

It's probably because we're depending on a font for those (they're Unicode characters) and the fonts differ between Windows and Mac OS.

@chadwhitacre chadwhitacre reopened this Sep 8, 2015
@mattbk
Copy link
Contributor

mattbk commented Sep 9, 2015

It starts by loading the image below, then switches to what I posted previously when the page is done loading. Windows Chrome. Is it loading emoji first, and then images?

2015-09-09_08-12-34

@chadwhitacre
Copy link
Contributor Author

The colored images are actually still emoji, but my hunch is it's starting with a default font and then upgrading to another font. We should probably just use images instead of emoji for consistency.

@mattbk
Copy link
Contributor

mattbk commented Sep 9, 2015 via email

@chadwhitacre
Copy link
Contributor Author

Ah, okay. But #3739 (comment) is still bad, right?

@mattbk
Copy link
Contributor

mattbk commented Sep 9, 2015 via email

@chadwhitacre
Copy link
Contributor Author

Right, so for consistency's sake I think we should use the same icon font method we're using for the sign-in icons and the footer icons.

@chadwhitacre
Copy link
Contributor Author

I've deployed #3765. How do the status icons look now, @mattbk?

@chadwhitacre
Copy link
Contributor Author

Here's what I see:

screen shot 2015-09-10 at 2 57 12 am

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants