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
Letter I review #5496
Letter I review #5496
Conversation
- Update source - Add guidelines
- Update source
- Add guidelines
- Update source
- Add guidelines
- Add guidelines
- Update hex - Update source - Add guidelines
- Update source - Add guidelines
- Update source - Add guidelines
- Update source - Add guidelines
- Update source
- Update source
- Update source - Add license
- Update source - Add guidelines
- Update source
- Add guidelines
- Update source
- Update source
- Update source
- Add guidelines
- Add guidelines
- Update source
I don't know that we'd need gaps between the various shades of orange but I do think we should do something to provide some separation between the purple shape and the rest
Agreed. To review the first few changes in this PR:
I'll be back to review the rest later, if someone else doesn't beat me to it. |
Continuing on ...
I'll get the last few done later on today. |
- Update guidelines
- Add guidelines
- Update hex
- Remove guidelines
- Update hex
Thank you @PeterShaggyNoble for the review, really appreciated!
You're right, done!
You're right. I had it noted, but forgot to add it as a note in my PR. Thanks for catching it!
This section mention the trademark symbol, but I thought it is for "printed literature" only... Not too sure...
👍
Indeed, done!
It is, for sure, the primary color of InfluxData. InfluxDB tend to use blue through their interface.
Agreed, I updated it to use |
They mention videos as well so it would seem to apply to physical and digital media. And while it mentions it only being required in the US, I think the language for the ret of the world is ambiguous enough for us to invoke our own consensus on its inclusion. Anyway, on to the last few - didn't realise there were so few left or I would have kept going! - and then I'll take another pass over it:
|
- Update source
- Update source
Alright, done!
Weird, was not able to access this page 2 days ago 🤔 Reverted! |
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.
All looks good to me, @service-paradis, nice work 👍🏻
If anyone wants to propose alternative colours for IBM or Internet Archive or feel we should update the colour for InfluxDB, we can address those separately.
# New Icons - Automattic (#5489) - Datto (#5493) - FamPay (#5505) - Forestry (#5454) - Immer (#5437) - KashFlow (#5465) - Kdenlive (#5193) - LIFX (#5448) - Photopea (#5518) - pnpm (#5482) # Updated Icons - Gitpod (#5464) - IBM (#5496) - IBM Watson (#5496) - ICON (#5496) - Iconfinder (#5496) - iDEAL (#5496) - IEEE (#5496) - iFixit (#5496) - iFood (#5496) - IFTTT (#5496) - iHeartRadio (#5496) - IKEA (#5496) - IMDb (#5496) - Imgur (#5496) - Imou (#5496) - Indeed (#5496) - Infiniti (#5496) - InfluxDB (#5496) - Informatica (#5496) - Infosys (#5496) - Ingress (#5496) - Inkscape (#5496) - Insomnia (#5496) - Instagram (#5496) - Instapaper (#5496) - Intel (#5496) - IntelliJ IDEA (#5496) - Intercom (#5496) - Internet Archive (#5496) - Internet Explorer (#5496) - Invoice Ninja (#5496) - ioBroker (#5496) - Ionic (#5496) - Issuu (#5496) - Itch.io (#5496) - PayPal (#5467) - Plus Codes (#5468) - R (#5285) - Zotero (#5491)
Issue: #5251
Notes