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

Increase width of event columns on Events page #4271

Closed
6 tasks done
jdingeman opened this issue Mar 24, 2023 · 7 comments · Fixed by #4425
Closed
6 tasks done

Increase width of event columns on Events page #4271

jdingeman opened this issue Mar 24, 2023 · 7 comments · Fixed by #4425
Assignees
Labels
Complexity: Medium P-Feature: Events https://www.hackforla.org/events/ role: front end Tasks for front end developers size: 2pt Can be done in 7-12 hours

Comments

@jdingeman
Copy link
Member

jdingeman commented Mar 24, 2023

Overview

The white columns on the Events page are very narrow, which causes the events and meeting title text to wrap. We want to make the columns on the Events page wider so that the content is easier to read.

Action Items

  • Edit the code in /sass/components/_events.scss to make the columns wider
  • First try matching the width of the columns to meet the edge of the "Our Locations" images below it
Referenced edges

image

  • Try out other widths and placements that may look good instead of the current layout
  • Add images of different potential solutions to this issue
  • Consult with Product/Bonnie on the best layout from your proposals
  • Create a pull request with the changes of the final decision

Resources/Instructions

Full page screenshot of Events page

image

@github-actions
Copy link

Hi @kwangric, 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 :)

@kwangric
Copy link
Member

i. Availability: Weekdays after 1pm PDT
ii. ETA: 3/28 afternoon

@kwangric
Copy link
Member

Below are two potential solutions.

Solution A
  • Increased overall width of the event columns to match the width of the "Our Locations" images below. Also centered time zone text.
  • Increased margins between the event columns to 32px to match the margins between the location images.

Solution A

Solution B
  • Same changes as Solution A but kept the margins between the event columns the same as before (16 px).

Solution B

Current Layout

Current

@jdingeman
Copy link
Member Author

@kwangric - nice job! I think both layouts look really good, but let's hear from @ExperimentsInHonesty for which solution may be the best to go with. I am going to put this into Questions/In Review and add the label so she has visibility of your potential solutions.

@ExperimentsInHonesty
Copy link
Member

@kwangric I literally can't tell the difference between A and B

Perhaps screenshots of the parts that are different side by side or put stars next to the difference.

lol, this is like one of those can you spot the differences games!

@kwangric
Copy link
Member

kwangric commented Apr 3, 2023

@ExperimentsInHonesty Below are the differences between A and B circled in green. The only difference is the width between the columns. Solution B has the original width, while in Solution A I increased the width to match the location pictures at the bottom of the page. Let me know if that makes it any more clear!

Changes

@ExperimentsInHonesty
Copy link
Member

@kwangric Thank you!!! that was really helpful. Solution A please.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Complexity: Medium P-Feature: Events https://www.hackforla.org/events/ role: front end Tasks for front end developers size: 2pt Can be done in 7-12 hours
Projects
Development

Successfully merging a pull request may close this issue.

3 participants