Skip to content

add some adjustment to the home page#21

Open
vnxx wants to merge 2 commits into
fhrf:masterfrom
vnxx:master
Open

add some adjustment to the home page#21
vnxx wants to merge 2 commits into
fhrf:masterfrom
vnxx:master

Conversation

@vnxx

@vnxx vnxx commented Oct 28, 2020

Copy link
Copy Markdown

Hi @cTxplorer , I add some adjustment to the home page, here I attach some screenshot of my changes:

I made the front-cover full (width) and the text center.
Screenshot 2020-10-28 at 12 32 09 pm

I removed the shadow and change the icon color so it looks more clear (I guess)
Screenshot 2020-10-28 at 12 32 45 pm

and for mobile screen i add margin between the card
Screenshot 2020-10-28 at 12 54 57 pm

@vercel

vercel Bot commented Oct 28, 2020

Copy link
Copy Markdown

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/fhrf/fhrfin/3qtrs4nui
✅ Preview: https://fhrfin-git-master.fhrf.vercel.app

@cTxplorer

Copy link
Copy Markdown
Collaborator

Hey @vnxx thanks for the PR.
May I know what are is resolved with the first point that you mentioned?

I made the front-cover full (width) and the text center.

@vnxx

vnxx commented Oct 30, 2020

Copy link
Copy Markdown
Author

Thanks for the response @cTxplorer , maybe this image can explain what I mean 😅

exp

@cTxplorer cTxplorer left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

oh, okay 👍

Comment thread styles/config.css Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants