Conversation
There was a problem hiding this comment.
Pull request overview
Modernizes the portfolio site by removing the legacy jekyll-agency theme/Bootstrap stack and replacing it with a custom Jekyll structure (layouts/includes + data-driven sections), updated styling, and simplified content pages suitable for GitHub Pages deployment.
Changes:
- Replaces legacy Bootstrap/theme assets and includes with a custom
site.scss+site.jsand new header/footer/homepage section includes. - Moves homepage content into
_data/*(home/projects/expertise/experience/contact/navigation) and updates layouts accordingly. - Updates legal/content pages (privacy, app privacy, imprint, 404), repo docs/templates, and GitHub Pages workflow.
Reviewed changes
Copilot reviewed 65 out of 94 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| screenshot.PNG | Repository screenshot asset (binary). |
| privacy.md | Rewrites Privacy Policy content and adds description front matter. |
| jekyll-agency.gemspec | Removes the legacy theme gemspec (theme no longer gem-based). |
| imprint.md | Updates imprint formatting/content and adds description front matter. |
| assets/js/site.js | Adds lightweight header scroll + mobile nav toggle behavior. |
| assets/js/jquery.easing.min.js | Removes legacy jQuery easing plugin. |
| assets/js/jqBootstrapValidation.js | Removes legacy contact form validation plugin. |
| assets/js/contact_me.js | Removes legacy Formspree/jQuery contact form script. |
| assets/js/bootstrap.bundle.min.js | Removes bundled Bootstrap JS (no longer used). |
| assets/js/agency.min.js | Removes legacy theme JS (scroll/spy/shrink nav). |
| assets/img/timeline/4.jpg | Legacy theme asset (binary). |
| assets/img/timeline/3.jpg | Legacy theme asset (binary). |
| assets/img/timeline/2.jpg | Legacy theme asset (binary). |
| assets/img/timeline/1.jpg | Legacy theme asset (binary). |
| assets/img/team/500x500.jpg | Legacy theme asset (binary). |
| assets/img/team/2.jpg | Legacy theme asset (binary). |
| assets/img/portfolio/twistreader_logo_256px.png | Portfolio asset (binary). |
| assets/img/portfolio/timetraversehub_logo_256px.png | Portfolio asset (binary). |
| assets/img/portfolio/symlinkhelper_logo_256px.png | Portfolio asset (binary). |
| assets/img/portfolio/fishingknots_logo_256px.png | Portfolio asset (binary). |
| assets/img/header.png | Site image asset (binary). |
| assets/img/contact.png | Site image asset (binary). |
| assets/img/company/logo.png | Company/logo asset (binary). |
| assets/img/company/header_bg_new.jpg | Company/header background asset (binary). |
| assets/img/company/happy-people-small.jpg | Company image asset (binary). |
| assets/img/clients/themeforest.jpg | Legacy theme asset (binary). |
| assets/img/clients/envato.jpg | Legacy theme asset (binary). |
| assets/img/clients/designmodo.jpg | Legacy theme asset (binary). |
| assets/img/clients/creative-market.jpg | Legacy theme asset (binary). |
| assets/css/site.scss | Adds new custom design system and responsive layout styles. |
| assets/css/agency.scss | Removes legacy theme SCSS entrypoint. |
| appprivacy.md | Rewrites app privacy baseline and adds description front matter. |
| _sass/layout/_timeline.scss | Removes legacy timeline SCSS partial. |
| _sass/layout/_team.scss | Removes legacy team SCSS partial. |
| _sass/layout/_services.scss | Removes legacy services SCSS partial. |
| _sass/layout/_portfolio.scss | Removes legacy portfolio SCSS partial. |
| _sass/layout/_masthead.scss | Removes legacy masthead SCSS partial. |
| _sass/layout/_footer.scss | Removes legacy footer SCSS partial. |
| _sass/layout/_contact.scss | Removes legacy contact SCSS partial. |
| _sass/components/client-scroll.scss | Removes legacy client scroll SCSS partial. |
| _sass/components/_navbar.scss | Removes legacy navbar SCSS partial. |
| _sass/components/_buttons.scss | Removes legacy buttons SCSS partial. |
| _sass/base/_variables.scss | Removes legacy bootstrap variable overrides. |
| _sass/base/_page.scss | Removes legacy global page styling. |
| _sass/base/_mixins.scss | Removes legacy font mixins. |
| _portfolio/macos_project_2.md | Removes legacy portfolio collection item (migrated to _data/projects.yml). |
| _portfolio/macos_project_1.md | Removes legacy portfolio collection item (migrated to _data/projects.yml). |
| _portfolio/ios_project_2.md | Removes legacy portfolio collection item (migrated to _data/projects.yml). |
| _portfolio/ios_project_1.md | Removes legacy portfolio collection item (migrated to _data/projects.yml). |
| _layouts/page.html | Redesigns page layout structure (title/description + prose wrapper). |
| _layouts/home.html | Replaces old includes with new section-based home structure. |
| _layouts/default.html | Switches to new site header/footer and loads site.js only. |
| _includes/timeline.html | Removes legacy timeline include. |
| _includes/team.html | Removes legacy team include. |
| _includes/site-header.html | Adds new sticky header + navigation markup. |
| _includes/site-footer.html | Adds new footer with contact/social links + legal links. |
| _includes/services.html | Removes legacy services include. |
| _includes/projects.html | Adds new projects section driven by _data/projects.yml. |
| _includes/portfolio_grid.html | Removes legacy portfolio grid include (modals-based). |
| _includes/navheader.html | Removes legacy nav + masthead include. |
| _includes/nav.html | Removes legacy nav include used on non-home pages. |
| _includes/modals.html | Removes legacy portfolio modal include. |
| _includes/hero.html | Adds new hero section driven by _data/home.yml. |
| _includes/head.html | Simplifies head, adds fonts, loads site.css, adds {% seo %}. |
| _includes/footer.html | Removes legacy footer include and page-short JS. |
| _includes/expertise.html | Adds expertise section driven by _data/expertise.yml. |
| _includes/experience.html | Adds experience section driven by _data/experience.yml. |
| _includes/contact.html | Removes legacy contact form section include. |
| _includes/clients.html | Removes legacy clients include. |
| _includes/about.html | Replaces legacy about include with new data-driven content. |
| _data/style.yml | Removes legacy theme styling configuration data. |
| _data/sitetext.yml | Removes legacy localization-driven site text data. |
| _data/projects.yml | Adds new project list data model for homepage projects section. |
| _data/navigation.yml | Simplifies navigation to new section anchors. |
| _data/home.yml | Adds home section copy (hero/projects/expertise/experience/about). |
| _data/expertise.yml | Adds expertise items for the expertise section. |
| _data/experience.yml | Adds experience entries for the experience section. |
| _data/contact.yml | Adds contact/social links used by the footer. |
| _config.yml | Reconfigures site settings for custom Jekyll build + SEO/sitemap plugins. |
| README.md | Replaces theme README with repo-specific development and structure notes. |
| Gemfile.lock | Adds/commits dependency lockfile for reproducible builds. |
| Gemfile | Removes gemspec usage and declares explicit Jekyll/plugin dependencies. |
| 404.html | Simplifies 404 page and adds description + explicit permalink. |
| .gitignore | Stops ignoring Gemfile.lock so it can be committed. |
| .github/workflows/jekyll.yml | Updates Pages workflow to build on PRs and deploy only on pushes. |
| .github/PULL_REQUEST_TEMPLATE.md | Simplifies PR template to repo-relevant sections. |
| .github/ISSUE_TEMPLATE/support.md | Removes legacy support issue template. |
| .github/ISSUE_TEMPLATE/feature_request.md | Updates feature request template to portfolio-site context. |
| .github/ISSUE_TEMPLATE/bug_report.md | Updates bug report template to portfolio-site context. |
| .github/FUNDING.yml | Removes legacy funding configuration. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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
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.
closes #1
closes #2
closes #4
closes #5
closes #3