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

Fix CSS alignment of social share buttons #3

Open
Ckbhatia opened this issue Oct 1, 2020 · 41 comments
Open

Fix CSS alignment of social share buttons #3

Ckbhatia opened this issue Oct 1, 2020 · 41 comments
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers hacktoberfest

Comments

@Ckbhatia
Copy link
Owner

Ckbhatia commented Oct 1, 2020

Fix social share alignment for following screens

  1. The social share position should not break when the article text isn't enough ( Desktop screen ).

You can refer to this for the issue or see image below: https://chetanblog.netlify.app/blog/introToGit

social_share

  1. It should not overlap the heading below it ( Mobile screen: width 414):

social_share_mobile

Note: please work on your fork with some dummy article data to reproduce the same issue on your local machine.

Please don't hesitate to ask for help if needed.

@Ckbhatia Ckbhatia added enhancement New feature or request good first issue Good for newcomers hacktoberfest up-for-grabs Developer can start working on issues that are labeled up for grabs labels Oct 1, 2020
@adsingh14
Copy link

I can help you with this. Just started working on it. Before starting, I have some questions:

  1. In screenshot 1, do you want to fix the position of social icons ?
  2. Are you using Chrome browser for this website ?

@Ckbhatia
Copy link
Owner Author

Ckbhatia commented Oct 1, 2020

Hi @adsingh14

  1. I want to fix positions of social icons.
  2. Yes, these screenshots are from chrome.

Please make sure about firefox too if possible.

@adsingh14
Copy link

adsingh14 commented Oct 1, 2020

IRT point 1. Do you want to keep the icons under stats text and on screen all the time?

No problem about firefox. I'm Firefox user.

@Ckbhatia
Copy link
Owner Author

Ckbhatia commented Oct 1, 2020

Hello @adsingh14

Can you achieve this? ( Optional )

Author profile and social icons should be aligned well with the cover image of the article like this:
tobe

@adsingh14
Copy link

Yes ! Does the icons and profile side remains fix on tablets and desktop ? (Take a look at the attachment)

image

@Ckbhatia
Copy link
Owner Author

Ckbhatia commented Oct 1, 2020

No, It is not fixed. I want you to fix like this.

@Ckbhatia
Copy link
Owner Author

Ckbhatia commented Oct 2, 2020

@adsingh14 Are you working on this?

@adsingh14
Copy link

Project isn't running on local. I've run yarn command and it stuck on step 2.

@niftyrinki
Copy link

niftyrinki commented Oct 2, 2020

@Ckbhatia Can I take this up?

@Ckbhatia
Copy link
Owner Author

Ckbhatia commented Oct 2, 2020

@adsingh14 What is the error?

@Ckbhatia
Copy link
Owner Author

Ckbhatia commented Oct 2, 2020

Have you followed the project setup guide?

@adsingh14
Copy link

adsingh14 commented Oct 2, 2020

Yes. I followed up Access Locally step from guide.

@adsingh14
Copy link

In the guide you've marked contentful step optional but it asks about .env.production.

Error: Failed to find .env file at path: .env.production

@adsingh14
Copy link

I skip this issue from my side. I can do CSS thing but incomplete guide stops me doing this. You can assign this to anyone.

@Ckbhatia
Copy link
Owner Author

Ckbhatia commented Oct 2, 2020

Can you please create .env.production and paste the same env variables?

@Ckbhatia
Copy link
Owner Author

Ckbhatia commented Oct 2, 2020

@niftyrinki Have you tried to set up this project?

@Ckbhatia
Copy link
Owner Author

Ckbhatia commented Oct 5, 2020

Please feel free to pick this issue as no one else picked it yet

@sachinttn9663
Copy link

Please feel free to pick this issue as no one else picked it yet

@Ckbhatia PLease assign it to me, want to work on it.

@Ckbhatia
Copy link
Owner Author

Ckbhatia commented Oct 5, 2020

@sachinttn9663 I am glad to see your interest to work on this issue. I will be happy to see your PR for this issue.

@sachinttn9663
Copy link

@Ckbhatia , I'm getting the following error Can you please check
Screenshot from 2020-10-05 13-24-07

@Ckbhatia
Copy link
Owner Author

Ckbhatia commented Oct 5, 2020

@sachinttn9663 Please use these env variables for development.

CONTENTFUL_SPACE_ID=0dxfiyfxoovv
CONTENTFUL_ACCESS_TOKEN=aQ8VH0ajavUGJur3I6LGRTZ

@sachinttn9663
Copy link

Still getting error
Screenshot from 2020-10-05 22-18-26

@Ckbhatia
Copy link
Owner Author

Ckbhatia commented Oct 5, 2020

Please check your internet connectivity and try again.

This kind of error is related to internet connectivity issue.

@Hackerbone
Copy link

I would Like to work on this as well

@Ckbhatia
Copy link
Owner Author

Ckbhatia commented Oct 6, 2020

@Hackerbone Please go ahead. I would love to see your PR.

@Ckbhatia
Copy link
Owner Author

Ckbhatia commented Sep 10, 2021

Hi @Hackerbone, Do you have any updates on this issue?

If not, Can you please work on it and raise PR before the upcoming HactoberFest, So I can close issues?

@Mukta-Sawant
Copy link

@Ckbhatia Can I work on this as a part of Hacktoberfest 2021?

@Ckbhatia
Copy link
Owner Author

Ckbhatia commented Oct 1, 2021 via email

@ayasahmad7655
Copy link

can I work on it as well?

@ayasahmad7655
Copy link

I will improve the social bar in the mobile app.

@Ckbhatia
Copy link
Owner Author

Ckbhatia commented Oct 4, 2021

@Mukta-Sawant Any update?

@Mukta-Sawant
Copy link

@Ckbhatia sir I need help in setting up the project locally.

@Ckbhatia
Copy link
Owner Author

Ckbhatia commented Oct 4, 2021 via email

@adsingh14
Copy link

Hi @Ckbhatia ! I would like to work on this. Shall I wait to get assigned ?

@adsingh14

This comment has been minimized.

@Ckbhatia
Copy link
Owner Author

Ckbhatia commented Oct 10, 2021 via email

@adsingh14
Copy link

adsingh14 commented Oct 10, 2021

Have you updated components\Mode.js in your system? That file also needs to update. Apart from that, I've updated the node-sass package but still getting webpack error.

@Ckbhatia Ckbhatia removed the up-for-grabs Developer can start working on issues that are labeled up for grabs label Oct 10, 2021
@Ckbhatia
Copy link
Owner Author

No component isn't updated. I have an m1 mac. There are some dependency issues on this system. So, I need to dockerize to run it.

@adsingh14
Copy link

It'd be great if you add docker commands in this repo, if that solves the issue.

@Ckbhatia
Copy link
Owner Author

On most other system, it doesn't need docker.

@Erum640
Copy link

Erum640 commented Oct 6, 2022

Please assign me this issue i want to work on it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers hacktoberfest
Projects
None yet
Development

No branches or pull requests

8 participants