Skip to content

Conversation

@GBKS
Copy link
Contributor

@GBKS GBKS commented Sep 19, 2022

Without this promo there's no way to discover the event from the website. I put this together fairly quickly for the announcement today, feedback appreciated.

Preview

@GBKS GBKS added Copy Task is about improving text. Design Task is about designing something. labels Sep 19, 2022
@GBKS GBKS self-assigned this Sep 19, 2022
@netlify
Copy link

netlify bot commented Sep 19, 2022

Deploy Preview for bitcoin-design-site ready!

Name Link
🔨 Latest commit 7c539f7
🔍 Latest deploy log https://app.netlify.com/sites/bitcoin-design-site/deploys/63298c992ac976000994518d
😎 Deploy Preview https://deploy-preview-924--bitcoin-design-site.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 settings.

sbddesign
sbddesign previously approved these changes Sep 20, 2022
Copy link
Collaborator

@sbddesign sbddesign left a comment

Choose a reason for hiding this comment

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

Looks nice! Left a suggestion to make the paragraph perkier with exclamation marks, but don't consider that a blocking change.

Also would be nice:

  • if the event was higher above the fold (but this is time sensitive and that would take a little work to make that look right)
  • if the image could be a link (but requires venturing out from the traditional jekyll includes we use)

So having said all that, I think we should get it on the site and start promoting it. Can tweak it later.

Bosch-0
Bosch-0 previously approved these changes Sep 20, 2022
@Bosch-0
Copy link
Collaborator

Bosch-0 commented Sep 20, 2022

We could add the persistent banner stickied to the top of the page. I know you didn't like this approach Christoph but It looks fine and brings it ATF without having to move too many things around moving too many parts.

Co-authored-by: Stephen DeLorme <stephen@stephendelorme.com>
@GBKS GBKS dismissed stale reviews from Bosch-0 and sbddesign via 27124f9 September 20, 2022 09:40
Using HTML instead of our usual include, because otherwise it won't render the link for some reason.
@GBKS
Copy link
Contributor Author

GBKS commented Sep 20, 2022

Thanks for the feedback. I just accepted the copy change and updated the image to be a link (using HTML instead of markdown - I think it's OK because this is a temporary short-term thing).

For a persistent banner, please also consider that we have a sticky nav bar, and an expanding menu on mobile. If that banner is also sticks and sits above the nav bar, the code for the menu needs to be updated to make that positioning work, and for the expanding menu to account for the height difference. It takes some CSS and JS fiddling to get this right.

Why not just take over the top banner altogether?
image

Only downside is that it wouldn't be a direct link, people would have to scroll down a bit to get to that. Not a huge issue though.

Copy link
Contributor

@danielnordh danielnordh left a comment

Choose a reason for hiding this comment

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

LGTM

@GBKS GBKS merged commit 84c9a27 into master Sep 20, 2022
@GBKS
Copy link
Contributor Author

GBKS commented Sep 20, 2022

Merged this one, as the bolt.fun announcement will happen in the next hour (fingers crossed) and there are some links floating around that lead to our home page, not the event page.

@GBKS GBKS deleted the feature/designathon-promo branch December 20, 2022 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Copy Task is about improving text. Design Task is about designing something.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants