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

feature-homepage-launch branch: Fix projects' info pages' styling of the header #3377

Closed
11 tasks done
JessicaLucindaCheng opened this issue Jul 17, 2022 · 7 comments
Closed
11 tasks done
Assignees
Labels
Bug Something isn't working Complexity: Small Take this type of issues after the successful merge of your second good first issue Feature: Feature Branch Requires Branching off a Feature Branch instead of gh-pages P-Feature: Project Info and Page A project's detail page (e.g. https://www.hackforla.org/projects/100-automations) role: front end Tasks for front end developers size: 0.5pt Can be done in 3 hours or less Status: Updated No blockers and update is ready for review

Comments

@JessicaLucindaCheng
Copy link
Member

JessicaLucindaCheng commented Jul 17, 2022

Overview

As developers, we need to fix the styling of the first header in the projects' info pages in the feature-homepage-launch branch so that it matches the styling of those pages on the live website.

Details

Click here to see examples of what the header should look like on the current projects' info pages 100Automations Home Unite Us Spare
Click here to see examples of what the header looks like on the feature-homepage-launch projects' info pages 100 Automations Home Unite Us Spare

On the feature-homepage-launch branch, the problem crept in on the project info pages when the SCSS class hero-inner was removed from home.scss as part of pr #2373, which was the initial pr on the feature-homepage-launch branch. I think the problem happened because a lot of the code originally in home.scss was removed/changed in order to style the redesigned homepage, including the removal of class hero-inner.

Action Items

Resources/Instructions

@JessicaLucindaCheng JessicaLucindaCheng added Bug Something isn't working role: front end Tasks for front end developers Complexity: Large P-Feature: Project Info and Page A project's detail page (e.g. https://www.hackforla.org/projects/100-automations) Feature: Feature Branch Requires Branching off a Feature Branch instead of gh-pages size: 1pt Can be done in 4-6 hours Draft Issue is still in the process of being created labels Jul 17, 2022
@ExperimentsInHonesty

This comment was marked as resolved.

@JessicaLucindaCheng JessicaLucindaCheng added Ready for Prioritization and removed Draft Issue is still in the process of being created labels Sep 12, 2022
@JessicaLucindaCheng

This comment was marked as resolved.

@ExperimentsInHonesty
Copy link
Member

@JessicaLucindaCheng I am confused, how did a problem creep in on the project info pages? Why did they change at all? What do they have to do with the Homepage launch?

@ExperimentsInHonesty ExperimentsInHonesty added ready for dev lead Issues that tech leads or merge team members need to follow up on and removed Ready for Prioritization labels Sep 21, 2022
@JessicaLucindaCheng JessicaLucindaCheng added Complexity: Small Take this type of issues after the successful merge of your second good first issue size: 0.5pt Can be done in 3 hours or less and removed Complexity: Large size: 1pt Can be done in 4-6 hours labels Sep 21, 2022
@JessicaLucindaCheng
Copy link
Member Author

JessicaLucindaCheng commented Sep 21, 2022

@ExperimentsInHonesty

I am confused, how did a problem creep in on the project info pages?

The problem crept in on the project info pages when the SCSS class hero-inner was removed from home.scss as part of pr #2373, which was the initial pr on the feature-homepage-launch branch.

Why did they change at all?

I think it changed/problem happened because a lot of the code originally in home.scss was removed/changed in order to style the redesigned homepage, including the removal of class hero-inner.

What do they have to do with the Homepage launch?

The problem had to do with the Homepage launch because

  • the problem only occurs on the feature-homepage-launch branch and not on the gh-pages branch
  • and it happened because of the removal of the hero-inner class from home.scss, when a lot of the code in home.scss was changed/removed in order to style the redesigned homepage.

Thus, I thought it would be best to address the problem in the feature-homepage-launch branch since that is where the problem is visible.

Also, I have updated the issue with more details about the problem. I am adding the ready for milestone back on because I am not sure what label to add to let you or other PMs know to take a look at the issue again.

@JessicaLucindaCheng JessicaLucindaCheng added Ready for Prioritization and removed ready for dev lead Issues that tech leads or merge team members need to follow up on labels Sep 21, 2022
@GRISONRF GRISONRF self-assigned this Sep 26, 2022
@github-actions
Copy link

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

@GRISONRF
Copy link
Member

Availability: 3h 09/26/2022
ETA: 09/26/2022

@ExperimentsInHonesty
Copy link
Member

Manually closing (instead of the PR closing it on merge) this issue because its on a feature branch, so build in automations are not working.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Complexity: Small Take this type of issues after the successful merge of your second good first issue Feature: Feature Branch Requires Branching off a Feature Branch instead of gh-pages P-Feature: Project Info and Page A project's detail page (e.g. https://www.hackforla.org/projects/100-automations) role: front end Tasks for front end developers size: 0.5pt Can be done in 3 hours or less Status: Updated No blockers and update is ready for review
Projects
Development

No branches or pull requests

4 participants