meshery.io icon indicating copy to clipboard operation
meshery.io copied to clipboard

Added Logo

Open UtkarshMishra12 opened this issue 2 years ago β€’ 9 comments

Signed-off-by: Utkarsh Mishra [email protected]

Description Changed the Meshery logos for dark/light mode with high contrast This PR fixes #844

Notes for Reviewers

Signed commits

  • [x] Yes, I signed my commits.

UtkarshMishra12 avatar Oct 01 '22 11:10 UtkarshMishra12

Deploy Preview for mesheryio-preview ready!

Name Link
Latest commit 0af18b3301d3decbde7839d210b0b741511e4c16
Latest deploy log https://app.netlify.com/sites/mesheryio-preview/deploys/634854d65a41c900087a0567
Deploy Preview https://deploy-preview-846--mesheryio-preview.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

netlify[bot] avatar Oct 01 '22 11:10 netlify[bot]

@leecalcote Yes, the code I have used is taken from the README.md of Meshery Repo as suggested by Kamal in other issue also link

UtkarshMishra12 avatar Oct 02 '22 02:10 UtkarshMishra12

Will you please confirm that this is the same code (verbatim) of the other readmes in the repos in the Meshery org?

although this creates a dependency that the image should be there in main meshery/meshery repo with using this https://raw.githubusercontent.com/meshery/meshery/master/.github/assets/images/meshery/meshery-logo-dark-text-side.svg but @leecalcote that seems fair to me as it is our main repo and we don't have to manually add the same logo to each repo.

kamalbuilds avatar Oct 02 '22 03:10 kamalbuilds

@leecalcote he(Kamal) has tested the changes.

UtkarshMishra12 avatar Oct 02 '22 07:10 UtkarshMishra12

Will you please confirm that this is the same code (verbatim) of the other readmes in the repos in the Meshery org?

although this creates a dependency that the image should be there in main meshery/meshery repo with using this https://raw.githubusercontent.com/meshery/meshery/master/.github/assets/images/meshery/meshery-logo-dark-text-side.svg but @leecalcote that seems fair to me as it is our main repo and we don't have to manually add the same logo to each repo.

@legendarykamal you're right. I neglected to be more precise that the desire is for images to be locally referenced (as they are stored in each repo), but that the rest of the html/markdown was to be identical. Ideally, even the references are identical - - they would be if relative references were used.

leecalcote avatar Oct 02 '22 19:10 leecalcote

@leecalcote What should I do now, Can you guide me with this PR?

UtkarshMishra12 avatar Oct 02 '22 19:10 UtkarshMishra12

I'll defer to @legendarykamal.

leecalcote avatar Oct 02 '22 19:10 leecalcote

Hey, @leecalcote so I have to take the Logo image from the local repo(store the media inside the repo, and refer to its path for showing it on Readme.md), not from the other repo?

UtkarshMishra12 avatar Oct 03 '22 13:10 UtkarshMishra12

yes @UtkarshMishra12 store it locally and then use relative referencing to render it.

kamalbuilds avatar Oct 03 '22 14:10 kamalbuilds

@leecalcote @legendarykamal As per your suggestion, I have stored the logos locally and then used relative referencing to render them. Let me know your thought on it.

UtkarshMishra12 avatar Oct 13 '22 17:10 UtkarshMishra12

@Nikhil-Ladha I'll remove the new from the filename.

UtkarshMishra12 avatar Oct 13 '22 17:10 UtkarshMishra12

@Nikhil-Ladha Changes are doneπŸ™Œ

UtkarshMishra12 avatar Oct 13 '22 18:10 UtkarshMishra12

sharp eye @Nikhil-Ladha βœ… πŸ‘

kamalbuilds avatar Oct 13 '22 18:10 kamalbuilds