Skip to content

Conversation

peterkos
Copy link
Contributor

@peterkos peterkos commented Jun 24, 2021

Background

Implements #1200

This hints at the windows 95-style retro look for the full BH8 website.

Textbox functionality will be added in next PR!

Preview

CleanShot 2021-06-24 at 16 32 03

@peterkos peterkos added the BH8 Ohhhh yea! label Jun 24, 2021
@peterkos peterkos self-assigned this Jun 24, 2021
sass/club.scss Outdated
Comment on lines 19 to 20
transform: scale(1.5);
transform-origin: center;
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A teensy bit hacky -- the whole site was really small and a bump up in scaling makes it look much nicer.

An important note from MDN on zoom vs. transform:

[zoom CSS property] is non-standard and is not on a standards track. Do not use it on production sites facing the Web: it will not work for every user. There may also be large incompatibilities between implementations and the behavior may change in the future.

Note that it's not supported in Firefox either due to it not being in the CSS standard, which is why I used transform.

@netlify
Copy link

netlify bot commented Jun 24, 2021

✔️ Deploy Preview for infallible-lumiere-e9d357 ready!

🔨 Explore the source changes: adeed84

🔍 Inspect the deploy log: https://app.netlify.com/sites/infallible-lumiere-e9d357/deploys/60da88baa8c51500079e8973

😎 Browse the preview: https://deploy-preview-1201--infallible-lumiere-e9d357.netlify.app/

@JeremyRudman
Copy link
Contributor

JeremyRudman commented Jun 25, 2021

When you take the keyboard out on android for the footer follows the keyboard up and overlaps the images on smaller screens (photo from base S21) for large screen phones it stays below the images and is fine. The brick hack logo also gets cut off the page on smaller screen phones
image

@JeremyRudman
Copy link
Contributor

On IOS the logos have no margin and are all right next to each other and the windows 95 text box.
chrome_2021-06-24_21-44-08

@JeremyRudman
Copy link
Contributor

Just a thought and you can ignore it but have the maximize button cause the browser to go into/out of full screen mode.

Copy link

@skyegallup skyegallup left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I really like the site! There's a few code style issues that should get fixed, however.

Also, I can confirm that the clipping issue @JeremyRudman pointed out above also show up on my mobile device (Galaxy S8, Firefox 89) when I open my keyboard to add my email. The button spacing works fine on my device, though.

clippy

@skyegallup
Copy link

On IOS the logos have no margin and are all right next to each other and the windows 95 text box.
chrome_2021-06-24_21-44-08

I think this is because we're using the gap CSS attribute to specify the space between each button. Safari doesn't support this for multi-column layouts yet, so this might be an issue for a while. Can we add horizontal margin to the center image instead? It's universally compatible and works in all cases, since the buttons shouldn't overflow to multiple rows on any realistic screen size.

peterkos added 7 commits June 25, 2021 00:52
Notable is the height constraint added tot he media query, and width->max-width to allow for better mobile resizing
Normally i'd use a % but window glitches when too small and imo cutoff is better
@peterkos peterkos marked this pull request as ready for review June 29, 2021 01:03
@skyegallup skyegallup self-requested a review June 29, 2021 02:30
Copy link

@skyegallup skyegallup left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

a few nitpicks, but mostly good

index.html Outdated
</div>
</section>
<footer>
<p>Interested in sponsoring? <span class="text-break">E-mail <a href="mailto:[email protected]">[email protected]</a></span></p>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be using a brickhack.io email instead of a coderit.org one, if at all possible.

Co-authored-by: Claire Gallup <[email protected]>
peterkos and others added 5 commits June 28, 2021 22:39
Co-authored-by: Claire Gallup <[email protected]>
Co-authored-by: Claire Gallup <[email protected]>
Co-authored-by: Claire Gallup <[email protected]>
Co-authored-by: Claire Gallup <[email protected]>
@peterkos peterkos requested a review from skyegallup June 29, 2021 02:43
@cbaudouinjr cbaudouinjr merged commit 3581fc2 into develop Jun 29, 2021
@cbaudouinjr cbaudouinjr deleted the bh-1200 branch June 29, 2021 02:45
@peterkos peterkos mentioned this pull request Jul 1, 2021
3 tasks
peterkos pushed a commit that referenced this pull request Aug 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BH8 Ohhhh yea!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants