-
Notifications
You must be signed in to change notification settings - Fork 320
UI style: restyle theme with new fonts and color palette #1028
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,36 +1,31 @@ | ||
| /* This Source Code Form is subject to the terms of the Mozilla Public | ||
| * License, v. 2.0. If a copy of the MPL was not distributed with this | ||
| * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ | ||
| - License, v. 2.0. If a copy of the MPL was not distributed with this | ||
| - file, You can obtain one at http://mozilla.org/MPL/2.0/. */ | ||
|
|
||
| import { hasTransformer } from '@/files/transformers' | ||
| import { useShowSideview } from '@/content-view/context' | ||
| import { getAttachments } from '@/lib/attachments' | ||
| import type { ThunderboltUIMessage } from '@/types' | ||
| import { buildDocumentSideviewId } from '@/types/citation' | ||
| import { getAttachments } from '@/chats/message-utils' | ||
| import { useShowSideview } from '@/sideview/sideview-store' | ||
| import type { ThunderboltUIMessage } from '@/chats/types' | ||
| import { buildDocumentSideviewId } from '@/sideview/sideview-id-builder' | ||
| import type { UIMessage } from 'ai' | ||
| import { FileCard } from './file-card' | ||
| import { MemoizedMarkdown } from './memoized-markdown' | ||
|
|
||
| /** Re-deliver a single attachment as text/images and re-run the turn. */ | ||
| export type ResendAttachmentHandler = (localFileId: string, target: 'text' | 'images') => void | ||
| import { FileCard } from './file-card' | ||
| import type { ResendAttachmentHandler } from './file-card' | ||
|
|
||
| type MessageBubblesProps = { | ||
| message: UIMessage | ||
| onResendAttachment?: ResendAttachmentHandler | ||
| } | ||
|
|
||
| <<<<<<< HEAD | ||
| export const MessageBubbles = ({ message, onResendAttachment }: MessageBubblesProps) => { | ||
| const showSideview = useShowSideview() | ||
| const attachments = getAttachments(message as ThunderboltUIMessage) | ||
|
|
||
| return ( | ||
| <> | ||
| {attachments.length > 0 && ( | ||
| <div className="ml-auto mt-6 flex max-w-3/4 flex-wrap justify-end gap-2"> | ||
| {attachments.map((attachment) => { | ||
| // Alternative delivery modes to offer, but only on the latest turn and | ||
| // only once a non-native mode is in effect (i.e. remediation already | ||
| // converted this file) — so a clean native send shows no resend noise. | ||
| const resendTargets = | ||
| onResendAttachment && attachment.deliverAs | ||
| ? (['text', 'images'] as const).filter( | ||
|
|
@@ -60,12 +55,21 @@ | |
| /> | ||
| ) | ||
| })} | ||
| ======= | ||
| export const MessageBubbles = ({ message }: MessageBubblesProps) => | ||
| message.parts | ||
| .filter((part) => part.type === 'text') | ||
| .map((part, j) => ( | ||
| <div key={j} className="px-4 rounded-2xl max-w-3/4 bg-accent dark:bg-secondary/60 ml-auto mt-6 text-[14px]"> | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. User bubble ignores responsive fontMedium Severity User message bubbles use a fixed Reviewed by Cursor Bugbot for commit fae3d2c. Configure here. |
||
| <div className="space-y-2"> | ||
| <MemoizedMarkdown id={`${message.id}_${j}`} content={part.text || ''} /> | ||
| >>>>>>> 53c39843 (style: eliminate pure white/black and swap heading font to EB Garamond) | ||
| </div> | ||
| )} | ||
| {message.parts | ||
| .filter((part) => part.type === 'text') | ||
| .map((part, j) => ( | ||
| <div key={j} className="px-4 rounded-2xl max-w-3/4 bg-muted dark:bg-secondary/60 ml-auto mt-6"> | ||
| <div key={j} className="px-4 rounded-2xl max-w-3/4 bg-accent dark:bg-secondary/60 ml-auto mt-6 text-[14px]"> | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Unmerged conflict breaks MessageBubblesHigh Severity
Reviewed by Cursor Bugbot for commit fd56588. Configure here. |
||
| <div className="space-y-2"> | ||
| <MemoizedMarkdown id={`${message.id}_${j}`} content={part.text || ''} /> | ||
| </div> | ||
|
|
||


There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unmerged conflict markers remain
High Severity
The chat prompt
classNameprop still contains Git conflict markers (<<<<<<<,=======,>>>>>>>) with two competing values. That invalid JSX prevents the file from parsing and blocks the app from building until the merge is resolved.Reviewed by Cursor Bugbot for commit fd56588. Configure here.