fix: mobile spacing for recommendation form labels (#2)#64
fix: mobile spacing for recommendation form labels (#2)#64KaleSujit9011 wants to merge 4 commits into
Conversation
|
@KaleSujit9011 is attempting to deploy a commit to the komalsony234-1530's projects Team on Vercel. A member of the Team first needs to authorize it. |
komalharshita
left a comment
There was a problem hiding this comment.
Nice work identifying the root cause and keeping the fix scoped specifically to mobile screens. The solution is clean, minimal, and resolves the spacing issue without affecting desktop layouts.
Before merging, please sync your branch with the latest main branch and resolve the current merge conflict in static/style.css.
|
Addressed the requested change and resolved the merge conflict in static/style.css. |
komalharshita
left a comment
There was a problem hiding this comment.
Thanks for the contribution and for resolving the merge conflicts.
However, this issue has already been fixed and merged through another PR with the same implementation approach. Since the functionality now already exists in the latest main branch, this PR would duplicate the same changes.
Because of that, this PR will be closed as a duplicate contribution. We still appreciate the effort you put into resolving the issue and updating the branch.
|
@KaleSujit9011 Please feel free to check out other open issues in the repository — there are several beginner and intermediate-friendly issues still available, especially in UI/UX improvements, accessibility, and feature enhancements. We’d love to see more contributions from you. |
Summary
On mobile screens below 640 px, the Experience Level label inside the
recommendation form was sitting too close to the input below it.
This happened because label elements are inline by default, which
causes margin-bottom to be ignored by the browser.
The fix adds display: block and margin-bottom: 8px to
.form-group label inside the existing @media (max-width: 640px)
block in static/style.css. This only affects mobile screens and leaves
desktop layout unchanged.
Related Issue
Closes #2
Type of Change
What Was Changed
static/style.cssHow to Test
Test Results
27 passed, 0 failed out of 27 tests
Screenshots
Before :

After Change :

Self-Review Checklist
Notes for Reviewer
The Python files show no errors from flake8.