Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Clean-up README (#80) #85

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

Merged
shenxianpeng merged 1 commit into master from update-readme
Aug 13, 2022
Merged

Clean-up README (#80) #85

shenxianpeng merged 1 commit into master from update-readme
Aug 13, 2022

Conversation

@shenxianpeng
Copy link
Collaborator

@shenxianpeng shenxianpeng commented Aug 13, 2022

Resolve #80

@shenxianpeng shenxianpeng added the documentation Improvements or additions to documentation label Aug 13, 2022
Copy link
Collaborator

@2bndy5 2bndy5 left a comment

Choose a reason for hiding this comment

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

I guess this should be titled "clean-up README" 😄

Copy link
Collaborator

2bndy5 commented Aug 13, 2022

Was the icon_large.xcf file (my GIMP project) moved to the .github repo?

@shenxianpeng shenxianpeng changed the title (削除) Update README (#80) (削除ここまで) (追記) Clean-up README (#80) (追記ここまで) Aug 13, 2022
Copy link
Collaborator Author

shenxianpeng commented Aug 13, 2022
edited
Loading

Was the icon_large.xcf file (my GIMP project) moved to the .github repo?

No, what is icon_large.xcf used for? You or I move to the .github repo if needed.

@shenxianpeng shenxianpeng merged commit eddec1b into master Aug 13, 2022
@shenxianpeng shenxianpeng deleted the update-readme branch August 13, 2022 10:05
Copy link
Collaborator

2bndy5 commented Aug 13, 2022

what is icon_large.xcf used for?

I do all my image manipulation using GIMP. I used the xcf file to save my work on the favicon we've been using:
image

Personally, I like how the yellow circle looks better in dark theme. The current icon is a bit bland in dark theme:
image

Copy link
Collaborator Author

shenxianpeng commented Aug 13, 2022
edited
Loading

En, yellow should look good in both dark and default themes, the pre-commit icon is also yellow, but our yellow icon is not sharp enough that's why I want to remove it, see https://github.com/cpp-linter/cpp-linter-action/blob/894a54988abed9f2c7a37f5b54694e819de7f462/docs/images/icon_large.png

If the new yellow icon could display well(sharp and lossless) as the current icon, we can change it to yellow. https://github.com/cpp-linter/cpp-linter-action/blob/894a54988abed9f2c7a37f5b54694e819de7f462/docs/images/icon.png

Copy link
Collaborator

2bndy5 commented Aug 13, 2022

oh,

  1. I added a slight bevel (with shadow) to the circle so the checkmark would pop out.
  2. I also added a shadow to the check mark to make it seem 3D layered.
  3. There is some slight pixelation around the edges because I scaled it up a bit to reduce the empty space around it. Then I tried to hide that with some bevel shadows.

If you're looking for flat, that's easy.
image

Copy link
Collaborator Author

This yellow circle icon looks great 👍 I would also like to see the flat

Copy link
Collaborator

2bndy5 commented Aug 13, 2022

I still don't like the empty space around it, but when I scale it up, it gets pixelated.
image

What did you use to create the logo?

Copy link
Collaborator Author

I found the logo from google or Canva, I forgot it 😟

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@2bndy5 2bndy5 2bndy5 approved these changes

Assignees

No one assigned

Labels

documentation Improvements or additions to documentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Clean-up README

AltStyle によって変換されたページ (->オリジナル) /