Repository navigation
Show the BIQ company picker as logo tiles - #75
Conversation
There was a problem hiding this comment.
🟢 Approval recommended
The picker behavior, accessibility model, compatibility path, assets, documentation, and tests are consistent and complete.
0 open findings
What changed in this PR
Replaces the company dropdown with an accessible, responsive logo-tile picker while preserving URL and analytics behavior.
Changes:
- Adds keyboard-accessible company tiles and responsive styling.
- Bundles seven attributed company logos with a name-only fallback.
- Updates copy, configuration documentation, and framing tests.
| File | Description |
|---|---|
js/biq.js |
Implements tile rendering, selection, keyboard controls, and analytics. |
css/biq.css |
Styles the responsive tile grid and selected state. |
index.html |
Replaces the dropdown with a radio group. |
data/ui.json |
Updates picker copy and universal label. |
tests/test_framing.py |
Verifies copy, markup, behavior hooks, and assets. |
README.md |
Documents tile behavior and logo configuration. |
css/logos/README.md |
Records logo sources and licensing. |
css/logos/amazon.svg |
Adds the Amazon logo. |
css/logos/arm.svg |
Adds the Arm logo. |
css/logos/blue-origin.svg |
Adds the Blue Origin logo. |
css/logos/coupang.svg |
Adds the Coupang logo. |
css/logos/delivery-hero.svg |
Adds the Delivery Hero logo. |
css/logos/gitlab.svg |
Adds the GitLab logo. |
css/logos/toyota.svg |
Adds the Toyota logo. |
🧠 Review effort: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
@codex review |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
@copilot review |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c3116e81b1
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
|
Codex Review: Didn't find any major issues. 🎉 Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
|
@codex review |
|
Codex Review: Didn't find any major issues. Breezy! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Replaces the BIQ company dropdown with the same always-visible logo tiles Porridge got in kindel/porridge#71.
css/logos/dawn.png), used with permission of Dawn Aerospace (Tig Kindel, owner). Porridge gets the same logo in Add the Dawn Aerospace logo to the company tiles porridge#72.biq_companysends the same payload the dropdown did (empty for the universal set), andkld_companyfires withsource: "picker".?c=behavior is unchanged.data/ui.jsondropscompanyPlaceholderand addsuniversalLabel.css/logos/with the same licensing README. Sharing them through app-kit would mean a new mount in every host for seven small files, so a copy is simpler. On kindel.com both modules mountcsstostatic/css, so the files land at the same path either way.window.BIQ.logoBaseandlogosare documented in the README. If a host shell still has the old<select>, biq.js swaps it for the tile group, so merge order with kindelwww does not matter.