stratum-mining / stratumprotocol.org

Website for the Stratum V2 bitcoin mining protocol
https://stratumprotocol.org
MIT License
13 stars 20 forks source link

moved past supporters tab to be inline with others #212 #220

Closed omerskywalker closed 5 months ago

omerskywalker commented 5 months ago

Moved the Past Supporters tab to be inline with other tabs

Screen Shot 2024-05-05 at 2 31 46 PM

This will create an issue for the layout on mobile as the row will exceed the viewWidth -- this can be resolved by adding a media query for display widths under 440px, then changing the font-size to the text-links to 0.85rem -- will create a separate issue and PR for this.

netlify[bot] commented 5 months ago

Deploy Preview for startling-capybara-2231ea ready!

Name Link
Latest commit 7cfaea0efc3d2f0a2bac8d6c05d69aa9a271020c
Latest deploy log https://app.netlify.com/sites/startling-capybara-2231ea/deploys/6639a64f567cee000898383e
Deploy Preview https://deploy-preview-220--startling-capybara-2231ea.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

pavlenex commented 5 months ago

Hey @omerskywalker, tested and it looks good on me except on mobile. Any way we can make that one look better? Tested on Safari, iOS 13 IMG_5173

omerskywalker commented 5 months ago

Hi @pavlenex - I've reduced the font-size in mobile layout to allow tabs to fit inside of the screen width.

I also noticed a discrepancy in capitalization formatting between the tabs, and standardized them to capitalize multi-worded tabs.

Screen Shot 2024-05-06 at 10 57 12 PM

I included a small adjustment for increased padding in the config tabs on mobile view - to maximize the available real estate on the page without breaking the layout.

Screen Shot 2024-05-06 at 10 43 03 PM

Screen Shot 2024-05-06 at 10 43 14 PM

Let me know your thoughts on these.