-
-
Notifications
You must be signed in to change notification settings - Fork 29
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
feat(assets): Replace old logos with new ones #378
Conversation
5bdaedb
to
a8fd9b1
Compare
0c48c47
to
4cde629
Compare
Looks ok, but the small icons need work. I think we should adjust the pixels by hand to balance the lines better. The middle one looks thick and I wouldn't want to look at it in my browser every day. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, but let's refine these icons a bit more.
@ERosendo, I just pushed the latest icons as from Slack. If you're happy, let's merge. Before we ship, it'd be really nice to use our new notification/banner thing to let people know this change happened. Want to take a stab at that? I assume it's pretty easy, right? |
Sure!
Yes. @mlissner Would you like to display this banner just on the Options page, or consider including it on other PACER pages as well (similar to the login page banner)? For the Options page: The banner will appear only when users click the recap icon, and it will look like this (I'll use a different message): |
Yes, the way going forward is that all news goes on the login page and the options page, per the discussion here: freelawproject/recap#367 (so doing this should fix that issue too). |
@mlissner How about adding a badge to the extension icon? This could be a great way to grab the attention of users who have pinned it to their browser toolbar. 🤔 |
Badge, login page, and options menu, please! Let's see if we can get people's attention. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This icon should be gray, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Icons look great! Let's address the comment about the color of the disabled
version and then merge.
Oops, I responded, but forgot to submit:
|
Here are screnshots of the new logos and its variants:
Recap Icon next to a docket entry:
Button to accept charges and banner to let users know the document is available in the RECAP archive:
Button and banner from the docket search page in PACER
Warning about combined documents
Options page: