NIAEFEUP / nijobs-fe

[FRONTEND] A platform for companies to advertise their job opportunities to students
GNU General Public License v3.0
23 stars 3 forks source link

Feature/privacyPolicyandTermsandConditions #185

Closed CarlosMealha closed 2 years ago

CarlosMealha commented 2 years ago

closes #185 Hey there, the Privacy Policy and Terms and Condition pages are already done, just need to style .md files.

CarlosMealha commented 2 years ago

Already changed the .md files to .js, because there were some problems testing TermsAndConditionsPage and PrivacyAndPolicyPage, and still don't know if the tests that I've done are enough.

CarlosMealha commented 2 years ago

Okay, I will see what I can do to make it more consistent! Do you prefer PP title or T&C?

CarlosMealha commented 2 years ago

Currently the pages are looking like this, but I'm using <b> instead of useStyles, because when I try to use it, the page is all white. The useStyles that I was using looks like this: const useStyles = makeStyles ({ bold: { fontWeight: 600, }, }); And to call it in the content function, I used: <Typography className={ classes.bold }>

If you have any idea about how I can solve this problem, please tell me, otherwise I will use <b>

Screenshot from 2022-03-01 23-42-38

TiagooGomess commented 2 years ago

Currently the pages are looking like this, but I'm using <b> instead of useStyles, because when I try to use it, the page is all white. The useStyles that I was using looks like this: const useStyles = makeStyles ({ bold: { fontWeight: 600, }, }); And to call it in the content function, I used: <Typography className={ classes.bold }>

If you have any idea about how I can solve this problem, please tell me, otherwise I will use <b>

Screenshot from 2022-03-01 23-42-38

const classes = useStyles(); should be inside the component.

TiagooGomess commented 2 years ago

Okay, I will see what I can do to make it more consistent! Do you prefer PP title or T&C?

PP title.

CarlosMealha commented 2 years ago

Currently the pages are looking like this, but I'm using <b> instead of useStyles, because when I try to use it, the page is all white. The useStyles that I was using looks like this: const useStyles = makeStyles ({ bold: { fontWeight: 600, }, }); And to call it in the content function, I used: <Typography className={ classes.bold }> If you have any idea about how I can solve this problem, please tell me, otherwise I will use <b> Screenshot from 2022-03-01 23-42-38

const classes = useStyles(); should be inside the component.

I have already done that, but it keeps returning white screen.

CarlosMealha commented 2 years ago

Currently the pages are looking like this, but I'm using <b> instead of useStyles, because when I try to use it, the page is all white. The useStyles that I was using looks like this: const useStyles = makeStyles ({ bold: { fontWeight: 600, }, }); And to call it in the content function, I used: <Typography className={ classes.bold }> If you have any idea about how I can solve this problem, please tell me, otherwise I will use <b> Screenshot from 2022-03-01 23-42-38

const classes = useStyles(); should be inside the component.

I have already done that, but it keeps returning white screen.

CarlosMealha commented 2 years ago

I still need to move the gray CSS property to the AppTheme.

TiagooGomess commented 2 years ago

Update this broken link.

CarlosMealha commented 2 years ago

Update this broken link.

Okay, it's already done.

TiagooGomess commented 2 years ago

Rebase and it's ready

CarlosMealha commented 2 years ago

Rebase and it's ready

Okay, I'll do it now.