-
Notifications
You must be signed in to change notification settings - Fork 190
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
Coding: Develop the following section for Home Page #110
Comments
@anitab-org/coding-team can anyone take up this issue? |
@vj-codes the margins are incorrect |
@Sonal240 which ones? I checked in the doc didn't find any |
overall margin: 32pt |
@Sonal240 I see a mess has been sent on zulip to , will change accordingly.BTW the metrics say body is still 14 pt , when was it changed to 18pt? |
Well I remember last time when Anna had introduced the doc she mentioned this...also note that if body is 14pt then how can View More be 16pt? Body should be given more emphasis than view more right? |
@vj-codes |
@Sonal240 @nandini45 check once please! |
LGTM @vj-codes !! |
I am willing to work on this. |
Will you be providing the four images as present in design or can i use any sample image? |
Give me 10 min I'll share link for development |
https://xd.adobe.com/view/09a9fdf0-a193-4b3f-47c2-216055c7ca42-7f98/ |
Yes now its working, i am able to download |
I also wanted to ask that what are the dimensions of the image to be displayed? |
kindly click on the image and see the properties |
@raksh543 Are you working on the changes requested in the PR? |
@nandini45 @Sonal240 I don't see updates from the current assignee for the requested changes in the PR . |
@Sonal240 I would continue to work on this. Pr for this issue have already been made. There are some minor fixes needed, so i would complete that. |
Kindly complete them asap @raksh543 |
@nandini45 I think this work is already done. Can you please confirm? |
@nandini45 No, I think the work is already merged as it's visible to me and we no longer require that PR (or this issue). |
@nandini45 hey is this issue still open, can I work on this? |
1 similar comment
@nandini45 hey is this issue still open, can I work on this? |
@shades-7 i dont know the current status of the issue. i will get back to you |
Hi! Is this issue still open? I would like to work on it. |
This issue must be closed since all the asked sections were already merged |
Is your feature request related to a problem? Please describe.
As a developer,
I need the following section to be developed so that I can add is as a Home component here
Mocks
Develop this section only
overall margin: 32pt
Margin top for GETTING STARTED heading: 64pt
In between each section: 32pt
distance between image and text: 32pt
distance from horizontal line: 32pt
Avenir Body light 18pt
Heading 24pt
Text Color:#103B81
Design link for development
Acceptance Criteria
Update [Required]
Definition of Done
Estimation
6-8 hours
The text was updated successfully, but these errors were encountered: