Skip to content

Added dark_mode to the website. - #1551

Closed
sparshsingh12312 wants to merge 3 commits into
JustVugg:devfrom
sparshsingh12312:dark_mode
Closed

sparshsingh12312 wants to merge 3 commits into
JustVugg:devfrom
sparshsingh12312:dark_mode

Conversation

@sparshsingh12312

@sparshsingh12312 sparshsingh12312 commented Sep 15, 2026

Copy link
Copy Markdown

Summary

This PR addresses the lack of dark mode configuration options on the landing page and refactors several core application layouts (Dashboard, Kanban, Timeline, UI Buttons) that used hardcoded backgrounds and colors, leading to visibility issues in dark mode.

Related Issue

#1550

Validation

-make -C c check
-CUDA changes were tested with make -C c cuda-test (if applicable)

  • Performance claims include hardware, commands, and repeatable measurements
  • Performance claims include a validated experiment manifest with raw evidence

Compatibility

-The default CPU build remains dependency-free

  • No model files, generated binaries, or benchmark artifacts are included

Proof

Screenshot 2026-09-15 at 10 25 52 PM

JustVugg and others added 3 commits September 15, 2026 14:39
…OTICE

The LICENSE file still carried the Apache 2.0 template placeholders
(Copyright [yyyy] [name of copyright owner]), so the notice was never
actually applied to this work. Fill it in, add the NOTICE file Apache 2.0
expects for the copyright line, and point the four READMEs at both.
license: name the copyright holder and add NOTICE
@sparshsingh12312
sparshsingh12312 marked this pull request as draft September 15, 2026 16:58
@sparshsingh12312

Copy link
Copy Markdown
Author

Could you please merge my PR as it is required for the betterment of your websites UI.

@sparshsingh12312
sparshsingh12312 marked this pull request as ready for review September 16, 2026 06:09
@JustVugg
JustVugg changed the base branch from main to dev September 17, 2026 07:06
@sparshsingh12312

Copy link
Copy Markdown
Author

Is my PR merged or not because I still cannot see. the changes on the website. @JustVugg

@JustVugg

Copy link
Copy Markdown
Owner

Thank you for this, and sorry it is being closed rather than merged: the reason is the file, not the work.

The landing page was rewritten in #1635 (merged): the models became a searchable list instead of a card grid, a brio mode section was added, and the version and family names are now generated from c/version.py and c/family_registry.py so they cannot go stale again. That rewrite touches almost every line of site/index.html, so there was no way to land your patch on top of it.

Your dark mode is the one that shipped. #1635 uses your mechanism, not a different one:

  • the same colibri_theme key in localStorage
  • the same data-theme attribute, set by an inline script before first paint so the theme never flashes
  • the same D keyboard shortcut, with the same guard that skips it while an input is focused
  • the same sun and moon icons crossfading with a rotate and scale, and the same 300 ms theme-transition class
  • the same rule that the operating system preference is followed only until the visitor chooses for themselves

The commit and the pull request body both say where it came from. If you want to keep going on the site, the theme is done but two things next to it are open: the page has no mobile layout yet, and the model list has no per-model page behind each row.

One separate note: the LICENSE, NOTICE and README part of this pull request is already on dev, so that half had nothing left to apply.

@JustVugg JustVugg closed this Sep 20, 2026
jtinbergen pushed a commit to jtinbergen/colibri that referenced this pull request Sep 20, 2026
The landing page still described eight families, listed DeepSeek and Qwen3 as
planned when both have been running for a while, and showed the models as a
three-column card grid whose cards were as tall as their longest description.
Brio mode was not mentioned at all.

What changed

- Models are a list, not a grid: one row per family with the description, the
  tags, total and active parameters, the container size, the engine file, and
  the command that fetches the weights with a copy button. A search box and
  five tag filters narrow it in place. Nine families, up from the eight the
  page claimed, with DeepSeek V4.1 Flash no longer nested inside the V4 card.
- Every figure comes from the family table in the README or, where the README
  is silent, from the model's own card upstream. The download commands are the
  ones in each model's doc, and each row links to the doc that exists.
- A brio mode section: what it is for with three worked examples, the measured
  comparison against generating the same answer, the request and the reply of
  POST /v1/brio, and the calibration figures that are the point of the mode.
- A light and dark theme. The mechanism is the one sparshsingh12312 proposed
  in JustVugg#1551: the same colibri_theme key, the same data-theme attribute set
  before first paint so the theme never flashes, the same D shortcut, and the
  operating system preference followed only until the visitor chooses.
- Figtree for the page, Bytesized still for the wordmark, and the favicon is
  now the mark in its own four colours instead of flat ink.

The page stays what it was: one file, no build step, no framework. The
filtering, the theme and the copy buttons are about sixty lines of plain
JavaScript at the bottom.

Note for JustVugg#1551: that pull request rewrites the same file, so one of the two
will have to be rebased on the other. Its dark-mode design is the one kept
here.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants