Events Section - BrickHack Club Site #1219
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
✔️ Deploy Preview for infallible-lumiere-e9d357 ready! 🔨 Explore the source changes: d8d8540 🔍 Inspect the deploy log: https://app.netlify.com/sites/infallible-lumiere-e9d357/deploys/6125bf0bcb5aa60007880d06 😎 Browse the preview: https://deploy-preview-1219--infallible-lumiere-e9d357.netlify.app/bhclub |
peterkos
reviewed
Aug 23, 2021
That was an oopsie from me, used padding when trying to remove the margin-left on the text.
That is a design question. I made a ratio for the nav width based on the figma. We could decrease that width on larger screens? What do they want? I could also
Tried to fix this a bit by shortening the container on desktop
Adding |
peterkos
approved these changes
Aug 25, 2021
Co-authored-by: Peter Kos <[email protected]>
peterkos
added a commit
that referenced
this pull request
Aug 25, 2021
* file creation * site outline * making link from index * fixing typos * content outline + positioning navbar * Navbar styling (normal + mobile) * navbar functionality * file cleanup * more general styles * making the navbar close when clicked outside of it * fixes based on 7/22 design meeting * Fixes * footer * filling out content section * font spacing * image alt text * color fixes * design changes * attempting to avoid merge conflicts, pray for me * broken fixes * more fixes + making parcel work w/ temp fix * undo the temporary fix * fix clipping border radius * About Section - BrickHack Club Site (#1215) * about section + cards * about changes * changes from design meeting 7/29 * nit from github * more design fixes (colors + nav hover) * join link text fix * fixes * design fixes * fix for parcel * fixes * Landing Section - BrickHack Club Site (#1218) * landing section * Fix image scaling * Text width on mobile fix, add some moblbtn padding * content fixes * changing email Co-authored-by: Peter Kos <[email protected]> * Events Section - BrickHack Club Site (#1219) * events section * fixes * typo fix Co-authored-by: Peter Kos <[email protected]> Co-authored-by: Peter Kos <[email protected]> * Teams Section - BrickHack Club Site (#1220) * html + svgs * css * starting to fix the dropdown * fixing dropdown * making team info show/hide * updating svgs * fixes * BrickHack Club Site Fixes (#1222) * logo updates * fixes Co-authored-by: Peter Kos <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #1210