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

Add Socials Sections #34

Closed
wants to merge 6 commits into from

Conversation

newtoallofthis123
Copy link

This commit adds a social div to the website template.

It uses the same ionicons and also adds helpful comments.

image

This commit adds a social div to the website template
@newtoallofthis123
Copy link
Author

This closes #18

@divyanshudhruv
Copy link
Owner

Hey, nice work. But you can change the colors of the button to a flat color as it will look more nice.

@newtoallofthis123
Copy link
Author

I will do that...but what do you mean by flat color? Can you please share an example?

@divyanshudhruv divyanshudhruv self-requested a review October 1, 2023 05:59
@divyanshudhruv
Copy link
Owner

73924ce7401a7dd8e6c47bcfe94f1dd6

I mean color that seems soft to eyes

@newtoallofthis123
Copy link
Author

Oh! Okay Done...will also help resolve a style.css conflict that happened :)

@divyanshudhruv
Copy link
Owner

Ok thank you, I hope you are able to resolve the style.css file conflict. Also please share the image after you changes the colors to a flat style by pinging me @divyanshudhruv

@newtoallofthis123
Copy link
Author

Here are the colors @divyanshudhruv
image

@divyanshudhruv
Copy link
Owner

Nice, but you can increase the padding of button just like the Click me button and instead of going column wise, go row wise. Also set the width to fit content. Or just copy the button.css code, do necessary changes to color only and then write the code for button. I'll share you a pic soon about what we need for links

@newtoallofthis123
Copy link
Author

I'll look into it

@divyanshudhruv
Copy link
Owner

divyanshudhruv commented Oct 1, 2023

I hope you didn't mind if I merge this request after 6 Oct as currently I do not have my device to look into the code.

@newtoallofthis123
Copy link
Author

Oh that's alright! Take your time :)

@newtoallofthis123
Copy link
Author

Something like this @divyanshudhruv
image

@divyanshudhruv
Copy link
Owner

Yeah, just add more padding and increase border radius by 3-4 px. You may also add a small line explaining that These are some of my social media handels Just like this-

@newtoallofthis123
Copy link
Author

done

image

@divyanshudhruv
Copy link
Owner

divyanshudhruv commented Oct 1, 2023

Screenshot_20231001_115517_Chrome

Just like from LINE 51 to LINE 58 (it will add some apace after the title and the small line. It will Make the portolio look more minimal). Uncomment that do necessary changes and see the result.

Also use the fonts that are being used in the title and the small title

Use the prebuild button.css for styling the social media buttons

@divyanshudhruv
Copy link
Owner

Actually I am on my tab, and I can merge you pr only after 6oct. Sorry for late merge

@newtoallofthis123
Copy link
Author

Don't worry about it

@newtoallofthis123
Copy link
Author

😄 Look good?
image

@divyanshudhruv divyanshudhruv linked an issue Oct 1, 2023 that may be closed by this pull request
@newtoallofthis123
Copy link
Author

So I can commit?

@divyanshudhruv
Copy link
Owner

Yeah ! Just a little more margin bottom to the small title. Then it would be fine

@newtoallofthis123
Copy link
Author

Okay

@divyanshudhruv
Copy link
Owner

Is the button animation working? If not, copy the button code from the about section and put it here. 🙂

@newtoallofthis123
Copy link
Author

Animations would not work because the background is declared by the user.

@divyanshudhruv
Copy link
Owner

May share a picture of final commit. Other than this, nice work. ### Can I commit to your repo if I need some more changes ?

@newtoallofthis123
Copy link
Author

newtoallofthis123 commented Oct 1, 2023

cool :)

I did do the final commit

@divyanshudhruv
Copy link
Owner

Animations would not work because the background is declared by the user.

Ok, Can I fix that ? By pushing code to your main branch ?

@newtoallofthis123
Copy link
Author

sure!

@divyanshudhruv
Copy link
Owner

Can you send the pic of the final code(the social link section)

@newtoallofthis123
Copy link
Author

okay

image

@divyanshudhruv
Copy link
Owner

And the website ?

@divyanshudhruv
Copy link
Owner

Any updates ?

@newtoallofthis123
Copy link
Author

It is actually really weird to add button animations with the background color. It is trade off at this point. I mean, the animation we added to other buttons was a darkening effect...but there is no way we can do that dynamically in css unless we use sass.

What do you say?

@divyanshudhruv
Copy link
Owner

Hey can you send a pic ? Of the website (social section)

@divyanshudhruv
Copy link
Owner

Please resolve the conflicts @newtoallofthis123

@divyanshudhruv
Copy link
Owner

@newtoallofthis123 can you publish the forked repo to gh pages? Then I can easily see the changes. Also send me the link after u successfully published

@newtoallofthis123
Copy link
Author

Please resolve the conflicts @newtoallofthis123

Yeah I'll get to it

@newtoallofthis123
Copy link
Author

@newtoallofthis123 can you publish the forked repo to gh pages? Then I can easily see the changes. Also send me the link after u successfully published

I can't really publish to gh pages since I don't own the repo....I'll see if i can get you a love preview

@divyanshudhruv
Copy link
Owner

Nono I mean publish your forked repo. That you can do I think

@newtoallofthis123
Copy link
Author

Oh okay

@divyanshudhruv
Copy link
Owner

Ok

@Mukesh-ghildiyal
Copy link

if u don't mind shall i raise my new pr for this social sections,because i think its taking quite too much time

@divyanshudhruv
Copy link
Owner

@newtoallofthis123 yea, you can open a new PR

@divyanshudhruv divyanshudhruv added hacktoberfest-accepted Accepted for hacktoberfest 🔒 Closed The issue/pull request had been closed labels Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔒 Closed The issue/pull request had been closed 🎉 Hacktoberfest hacktoberfest-accepted Accepted for hacktoberfest
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🔥 Add social media links
3 participants