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
New Footer Style #38
New Footer Style #38
Conversation
- Define default style for hyperlink and heading
This pull request is being automatically deployed with Vercel (learn more). |
@nabeel- any idea why the test actions are not being fun? |
@mmahalwy can you check why Vercel deployment taking too long? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks so much for your help so far. Left a few comments!
src/components/Footer.tsx
Outdated
import React from 'react'; | ||
import { Container } from 'styled-bootstrap-grid'; | ||
import styled from 'styled-components'; | ||
import React from "react"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@nabeel- I feel like ESlint should have caught double quotes here, no?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think its better if we have prettier on the development level, as in during the coding time and it will auto format on save
@MistaPidaus Agreed! We had this in the commit/PR I mentioned above but it seems to have disappeared from master. There is also a pre-commit hook that will auto-fix errors, too
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@nabeel- I see. It looks like it.
Please, no more force push. It will ruin the project.
@MistaPidaus left some comments |
- Line height value to rem
…next into enhance/footer
@mmahalwy updated the PR |
@mmahalwy Updated PR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, storybook is failing for some reason
@mmahalwy added story |
Enhance/Improve Footer styling based on #9
cc: @mmahalwy