Skip to content
This repository has been archived by the owner on Jan 26, 2021. It is now read-only.

enh: Remove Systers' Logo #566

Closed
wants to merge 3 commits into from
Closed

enh: Remove Systers' Logo #566

wants to merge 3 commits into from

Conversation

gaurivn
Copy link
Member

@gaurivn gaurivn commented Jun 16, 2020

Description

Removed Systers' Logo

Fixes #546

Type of Change:

Delete irrelevant options.

  • User Interface

How Has This Been Tested?

Describe the tests you ran to verify your changes. Provide instructions or GIFs so we can reproduce. List any relevant details for your test.
Screenshot from 2020-06-16 12-22-21

Checklist:

Delete irrelevant options.

  • My PR follows the style guidelines of this project
  • I have performed a self-review of my own code or materials

@sakshi1499
Copy link

Is it as it looks in the screenshot? Because that looks really stretched @gaurivn

@sammy1997
Copy link
Contributor

@gaurivn can you squash the 2 commits?

@gaurivn
Copy link
Member Author

gaurivn commented Jun 19, 2020

Is it as it looks in the screenshot? Because that looks really stretched @gaurivn

Yes, I took the screenshot of running website after I cropped out Systers' logo.

@gaurivn
Copy link
Member Author

gaurivn commented Jun 19, 2020

@gaurivn can you squash the 2 commits?

Sure

Copy link
Contributor

@theyashshahs theyashshahs left a comment

Choose a reason for hiding this comment

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

@gaurivn the logo looks a bit stretched, can you take a higher resolution image :)

Copy link
Contributor

@sammy1997 sammy1997 left a comment

Choose a reason for hiding this comment

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

We should use a better resolution image or use something like .webp

Copy link

@sakshi1499 sakshi1499 left a comment

Choose a reason for hiding this comment

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

I'll see if I can look for a better resolution image. I'll link the file in the comments. If it isn't available we'll see for some other alternative, it currently looks really stretched.

@sakshi1499
Copy link

@gaurivn Were you able to find better resolution logo?

@gaurivn
Copy link
Member Author

gaurivn commented Jul 12, 2020

Sorry for the delay, I got a few AnitaB.org logos - better resolution ones, I'll try with them.

@sakshi1499
Copy link

@gaurivn awesome. Sure ☺️

@sakshi1499
Copy link

@gaurivn Any updates?

@gaurivn gaurivn requested a review from sakshi1499 July 22, 2020 20:42
@sakshi1499
Copy link

Can you add a photo of how it looks now?

@gaurivn
Copy link
Member Author

gaurivn commented Jul 30, 2020

I think I don't have the bandwidth for this atm..faced some issues while trying to run again after changing logos.. closing this PR and making it available.

@gaurivn gaurivn closed this Jul 30, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove Systers' Logo
4 participants