Repository navigation
feat: add I9kAskAi and an extra row in I9kFooter - #45
Merged
Merged
Conversation
I9kAskAi is a heading and pill links that open ChatGPT, Perplexity and Claude in a new tab with a question about the site already typed in. I9kFooter gains an `extra` slot, a row of its own between the columns and the bottom bar, where the sites put it. The ChatGPT, Perplexity and Claude logos join the icon set (simple-icons, CC0; OpenAI's from simple-icons 11, before it was withdrawn). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.
What
I9kAskAi: a heading plus three pill links (ChatGPT, Perplexity, Claude). Each opens the assistant in a new tab with a question already typed in, through itsqparameter:chatgpt.com/?q=perplexity.ai/search?q=claude.ai/new?q=prompt,heading,headingTag(defaulth2),providersandalign.align="center"is for the footer's centred layout.rel="noopener noreferrer", so the assistant gets the question but never the page's address.askAiUrl()andI9K_ASK_AI_PROVIDERSare exported.I9kFootergets anextraslot: a row of its own under a divider, between the columns and the bottom bar (after the tagline in the centred layout).The three sites use it in their footers, each with its own question that also names the other two:
Merge
Merge with a merge commit, not a squash. 9k.school locks
09d4b88in its lockfile. ismail9k.com and the9klabs.com pin it inpackage.json. The commit has to stay inmain's history.This branch starts from
cec6be3, not from currentmain. Currentmainalso has2a67c62("skip I9kModal showModal when the dialog left the page"), which fails 12 of 9k.school's admin dialog tests: they mount dialogs that aren't attached to the document. Basing the branch oncec6be3keeps that change out of these PRs. It still needs sorting out before 9k.school next updates the design system.Checks
npm run checkpasses: 1,132 tests, format, lint, types, library, Storybook and showcase builds.extraslot in both layouts./linksfooter.🤖 Generated with Claude Code