Update site content: Our Work, Research, Jobs, About; remove Blog - #91
Open
Caralie-fs wants to merge 3 commits into
Open
Update site content: Our Work, Research, Jobs, About; remove Blog#91Caralie-fs wants to merge 3 commits into
Caralie-fs wants to merge 3 commits into
Conversation
Contributor
|
Thanks for putting this up! A few feedback notes to feed claudio:
For (2), these both look a bit weirdly off center to me: 
|
Contributor
Author
|
@carlaKC ready for your eyes again! |
carlaKC
reviewed
Aug 11, 2026
carlaKC
left a comment
carlaKC
left a comment
Contributor
There was a problem hiding this comment.
A few review notes from claude about cleanups we need:
CSS breakage from the restructure
- projects.html:7 / research.html:7 — adding <div class="page-intro"> as the first child of .projects-container means the .projects-row:first-child rules (_sass/main.scss:133 plus several mobile overrides) no longer match anything. The first row on both pages loses its white top rule and padding on desktop, and the mobile first-row overrides stop applying. Switching to :first-of-type or an explicit class fixes it.
- about.html:30 — deleting the street-address block left id="first-row" on the email row, so the address-specific styling (main.scss:328: no top border, heavy white bottom border, 2rem padding) now mis-renders the contact list with a double divider. Remove the id or the rule.
- projects.html:24 — the .section-lead inside .page-intro never gets its margin-bottom: 0 because the nested .page-intro p rule wins on specificity, so lead-paragraph spacing is inconsistent with the same class used elsewhere on the page.
Blog removal fallout
- _config.yml:33 — jekyll-feed is still enabled with zero posts, so /feed.xml publishes as a permanently empty feed for existing RSS subscribers, and all old blog/post URLs now hard-404 with no redirects. Worth dropping the plugin and deciding whether the old URLs deserve redirects.
- _sass/main.scss — roughly 250 lines of now-dead CSS (.blog-*, .post-*, .author-*, jobs listing classes) plus orphaned _data/authors.yml (which still contains a placeholder test bio), _data/jobs.yml, and a dozen blog-era images. Cleaner to delete these in the same change.
| <h3 class="item-title">BITCOIN CORE</h3> | ||
| <h2 class="item-description"> | ||
| Chaincode engineers develop, review, test and maintain the | ||
| <a href="https://bitcoin.org" target="_blank" rel="noopener">reference Bitcoin implementation</a>. |
Contributor
There was a problem hiding this comment.
Let's link to GH instead: https://github.com/bitcoin/bitcoin
Comment on lines
+61
to
+62
| <div class="publications"> | ||
| <p class="section-lead">Our team has worked on:</p> |
Contributor
There was a problem hiding this comment.
I think this would look a bit better with the main paragraph, rather than underneath this list? What do you think?
| <a href="https://papers.ssrn.com/sol3/papers.cfm?abstract_id=3840374" target="_blank" rel="noopener">Lightning Network Economics: Channels</a> | ||
| </li> | ||
| <li> | ||
| <a href="https://arxiv.org/abs/1905.10518" target="_blank" rel="noopener">Bandwidth-Efficient Transaction Relay for Bitcoin</a> |
Contributor
There was a problem hiding this comment.
Please can we add [Nested MuSig2](https://eprint.iacr.org/2026/223), fresh of the press!
Contributor
|
Formatting here still looks slightly off to me, but if you think it looks fine then happy to just leave it! 
|
carlaKC
reviewed
Aug 11, 2026
| </div> | ||
| </div> | ||
| <div class="contact-row" id="first-row"> | ||
| <div class="about-icon"> |
Contributor
There was a problem hiding this comment.
If we remove <div class="contact-row" id="first-row"> we won't have the thicker line randomly below our email address
Contributor
|
Small patch to move some photos around: photo-swap.patch (can just ask claude to apply it!) |
The jekyll/builder:latest image now ships Ruby 3.4, which no longer bundles several stdlib gems that Jekyll 4.1 requires. Make the build reproducible and green again: - Add csv, base64, logger, bigdecimal and ostruct to the Gemfile. - Run jekyll via bundle exec in the workflow so the bundled gem versions are actually used. - Use the latest jekyll/builder image and bump the lockfile Bundler version to 2.6.9.
Replace the data-driven Projects/Research pages with hand-written copy, refresh the Jobs and About pages, and remove the blog: - Our Work: new intro, and sections for Bitcoin Core (linked to the GitHub repository), BIPs, Learning, Events, plus past projects (Optech, Podcast, Residency). - Research: new intro followed by a publications list (including the new Nested MuSig2 paper), then Research Day, Research Prize and Bitcoin Scholarship sections. - Jobs: new self-directed-work copy with a skyline photo. - About: drop the street-address row (and the first-row styling that would otherwise mis-render the email row), keep the office photo. - Remove the blog: posts, post layout, blog page and nav entry. - Styling: page-intro/section-lead/publications styles; first project row styling moved to an explicit projects-row-first class since page-intro is now the container's first child; consistent Bitcoin capitalization.
With the blog gone, drop everything that only existed to support it: - Remove the minima theme and jekyll-feed. All pages use the repo's own layouts and Sass, and dropping the theme is what actually stops an empty /feed.xml from being generated (Jekyll auto-loads theme runtime dependencies, so removing the plugin alone is not enough). - Delete ~250 lines of dead CSS: blog, post, author and job-listing rules across all media queries. - Delete orphaned _data/authors.yml and _data/jobs.yml. - Delete images that are no longer referenced anywhere: blog-era photos and UI assets, social-media icons, unused logo variants, and an ex-employee photo.
carlaKC
force-pushed
the
content-updates
branch
from
August 14, 2026 13:57
1c88287 to
163f2a1
Compare
Contributor
|
@Caralie-fs - I've pushed some changes, screenshots below! I couldn't really find a solution for the text alignment so decided to just live with it. If you think this looks good I'll go ahead and merge! 
|
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Summary
blog.html, thepostlayout, and old residency/internship postsjobs-1.png→jobs-skyline.jpg) and adds supporting styles in_sass/main.scssTest plan
bundle exec jekyll servebuilds without errors