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

Added space between Founder and footer section #161

Closed
wants to merge 1 commit into from
Closed

Added space between Founder and footer section #161

wants to merge 1 commit into from

Conversation

bishal7679
Copy link
Contributor

Signed-off-by: Bishal Das [email protected]

Fixes Issue #137

Closes #137

Changes proposed

Added space between Founder and footer section

Screenshots

Screenshot (278)

@bishal7679
Copy link
Contributor Author

@AvineshTripathi Please take a look

__
sema-logo  Summary: 🛠️ This code needs a fix  |  Tags: Inefficient

display: flex;
flex-direction: row;
background: #1B2731;
margin-bottom: 74px;
Copy link
Collaborator

Choose a reason for hiding this comment

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

DO you think making this relative would be better?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I didn't get you
Could you elaborate what do you mean by relative
Position relative or anything else

Copy link
Collaborator

Choose a reason for hiding this comment

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

position relative maybe in %

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You can coauthor this and I will commit your suggestion

Copy link
Collaborator

Choose a reason for hiding this comment

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

@bishal7679 this issue is being perfectly handled in the #175 PR by working on the footer component, If you want I can add you there as a collaborator

@AvineshTripathi
Copy link
Collaborator

will merge this PR and do that relative thing in next PRs comming

@bishal7679 bishal7679 closed this by deleting the head repository Sep 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE] Need some space between Founder and footer section
2 participants