ProjectPythia / projectpythia.github.io

https://projectpythia.org
Apache License 2.0
32 stars 18 forks source link

add link to event page from portal #328

Closed jukent closed 1 year ago

jukent commented 1 year ago

This PR adds a link to the event on the main portal page. Perhaps we want to link this differently, but this was the easiest solution I found.

Eventually we may want an "events" tab in the nav bar that lists past events, events we'll be present at or give talks/tutorials, and events that we host.

github-actions[bot] commented 1 year ago

👋 Thanks for opening this PR! The Cookbook will be automatically built with GitHub Actions. To see the status of your deployment, click below. 🔍 Git commit SHA: 0ffbeacc829a9a0c2476e2adf1ac21e5ee1e8a09 ✅ Deployment Preview URL: https://ProjectPythia.github.io/projectpythia.github.io/_preview/328

ktyle commented 1 year ago

@jukent looks good but wonder if we can make it stand out more? Maybe as simple as a bigger and/or bolder and/or different-colored font?

dcamron commented 1 year ago

@jukent looks good but wonder if we can make it stand out more? Maybe as simple as a bigger and/or bolder and/or different-colored font?

I've been playing around with sphinx-design (a JupyterBook dependency) and it's easy to create snazzy buttons. We already have a few buttons hanging around anyway (and can always just borrow the html from those), who doesn't love a good button?

jukent commented 1 year ago

Great idea @dcamron !

brian-rose commented 1 year ago

For some reason I can't see the preview for this PR. Following the preview link keeps redirecting to the home page at https://projectpythia.org.

ktyle commented 1 year ago

@brian-rose Yes the same for me. I had to clone this fork and build the site locally.

jukent commented 1 year ago

Any idea why that might be?

ktyle commented 1 year ago

@jukent I don't ... we should open a separate issue about this. But in the interest of getting this merged, the only thing I would suggest is to maybe have a different background color other than white for the Cookoff button? It would make it stand out just a bit more.

jukent commented 1 year ago

I can do that, just don't know what color is best. I'll work on it though (I changed it to green locally just to test that I could do it as I expected).

dcamron commented 1 year ago

on the cookoff site, I went with .secondary, which should be the navbar Pythia Grey. I haven't previewed this, so I'm not sure how that might look

jukent commented 1 year ago

Screen Shot 2023-04-13 at 10 57 52 AM

I changed the button color. It stands out a bit better, but still doesn't look the best. Is it obvious that it is clickable? The curser does change when you hover over it.

brian-rose commented 1 year ago

I think the screenshot looks good.

brian-rose commented 1 year ago

If I have a moment today, I'll try opening a bare bones PR on this repo to see if the preview is still failing.

jukent commented 1 year ago

Okay @brian-rose just merge when you're ready

brian-rose commented 1 year ago

I merged, but the changes didn't get published on the portal site.

There's something wrong with the actions that publish to GitHub Pages, which is affecting both the preview and the actual publication. The gh-pages branch of the repo hasn't seen any pushes in three months.

brian-rose commented 1 year ago

The new link is finally visible now at https://projectpythia.org!