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

Citizen Engagement (desktop): Update padding/margins and SDG image size #3738

Closed
8 tasks
Tracked by #3328
JessicaLucindaCheng opened this issue Nov 22, 2022 · 16 comments · Fixed by #3844
Closed
8 tasks
Tracked by #3328

Citizen Engagement (desktop): Update padding/margins and SDG image size #3738

JessicaLucindaCheng opened this issue Nov 22, 2022 · 16 comments · Fixed by #3844
Assignees
Labels
Complexity: Medium P-Feature: Citizen Engagement https://www.hackforla.org/citizen-engagement ready for design lead role: front end Tasks for front end developers size: 1pt Can be done in 4-6 hours Status: Updated No blockers and update is ready for review
Milestone

Comments

@JessicaLucindaCheng
Copy link
Member

JessicaLucindaCheng commented Nov 22, 2022

Overview

We want to update the size of the Sustainable Development Goal (SDG) image and adjust the padding and margins on the Citizen Engagement page for the desktop view so that

  • the website reflects the updated design
  • and is more accessible.

Details

Below is a screenshot of the updated Citizen Engagement page desktop design but please view and inspect the design in Figma to see specific details.

Expand this to see an updated Citizen Engagement page design

Citizen Engagement Desktop Final

Action Items

Developer

  • Read the Citizen Engagement wiki page for a list of code files
  • For desktop views,
    • Update the SDG image size
    • Check and update the padding and margins
  • Once your pull request has been merged,
    • Please update the Page screenshot (current version) section in the Citizen Engagement wiki page with an updated desktop and mobile screenshot from the website, and update the date.
      • Use the GoFullPage Chrome extension to screenshot the whole webpage at once.

Product Manager

Resources/Instructions

@JessicaLucindaCheng JessicaLucindaCheng added role: front end Tasks for front end developers Complexity: Medium P-Feature: Citizen Engagement https://www.hackforla.org/citizen-engagement size: 1pt Can be done in 4-6 hours Draft Issue is still in the process of being created labels Nov 22, 2022
@github-actions github-actions bot removed the Draft Issue is still in the process of being created label Nov 22, 2022
@JessicaLucindaCheng JessicaLucindaCheng added the ready for dev lead Issues that tech leads or merge team members need to follow up on label Nov 22, 2022
@JessicaLucindaCheng JessicaLucindaCheng added Draft Issue is still in the process of being created and removed ready for dev lead Issues that tech leads or merge team members need to follow up on labels Nov 22, 2022
@JessicaLucindaCheng

This comment was marked as resolved.

@JessicaLucindaCheng

This comment was marked as resolved.

@JessicaLucindaCheng JessicaLucindaCheng changed the title Citizen Engagement: Update SDG size and padding Citizen Engagement: Update padding/margins and SDG image size Dec 1, 2022
@JessicaLucindaCheng JessicaLucindaCheng changed the title Citizen Engagement: Update padding/margins and SDG image size Citizen Engagement (desktop): Update padding/margins and SDG image size Dec 1, 2022
@JessicaLucindaCheng

This comment was marked as resolved.

@github-actions
Copy link

Hi @SZwerling, thank you for taking up this issue! Hfla appreciates you :)

Do let fellow developers know about your:-
i. Availability: (When are you available to work on the issue/answer questions other programmers might have about your issue?)
ii. ETA: (When do you expect this issue to be completed?)

You're awesome!

P.S. - You may not take up another issue until this issue gets merged (or closed). Thanks again :)

@SZwerling
Copy link
Member

I will start on this tomorrow, 12/12/22. I plan to be done before the weekend.

@SZwerling
Copy link
Member

Hello. I have a blocker.
#3738 Update the SDG image size and Check and update the padding and margins.
Looking at the Figma design I cannot ascertain the padding and margins. I do see pixel distance between elements.
Also, update padding and margin just for the SDG image? Or for which elements?
Any help would be appreciated. Thank you.

@JessicaLucindaCheng
Copy link
Member Author

JessicaLucindaCheng commented Dec 15, 2022

@SZwerling

Greetings. I hope your day is going well.
I'm working on #3738 resizing the SDG and adding margin/padding.
I have a few questions. Like, how do I determine the margin and padding from a Figma file?
Also, this issue asks to "check and update padding and margins". For every element? Or part of the page in particular?
Is it alright that I'm asking you these questions via a direct message?
Thanks,
Seth

Hi Seth,
I got your message on Slack. Since we (Hack for LA) are on break until January 3rd, I'm not checking messages as often.

how do I determine the margin and padding from a Figma file?

To determine the distance between elements in Figma, you need to

  • Sign in to Figma
  • Go to the design you want to view and zoom in so you can see the element(s) of the design better
  • Make sure your cursor is set to the move button (the one that looks like an arrow) and click on the element you want to examine (the element you selected will have a blue outline around it)
  • To see its position/relation to other elements, you can hover your cursor over other elements.
  • For more info about the element you selected, you can select the Inspect tab on the right and scroll down in the tab to see more info
Click here to see a gif example of how I examine an element

how to examine a figma element

For every element? Or part of the page in particular?

Since I didn't design the page, I'm not sure exactly what margins and paddings were changed. However, I don't think the margins and padding within the webpage's header section, within the top navigation, and within the footer were changed.
How I would approach it would be the following: After adjusting the SDG image size, I would start with inspecting the padding and margin around the SDG image on the webpage and comparing it to the Figma design. Then expand checking from there. So, if I were doing it, I would check starting with the SDG image, the elements in the SDG mini-card, the project card, then possibly the lower half of the webpage if needed.

Is it alright that I'm asking you these questions via a direct message?

I'm okay with it but you may work on other issues where the issue writer has left the team or are no longer active at Hack for LA. Thus, I would recommend doing the following instead if you have a blocker:

  • In a comment in your issue, write what your blocker is and/or what questions you may have.

  • Get a link to your comment by clicking on the 3 dots on the top right of your comment and selecting "Copy link"

    If you need it, you can click here to see a gif of how to copy a link to a GitHub comment

    how to get a link

  • In a message, paste the link to your comment in #hfla-site Slack channel. This will allow more people on the Website Team to see it versus just a direct message and they may be able to give you an answer.
  • Also, when we have meetings, you can bring up your blocker during the meeting so that it can get addressed.

@github-actions github-actions bot added the Status: Updated No blockers and update is ready for review label Dec 16, 2022
@github-actions github-actions bot removed the Status: Updated No blockers and update is ready for review label Dec 23, 2022
@github-actions

This comment was marked as outdated.

@github-actions github-actions bot added the 2 weeks inactive An issue that has not been updated by an assignee for two weeks label Dec 23, 2022
@SZwerling
Copy link
Member

Progress: At the beginning.
Blockers: In the figma file there is no differentiation between padding and margin. If distances between elements/components have changed in the figma file, how do I determine to change padding and/or margin?
Availability: I will have time after Christmas this Sunday.
ETA: Not sure.

@SZwerling SZwerling added Status: Updated No blockers and update is ready for review and removed 2 weeks inactive An issue that has not been updated by an assignee for two weeks labels Dec 24, 2022
@github-actions github-actions bot added 2 weeks inactive An issue that has not been updated by an assignee for two weeks and removed Status: Updated No blockers and update is ready for review labels Jan 6, 2023
@github-actions

This comment was marked as outdated.

@ExperimentsInHonesty
Copy link
Member

@SZwerling if you still need an answer to this

In the figma file there is no differentiation between padding and margin. If distances between elements/components

Please reach out to the development leads. If you already have your answer, please post the answer on in a comment on this issue.

@SZwerling
Copy link
Member

Inspecting the Figma design for the Citizen Engagement Page, specifically the .sustainable-dev-goal cards, I see that the spacing is not consistent, but the size of the sdg-image is consistent.
My solution was to change the .sdg-image image height to 145px and the .sustainable-dev-goal padding to 20px for media query bp-tablet-up.
Checking to see if this is acceptable before I make a pull request.
Screenshot of my solution included.
screencapture-localhost-4000-citizen-engagement-2023-01-12-13_56_24

@ExperimentsInHonesty
Copy link
Member

@SZwerling I don't see the PR associated with this since you were assigned, but I see you closed this issue after making the comment 2 days ago. Can you let me know what's happening? If there is a PR, make sure its linked here.

@SZwerling
Copy link
Member

@ExperimentsInHonesty I did not mean to close the issue. That was an accident.
I am waiting to hear back if my solution is acceptable before making a pull request.

@JessicaLucindaCheng
Copy link
Member Author

JessicaLucindaCheng commented Jan 17, 2023

@SZwerling

Inspecting the Figma design for the Citizen Engagement Page, specifically the .sustainable-dev-goal cards, I see that the spacing is not consistent, but the size of the sdg-image is consistent. My solution was to change the .sdg-image image height to 145px and the .sustainable-dev-goal padding to 20px for media query bp-tablet-up. Checking to see if this is acceptable before I make a pull request. Screenshot of my solution included.

I'm sorry about the confusion with the spacing. However, thank you for catching that our design wasn't uniform in terms of spacing. I spoke with @ExperimentsInHonesty about this and she said the following:

  • The spacing should be 20px. (See the red annotations in the diagrams below.)
    • The bottom spacing should be 20px to whichever is longer: the image (like in diagram A) or the text block (like in diagram B).
  • The top of the image should align with the top of the text block (see diagram A's purple annotation).

Diagram A

Diagram B

@github-actions github-actions bot added Status: Updated No blockers and update is ready for review and removed 2 weeks inactive An issue that has not been updated by an assignee for two weeks labels Jan 20, 2023
@ExperimentsInHonesty
Copy link
Member

This is going to go into the new issue approval column for the design lead to make an issue to update the figma

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Complexity: Medium P-Feature: Citizen Engagement https://www.hackforla.org/citizen-engagement ready for design lead role: front end Tasks for front end developers size: 1pt Can be done in 4-6 hours Status: Updated No blockers and update is ready for review
Projects
Status: New Issue Approval
Development

Successfully merging a pull request may close this issue.

3 participants