Skip to content

feat(example): update app icons#1417

Merged
Gustl22 merged 10 commits into
mainfrom
gustl22/logo
Feb 9, 2023
Merged

feat(example): update app icons#1417
Gustl22 merged 10 commits into
mainfrom
gustl22/logo

Conversation

@Gustl22

@Gustl22 Gustl22 commented Feb 9, 2023

Copy link
Copy Markdown
Collaborator

Description

I created a logo for our library (or mostly for the exapmle app) in style of the Flutter logo. Idk if you more want sth that matches the Flame logo. E.g. a burning note (?).
Or we can change it again, if we crave for sth better :)

Checklist

  • The title of my PR starts with a Conventional Commit prefix (fix:, feat:, docs:, chore: etc).
  • I have read the Contributor Guide and followed the process outlined for submitting PRs.
  • I have updated/added tests for ALL new/updated/fixed functionality.
  • I have updated/added relevant documentation and added dartdoc comments with ///, where necessary.
  • I have updated/added relevant examples in example.

Breaking Change

  • Yes, this is a breaking change.
  • No, this is not a breaking change.

Related Issues

@Gustl22 Gustl22 marked this pull request as ready for review February 9, 2023 10:00
@Gustl22 Gustl22 requested review from luanpotter and spydon February 9, 2023 12:10
@spydon

spydon commented Feb 9, 2023

Copy link
Copy Markdown
Member

Woooow, looks great!
I don't think it has to look like Flame since it is completely separate from flame. :)

@erickzanardo erickzanardo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Love it! You should put it on the readme too :D

@spydon

spydon commented Feb 9, 2023

Copy link
Copy Markdown
Member

Love it! You should put it on the readme too :D

Agree! Maybe in a similar fashion like we do with the centered Flame logo? 😍

@Gustl22 Gustl22 marked this pull request as draft February 9, 2023 16:34

@renancaraujo renancaraujo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Love it!

@Gustl22

Gustl22 commented Feb 9, 2023

Copy link
Copy Markdown
Collaborator Author

Unfortunately CSS styling is not a thing in GH, so I have to add the text in the logo itself. I tried many combinations, and as the name is that long, it looks weird to put it under the icon, so I put it next to it like done on Flutter itself.
You can preview README here, as the link to the logo is not yet existant (only after merging) to provide absolute paths due to the monorepos subpackages.

Compact Horizontal Vertical
logo_ap_compact logo_ap_horizontal logo_ap_vertical

@Gustl22 Gustl22 marked this pull request as ready for review February 9, 2023 18:08
@spydon

spydon commented Feb 9, 2023

Copy link
Copy Markdown
Member

Unfortunately CSS styling is not a thing in GH, so I have to add the text in the logo itself. I tried many combinations, and as the name is that long, it looks weird to put it under the icon, so I put it next to it like done on Flutter itself. You can preview README here, as the link to the logo is not yet existant (only after merging) to provide absolute paths due to the monorepos subpackages.

Nice, the first one is my favourite!
Make sure to store both the logo separately and the logo with text in the repo so that we can use them as fitting in different contexts.

@erickzanardo

Copy link
Copy Markdown
Member

First one is my favorite as well!

@Gustl22

Gustl22 commented Feb 9, 2023

Copy link
Copy Markdown
Collaborator Author

For me personally or from Design perspective I think the second variant is best for the README header as it states the package name more clearly and possibly bringing less confusion to the users.
But I'm also happy to change it to the first variant! Won't make much of a difference in the end. I can add the other ones as well for completeness. I always provide vector files, which can be manipulated as desired.

Only for clearing the terms:
icon - the icon / brand itself
logo - the brand mostly with text

@Gustl22

Gustl22 commented Feb 9, 2023

Copy link
Copy Markdown
Collaborator Author

Changed to compact logo. Looks quite clear too :)

@Gustl22 Gustl22 enabled auto-merge (squash) February 9, 2023 19:07
@Gustl22 Gustl22 merged commit ac35df8 into main Feb 9, 2023
@Gustl22 Gustl22 deleted the gustl22/logo branch February 9, 2023 20:00
@luanpotter

Copy link
Copy Markdown
Member

OMG this is so awesome

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.

6 participants