Skip to content
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

Button width to auto and updated 'Our Programs' description #222

Merged
merged 7 commits into from
Oct 22, 2022

Conversation

KeneePatel
Copy link
Contributor

Proposed Changes

  • Width styling element of the "Check our Social Button"

Checklist

  • Documentation of the code
  • Add your name to the CONTRIBUTORS file
  • Testing your code in the local machine

Screenshots

image
This is how it looked like before the change

image
After the change.

It was really annoying on my screen to see the exclamation mark going out of the box and taking up a second line
@netlify
Copy link

netlify bot commented Oct 21, 2022

Deploy Preview for girlscript-asansol ready!

Name Link
🔨 Latest commit 3650174
🔍 Latest deploy log https://app.netlify.com/sites/girlscript-asansol/deploys/6352b1cc102ace000820bb42
😎 Deploy Preview https://deploy-preview-222--girlscript-asansol.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.

@KeneePatel
Copy link
Contributor Author

I am sorry if it's nothing big and bothered you, but it just really triggered my OCD lol.

@KeneePatel KeneePatel changed the title Kenee patel Button width to auto Oct 21, 2022
It was really annoying on my screen to see the exclamation mark going out of the box and taking up a second line
@KeneePatel KeneePatel changed the title Button width to auto Button width to auto and updated 'Our Programs' description Oct 21, 2022
@KeneePatel
Copy link
Contributor Author

I also added the pr for issue #212

@jayzsh
Copy link
Member

jayzsh commented Oct 22, 2022

@KeneePatel Haha no worries, we'll merge this.

@jayzsh jayzsh merged commit ec728b2 into gsasansol:main Oct 22, 2022
@girlscript-asansol
Copy link
Contributor

@KeneePatel did you mentioned your name in the contributors file? If not please make a seperate pr for it.

@KeneePatel
Copy link
Contributor Author

@KeneePatel did you mentioned your name in the contributors file? If not please make a seperate pr for it.

Don't worry, I have already done that

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

Successfully merging this pull request may close these issues.

None yet

3 participants