hackforla / hackforla/website

ER: Headers Are Different Heights From the Top Navigation Bar in Mobile Views

Open
#7,510 9 comments 0 reactions 1 assignee Claimed by @JessicaLucindaCheng View on GitHub
Complexity: Medium Draft ER HLC: S P-Feature: Donate P-Feature: Join Page role: back end/devOps role: front end size: 0.25pt
Dominant language
JavaScript
Stars
363
Forks
872
Avg merge
2d 21h
Merged PRs (30d)
18

Description

### Emergent Requirement - Problem
The Donate page header and the Join Us header are different heights from the navigation bar at the top of the page in mobile and tablet views (screen width 767px or less).

Click here to see a gif of the problem

![join donate mobile padding](https://github.com/user-attachments/assets/9e0820f6-249e-47a7-911d-bd12deaa2344)

### Issue you discovered this emergent requirement in
- Not applicable because from Task List Dev Leads

### Date discovered
2022-06-04

### Did you have to do something temporarily
- [ ] YES
- [x] NO

### Who was involved
@JessicaLucindaCheng
@ExperimentsInHonesty

### What happens if this is not addressed
The pages do not look uniformly formatted, which reduces the site's professionalism.

### Resources
- [Donate page](https://www.hackforla.org/donate/)
- [Join Us page](https://www.hackforla.org/join)
- [Site Map](https://www.hackforla.org/sitemap/)

### Recommended Action Items
- [x] Make a new issue
- [ ] Discuss with team
- [ ] Let a Team Lead know

### Potential solutions [draft]
Create an issue including but not limited to the following Action Items:
- [ ] Create a spreadsheet to track the results of the audit
- [ ] Audit spacing between the top navigation bar and the first header on each web page in tablet and mobile views (screen widths 767px and 479px)
- [ ] In a comment, write your recommendation(s) for how to solve this problem.
- [ ] Discuss with a Merge Team member on a plan to fix it. The plan can include just fixing the problem if the solution is simple, writing ER(s) to implement a solution, or it may require writing an epic because it is complex.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.