Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stampy redesign 591 #602

Merged
merged 8 commits into from
May 1, 2024
Merged

Stampy redesign 591 #602

merged 8 commits into from
May 1, 2024

Conversation

buddy-web3
Copy link
Member

@@ -248,7 +249,7 @@ export const Chatbot = ({question, questions}: {question?: string; questions?: s
})

return (
<div className="centered col-9 padding-bottom-128">
<div className="centered col-9 height-70">
Copy link
Contributor

Choose a reason for hiding this comment

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

I believe this should take up 10 columns
image

Copy link
Contributor

Choose a reason for hiding this comment

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

If you want to leave this to me when I do css that's totally fine

@@ -20,3 +20,29 @@
width: 100%;
}
}
.height-70 {
Copy link
Contributor

Choose a reason for hiding this comment

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

oh interesting. For now on we could do .height-70vh so we know the units!

width: 11vw;
}
.red {
color: #d40000;
Copy link
Contributor

Choose a reason for hiding this comment

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

We'll want to define this and reference it as a variable, plus put this class in the same place as the other color classes in root.css

primary={true}
link="https://chat.aisafety.info"
icon={<BotIcon />}
text="Stampy chatbot"
Copy link
Contributor

Choose a reason for hiding this comment

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

This is the global nav right? Could we make it say "AI Safety chatbot" instead? :)

@buddy-web3
Copy link
Member Author

@melissasamworth "If you want to leave this to me when I do css that's totally fine" - yes please, take over the design parts. Please note that all of those are unrelated to the tickets.. they are some design changes that I had stashed and now somehow got pushed.

@melissasamworth melissasamworth merged commit 5f3ee7c into stampy-redesign May 1, 2024
1 check passed
@melissasamworth melissasamworth deleted the stampy-redesign-591 branch May 1, 2024 18:13
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