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

Gen AI lab: styling pass #58004

Merged
merged 6 commits into from Apr 16, 2024
Merged

Gen AI lab: styling pass #58004

merged 6 commits into from Apr 16, 2024

Conversation

bencodeorg
Copy link
Contributor

@bencodeorg bencodeorg commented Apr 12, 2024

Warning!!

The AP CSP Create Performance Task is in progress. The most critical dates are from April 3 - April 30, 2024. Please consider any risk introduced by this PR that could affect our students taking AP CSP. Code.org students taking AP CSP primarily use App Lab for their Create Task, however a small percent use Game Lab. Carefully consider whether your change has any risk of alterering, changing, or breaking anything in these two labs. Even small changes, such as a different button color, are considered significant during this time period. Reach out to the Student Learning team or Curriculum team for more details.

Description

Friday afternoon timeboxed glow up 💅 for gen AI:

  • restyled copy button
  • blue border on textarea and input, standardize sizes
  • styling of added retrievals, with red x on hover to remove
  • smaller toggle button to get into presentation view
  • set max temperature to 1 with input from Dan and Tyrone
Update "After" screenshot
Updated color/size on inputs image
Updated copy chat button image
Smaller toggle image
Updated retrieval tab image

@bencodeorg bencodeorg changed the title Ben/gen ai UI cleanup Gen AI lab: styling pass Apr 12, 2024
@bencodeorg bencodeorg requested a review from a team April 15, 2024 15:49
@@ -25,22 +25,6 @@ import {AichatLevelProperties, ViewMode} from '@cdo/apps/aichat/types';
import {isProjectTemplateLevel} from '@cdo/apps/lab2/lab2Redux';
import ProjectTemplateWorkspaceIcon from '@cdo/apps/templates/ProjectTemplateWorkspaceIcon';

const renderChatWorkspaceHeaderRight = (onClear: () => void) => {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Tired of seeing helpers at the top of React components, so moved to bottom :)

Copy link
Contributor

@sanchitmalhotra126 sanchitmalhotra126 left a comment

Choose a reason for hiding this comment

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

✨ 💅 🧹

Copy link
Contributor

@fisher-alice fisher-alice left a comment

Choose a reason for hiding this comment

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

Awesome! Left a question about input box border which I believe Mark responded to already. Thanks!

}

input {
@extend %full-width;
height: 30px;
border-radius: 4px;
border-color: $lighter_gray;
Copy link
Contributor

Choose a reason for hiding this comment

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

Posted a question about this to @markabarrett on Slack.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Didn't fully implement the design system for inputs and textareas, but at least switched the outline to match the preferred color ($light_black)

@bencodeorg bencodeorg merged commit 1ad9b87 into staging Apr 16, 2024
2 checks passed
@bencodeorg bencodeorg deleted the ben/gen-ai-ui-cleanup branch April 16, 2024 17:30
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.

None yet

3 participants