Skip to content

feat: redo the ui - #35

Open
mihai-sysbio wants to merge 2 commits into
mainfrom
feat/ui-redo
Open

feat: redo the ui#35
mihai-sysbio wants to merge 2 commits into
mainfrom
feat/ui-redo

Conversation

@mihai-sysbio

Copy link
Copy Markdown
Contributor

This PR is a vibe-coded UI redo of the website. Albeit not fantastic code-wise, it does appear more well design than the current website. I'd be tempted to merge this in while we are waiting for a more proper redesign.

image

To view this and click around, just run bundle exec jekyll serve --livereload.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR refreshes the homepage layout and introduces a substantial set of site-wide style overrides to modernize the UI while continuing to use the ELIXIR Toolkit Theme (remote_theme ELIXIR-Belgium/elixir-toolkit-theme@5.0.0).

Changes:

  • Reworked index.md into a structured “hero + benefits” homepage with improved headings and more descriptive image alt text.
  • Added a large UX/style layer in _sass/custom_classes.scss, including updated brand tokens, navigation/footer styling, and homepage-specific components.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
index.md Replaces the homepage markup with a new hero/benefits structure and updated CTAs.
_sass/custom_classes.scss Adds global theme overrides and new homepage component styles (hero, benefit cards), plus navigation/footer refresh.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread _sass/custom_classes.scss
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In review

Development

Successfully merging this pull request may close these issues.

2 participants