Skip to content

refactor: update styles and prompt selection logic (#103) #124

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.

Sign up for GitHub

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

Merged
Junyi-99 merged 1 commit into from
Feb 18, 2026
Merged

refactor: update styles and prompt selection logic (#103) #124

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 4 additions & 3 deletions pkg/gen/api/chat/v2/chat.pb.go
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

12 changes: 11 additions & 1 deletion webapp/_webapp/src/components/message-entry-container/assistant.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -147,7 +147,17 @@ export const AssistantMessageContainer = ({
{((parsedMessage.regularContent?.length || 0) > 0 || parsedMessage.paperDebuggerContent.length > 0) && (
<div className="actions rnd-cancel noselect">
<Tooltip content="Copy" placement="bottom" size="sm" delay={1000}>
<span onClick={handleCopy} onKeyDown={(e) => { if (e.key === 'Enter' || e.key === ' ') { handleCopy(); } }} tabIndex={0} role="button" aria-label="Copy message">
<span
onClick={handleCopy}
onKeyDown={(e) => {
if (e.key === "Enter" || e.key === " ") {
Copy link

Copilot AI Feb 18, 2026

Choose a reason for hiding this comment

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

Missing e.preventDefault() call when handling Space key. When Space is pressed on an element with role="button", the browser will scroll the page by default. Add e.preventDefault() inside the if statement to prevent unwanted scrolling behavior.

Suggested change
if (e.key === "Enter" || e.key === " ") {
if (e.key === "Enter" || e.key === " ") {
e.preventDefault();

Copilot uses AI. Check for mistakes.
handleCopy();
}
}}
tabIndex={0}
role="button"
aria-label="Copy message"
>
<Icon icon={copySuccess ? "tabler:copy-check" : "tabler:copy"} className="icon" />
</span>
</Tooltip>
Expand Down
12 changes: 11 additions & 1 deletion webapp/_webapp/src/components/message-entry-container/tools/general.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,17 @@ export const GeneralToolCard = ({
// When there is a message, show the compact card with collapsible content
return (
<div className={cn("tool-card noselect compact", { animated: animated })}>
<div className="flex items-center gap-1 cursor-pointer" role="button" tabIndex={0} onClick={toggleCollapse} onKeyDown={(e) => { if (e.key === 'Enter' || e.key === ' ') { toggleCollapse(); } }}>
<div
className="flex items-center gap-1 cursor-pointer"
role="button"
tabIndex={0}
onClick={toggleCollapse}
onKeyDown={(e) => {
if (e.key === "Enter" || e.key === " ") {
Copy link

Copilot AI Feb 18, 2026

Choose a reason for hiding this comment

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

Missing e.preventDefault() call when handling Space key. When Space is pressed on an element with role="button", the browser will scroll the page by default. Add e.preventDefault() inside the if statement to prevent unwanted scrolling behavior.

Suggested change
if (e.key === "Enter" || e.key === " ") {
if (e.key === "Enter" || e.key === " ") {
if (e.key === " ") {
e.preventDefault();
}

Copilot uses AI. Check for mistakes.
toggleCollapse();
}
}}
>
<button
className="text-gray-400 hover:text-gray-600 transition-colors duration-200 rounded flex"
aria-label={isCollapsed ? "Expand" : "Collapse"}
Expand Down
6 changes: 5 additions & 1 deletion ...bapp/src/components/message-entry-container/tools/paper-score-comment/filter-controls.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,11 @@ export const FilterControls = ({
role="button"
tabIndex={0}
onClick={() => setIsSuggestionsExpanded(!isSuggestionsExpanded)}
onKeyDown={(e) => { if (e.key === 'Enter' || e.key === ' ') { setIsSuggestionsExpanded(!isSuggestionsExpanded); } }}
onKeyDown={(e) => {
if (e.key === "Enter" || e.key === " ") {
Copy link

Copilot AI Feb 18, 2026

Choose a reason for hiding this comment

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

Missing e.preventDefault() call when handling Space key. When Space is pressed on an element with role="button", the browser will scroll the page by default. Add e.preventDefault() inside the if statement to prevent unwanted scrolling behavior.

Suggested change
if (e.key === "Enter" || e.key === " ") {
if (e.key === "Enter" || e.key === " ") {
e.preventDefault();

Copilot uses AI. Check for mistakes.
setIsSuggestionsExpanded(!isSuggestionsExpanded);
}
}}
>
<button className="!flex !items-center !gap-2 !text-sm !font-semibold !text-gray-800 hover:!text-gray-600 !transition-colors truncate">
<Icon
Expand Down
12 changes: 10 additions & 2 deletions webapp/_webapp/src/components/message-entry-container/tools/xtramcp/generate-citations.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,11 @@ export const GenerateCitationsCard = ({ functionName, message, preparing, animat
role="button"
tabIndex={0}
onClick={() => setIsMetadataCollapsed(!isMetadataCollapsed)}
onKeyDown={(e) => { if (e.key === 'Enter' || e.key === ' ') { setIsMetadataCollapsed(!isMetadataCollapsed); } }}
onKeyDown={(e) => {
if (e.key === "Enter" || e.key === " ") {
Copy link

Copilot AI Feb 18, 2026

Choose a reason for hiding this comment

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

Missing e.preventDefault() call when handling Space key. When Space is pressed on an element with role="button", the browser will scroll the page by default. Add e.preventDefault() inside the if statement to prevent unwanted scrolling behavior.

Suggested change
if (e.key === "Enter" || e.key === " ") {
if (e.key === "Enter" || e.key === " ") {
e.preventDefault();

Copilot uses AI. Check for mistakes.
setIsMetadataCollapsed(!isMetadataCollapsed);
}
}}
>
<h3 className="tool-card-title">{functionName}</h3>
<div className="flex items-center gap-2">
Expand Down Expand Up @@ -74,7 +78,11 @@ export const GenerateCitationsCard = ({ functionName, message, preparing, animat
role="button"
tabIndex={0}
onClick={() => setIsMetadataCollapsed(!isMetadataCollapsed)}
onKeyDown={(e) => { if (e.key === 'Enter' || e.key === ' ') { setIsMetadataCollapsed(!isMetadataCollapsed); } }}
onKeyDown={(e) => {
if (e.key === "Enter" || e.key === " ") {
Copy link

Copilot AI Feb 18, 2026

Choose a reason for hiding this comment

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

Missing e.preventDefault() call when handling Space key. When Space is pressed on an element with role="button", the browser will scroll the page by default. Add e.preventDefault() inside the if statement to prevent unwanted scrolling behavior.

Suggested change
if (e.key === "Enter" || e.key === " ") {
if (e.key === "Enter" || e.key === " ") {
e.preventDefault();

Copilot uses AI. Check for mistakes.
setIsMetadataCollapsed(!isMetadataCollapsed);
}
}}
>
<button
className="text-gray-400 hover:text-gray-600 transition-colors duration-200 rounded flex"
Expand Down
12 changes: 10 additions & 2 deletions webapp/_webapp/src/components/message-entry-container/tools/xtramcp/online-search-papers.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,11 @@ export const OnlineSearchPapersCard = ({ functionName, message, preparing, anima
role="button"
tabIndex={0}
onClick={() => setIsMetadataCollapsed(!isMetadataCollapsed)}
onKeyDown={(e) => { if (e.key === 'Enter' || e.key === ' ') { setIsMetadataCollapsed(!isMetadataCollapsed); } }}
onKeyDown={(e) => {
if (e.key === "Enter" || e.key === " ") {
Copy link

Copilot AI Feb 18, 2026

Choose a reason for hiding this comment

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

Missing e.preventDefault() call when handling Space key. When Space is pressed on an element with role="button", the browser will scroll the page by default. Add e.preventDefault() inside the if statement to prevent unwanted scrolling behavior.

Suggested change
if (e.key === "Enter" || e.key === " ") {
if (e.key === "Enter" || e.key === " ") {
e.preventDefault();

Copilot uses AI. Check for mistakes.
setIsMetadataCollapsed(!isMetadataCollapsed);
}
}}
>
<h3 className="tool-card-title">{functionName}</h3>
<div className="flex items-center gap-2">
Expand Down Expand Up @@ -78,7 +82,11 @@ export const OnlineSearchPapersCard = ({ functionName, message, preparing, anima
role="button"
tabIndex={0}
onClick={() => setIsMetadataCollapsed(!isMetadataCollapsed)}
onKeyDown={(e) => { if (e.key === 'Enter' || e.key === ' ') { setIsMetadataCollapsed(!isMetadataCollapsed); } }}
onKeyDown={(e) => {
if (e.key === "Enter" || e.key === " ") {
setIsMetadataCollapsed(!isMetadataCollapsed);
}
}}
Comment on lines +85 to +89
Copy link

Copilot AI Feb 18, 2026

Choose a reason for hiding this comment

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

Missing e.preventDefault() call when handling Space key. When Space is pressed on an element with role="button", the browser will scroll the page by default. Add e.preventDefault() inside the if statement to prevent unwanted scrolling behavior.

Copilot uses AI. Check for mistakes.
>
<h3 className="tool-card-title">{functionName}</h3>
<CollapseArrowButton
Expand Down
12 changes: 10 additions & 2 deletions webapp/_webapp/src/components/message-entry-container/tools/xtramcp/review-paper.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,11 @@ export const ReviewPaperCard = ({ functionName, message, preparing, animated }:
role="button"
tabIndex={0}
onClick={() => setIsMetadataCollapsed(!isMetadataCollapsed)}
onKeyDown={(e) => { if (e.key === 'Enter' || e.key === ' ') { setIsMetadataCollapsed(!isMetadataCollapsed); } }}
onKeyDown={(e) => {
if (e.key === "Enter" || e.key === " ") {
Copy link

Copilot AI Feb 18, 2026

Choose a reason for hiding this comment

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

Missing e.preventDefault() call when handling Space key. When Space is pressed on an element with role="button", the browser will scroll the page by default. Add e.preventDefault() inside the if statement to prevent unwanted scrolling behavior.

Suggested change
if (e.key === "Enter" || e.key === " ") {
if (e.key === "Enter" || e.key === " ") {
e.preventDefault();

Copilot uses AI. Check for mistakes.
setIsMetadataCollapsed(!isMetadataCollapsed);
}
}}
>
<h3 className="tool-card-title">{functionName}</h3>
<div className="flex items-center gap-2">
Expand Down Expand Up @@ -119,7 +123,11 @@ export const ReviewPaperCard = ({ functionName, message, preparing, animated }:
role="button"
tabIndex={0}
onClick={() => setIsMetadataCollapsed(!isMetadataCollapsed)}
onKeyDown={(e) => { if (e.key === 'Enter' || e.key === ' ') { setIsMetadataCollapsed(!isMetadataCollapsed); } }}
onKeyDown={(e) => {
if (e.key === "Enter" || e.key === " ") {
Copy link

Copilot AI Feb 18, 2026

Choose a reason for hiding this comment

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

Missing e.preventDefault() call when handling Space key. When Space is pressed on an element with role="button", the browser will scroll the page by default. Add e.preventDefault() inside the if statement to prevent unwanted scrolling behavior.

Suggested change
if (e.key === "Enter" || e.key === " ") {
if (e.key === "Enter" || e.key === " ") {
e.preventDefault();

Copilot uses AI. Check for mistakes.
setIsMetadataCollapsed(!isMetadataCollapsed);
}
}}
>
<h3 className="tool-card-title">{functionName}</h3>
<CollapseArrowButton
Expand Down
12 changes: 10 additions & 2 deletions ...p/_webapp/src/components/message-entry-container/tools/xtramcp/search-relevant-papers.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,11 @@ export const SearchRelevantPapersCard = ({ functionName, message, preparing, ani
role="button"
tabIndex={0}
onClick={() => setIsMetadataCollapsed(!isMetadataCollapsed)}
onKeyDown={(e) => { if (e.key === 'Enter' || e.key === ' ') { setIsMetadataCollapsed(!isMetadataCollapsed); } }}
onKeyDown={(e) => {
if (e.key === "Enter" || e.key === " ") {
Copy link

Copilot AI Feb 18, 2026

Choose a reason for hiding this comment

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

Missing e.preventDefault() call when handling Space key. When Space is pressed on an element with role="button", the browser will scroll the page by default. Add e.preventDefault() inside the if statement to prevent unwanted scrolling behavior.

Suggested change
if (e.key === "Enter" || e.key === " ") {
if (e.key === "Enter" || e.key === " ") {
if (e.key === " ") {
e.preventDefault();
}

Copilot uses AI. Check for mistakes.
setIsMetadataCollapsed(!isMetadataCollapsed);
}
}}
>
<h3 className="tool-card-title">{functionName}</h3>
<div className="flex items-center gap-2">
Expand Down Expand Up @@ -86,7 +90,11 @@ export const SearchRelevantPapersCard = ({ functionName, message, preparing, ani
role="button"
tabIndex={0}
onClick={() => setIsMetadataCollapsed(!isMetadataCollapsed)}
onKeyDown={(e) => { if (e.key === 'Enter' || e.key === ' ') { setIsMetadataCollapsed(!isMetadataCollapsed); } }}
onKeyDown={(e) => {
if (e.key === "Enter" || e.key === " ") {
Copy link

Copilot AI Feb 18, 2026

Choose a reason for hiding this comment

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

Missing e.preventDefault() call when handling Space key. When Space is pressed on an element with role="button", the browser will scroll the page by default. Add e.preventDefault() inside the if statement to prevent unwanted scrolling behavior.

Suggested change
if (e.key === "Enter" || e.key === " ") {
if (e.key === "Enter" || e.key === " ") {
if (e.key === " ") {
e.preventDefault();
}

Copilot uses AI. Check for mistakes.
setIsMetadataCollapsed(!isMetadataCollapsed);
}
}}
>
<h3 className="tool-card-title">{functionName}</h3>
<CollapseArrowButton
Expand Down
12 changes: 10 additions & 2 deletions webapp/_webapp/src/components/message-entry-container/tools/xtramcp/verify-citations.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,11 @@ export const VerifyCitationsCard = ({ functionName, message, preparing, animated
role="button"
tabIndex={0}
onClick={() => setIsMetadataCollapsed(!isMetadataCollapsed)}
onKeyDown={(e) => { if (e.key === 'Enter' || e.key === ' ') { setIsMetadataCollapsed(!isMetadataCollapsed); } }}
onKeyDown={(e) => {
if (e.key === "Enter" || e.key === " ") {
Copy link

Copilot AI Feb 18, 2026

Choose a reason for hiding this comment

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

Missing e.preventDefault() call when handling Space key. When Space is pressed on an element with role="button", the browser will scroll the page by default. Add e.preventDefault() inside the if statement to prevent unwanted scrolling behavior.

Suggested change
if (e.key === "Enter" || e.key === " ") {
if (e.key === "Enter" || e.key === " ") {
if (e.key === " ") {
e.preventDefault();
}

Copilot uses AI. Check for mistakes.
setIsMetadataCollapsed(!isMetadataCollapsed);
}
}}
>
<h3 className="tool-card-title">{functionName}</h3>
<div className="flex items-center gap-2">
Expand Down Expand Up @@ -78,7 +82,11 @@ export const VerifyCitationsCard = ({ functionName, message, preparing, animated
role="button"
tabIndex={0}
onClick={() => setIsMetadataCollapsed(!isMetadataCollapsed)}
onKeyDown={(e) => { if (e.key === 'Enter' || e.key === ' ') { setIsMetadataCollapsed(!isMetadataCollapsed); } }}
onKeyDown={(e) => {
if (e.key === "Enter" || e.key === " ") {
Copy link

Copilot AI Feb 18, 2026

Choose a reason for hiding this comment

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

Missing e.preventDefault() call when handling Space key. When Space is pressed on an element with role="button", the browser will scroll the page by default. Add e.preventDefault() inside the if statement to prevent unwanted scrolling behavior.

Suggested change
if (e.key === "Enter" || e.key === " ") {
if (e.key === "Enter" || e.key === " ") {
e.preventDefault();

Copilot uses AI. Check for mistakes.
setIsMetadataCollapsed(!isMetadataCollapsed);
}
}}
>
<h3 className="tool-card-title">{functionName}</h3>
<CollapseArrowButton
Expand Down
18 changes: 15 additions & 3 deletions webapp/_webapp/src/components/message-entry-container/tools/xtramcp/xtramcp-generic-card.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,11 @@ export const XtraMcpGenericCard = ({ functionName, message, preparing, animated
role="button"
tabIndex={0}
onClick={() => setIsMetadataCollapsed(!isMetadataCollapsed)}
onKeyDown={(e) => { if (e.key === 'Enter' || e.key === ' ') { setIsMetadataCollapsed(!isMetadataCollapsed); } }}
onKeyDown={(e) => {
if (e.key === "Enter" || e.key === " ") {
Copy link

Copilot AI Feb 18, 2026

Choose a reason for hiding this comment

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

Missing e.preventDefault() call when handling Space key. When Space is pressed on an element with role="button", the browser will scroll the page by default. Add e.preventDefault() inside the if statement to prevent unwanted scrolling behavior.

Suggested change
if (e.key === "Enter" || e.key === " ") {
if (e.key === "Enter" || e.key === " ") {
e.preventDefault();

Copilot uses AI. Check for mistakes.
setIsMetadataCollapsed(!isMetadataCollapsed);
}
}}
>
<h3 className="tool-card-title">{functionName}</h3>
<div className="flex items-center gap-2">
Expand Down Expand Up @@ -77,7 +81,11 @@ export const XtraMcpGenericCard = ({ functionName, message, preparing, animated
role="button"
tabIndex={0}
onClick={() => setIsMetadataCollapsed(!isMetadataCollapsed)}
onKeyDown={(e) => { if (e.key === 'Enter' || e.key === ' ') { setIsMetadataCollapsed(!isMetadataCollapsed); } }}
onKeyDown={(e) => {
if (e.key === "Enter" || e.key === " ") {
Copy link

Copilot AI Feb 18, 2026

Choose a reason for hiding this comment

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

Missing e.preventDefault() call when handling Space key. When Space is pressed on an element with role="button", the browser will scroll the page by default. Add e.preventDefault() inside the if statement to prevent unwanted scrolling behavior.

Suggested change
if (e.key === "Enter" || e.key === " ") {
if (e.key === "Enter" || e.key === " ") {
if (e.key === " ") {
e.preventDefault();
}

Copilot uses AI. Check for mistakes.
setIsMetadataCollapsed(!isMetadataCollapsed);
}
}}
>
<h3 className="tool-card-title">{functionName}</h3>
<CollapseArrowButton
Expand Down Expand Up @@ -148,7 +156,11 @@ export const XtraMcpGenericCard = ({ functionName, message, preparing, animated
role="button"
tabIndex={0}
onClick={() => setIsMetadataCollapsed(!isMetadataCollapsed)}
onKeyDown={(e) => { if (e.key === 'Enter' || e.key === ' ') { setIsMetadataCollapsed(!isMetadataCollapsed); } }}
onKeyDown={(e) => {
if (e.key === "Enter" || e.key === " ") {
setIsMetadataCollapsed(!isMetadataCollapsed);
}
}}
Comment on lines +159 to +163
Copy link

Copilot AI Feb 18, 2026

Choose a reason for hiding this comment

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

Missing e.preventDefault() call when handling Space key. When Space is pressed on an element with role="button", the browser will scroll the page by default. Add e.preventDefault() inside the if statement to prevent unwanted scrolling behavior.

Copilot uses AI. Check for mistakes.
>
<h3 className="tool-card-title">{functionName}</h3>
</div>
Expand Down
12 changes: 10 additions & 2 deletions webapp/_webapp/src/components/onboarding-guide.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -205,7 +205,11 @@ function OnboardingStep({ step, imageUrl, imageError, onImageClick, onImageError
role="button"
tabIndex={0}
onClick={onImageClick}
onKeyDown={(e) => { if (e.key === 'Enter' || e.key === ' ') { onImageClick(); } }}
onKeyDown={(e) => {
if (e.key === "Enter" || e.key === " ") {
Copy link

Copilot AI Feb 18, 2026

Choose a reason for hiding this comment

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

Missing e.preventDefault() call when handling Space key. When Space is pressed on an element with role="button", the browser will scroll the page by default. Add e.preventDefault() inside the if statement to prevent unwanted scrolling behavior.

Suggested change
if (e.key === "Enter" || e.key === " ") {
if (e.key === "Enter" || e.key === " ") {
e.preventDefault();

Copilot uses AI. Check for mistakes.
onImageClick();
}
}}
onError={onImageError}
/>
</div>
Expand Down Expand Up @@ -241,7 +245,11 @@ function ImageModal({
role="button"
tabIndex={0}
onClick={onClose}
onKeyDown={(e) => { if (e.key === 'Enter' || e.key === ' ') { onClose(); } }}
onKeyDown={(e) => {
if (e.key === "Enter" || e.key === " ") {
onClose();
}
}}
Comment on lines +248 to +252
Copy link

Copilot AI Feb 18, 2026

Choose a reason for hiding this comment

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

Missing e.preventDefault() call when handling Space key. When Space is pressed on an element with role="button", the browser will scroll the page by default. Add e.preventDefault() inside the if statement to prevent unwanted scrolling behavior.

Copilot uses AI. Check for mistakes.
onError={onImageError}
/>
)}
Expand Down
8 changes: 7 additions & 1 deletion webapp/_webapp/src/components/tabs.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -151,7 +151,13 @@ export const Tabs = forwardRef<TabRef, TabProps>(({ items }, ref) => {
<>
<div className="pd-app-tab-items noselect relative" style={{ width: `${tabItemsWidth}px` }}>
{/* Resize handle on the right edge */}
<div ref={resizeHandleRef} className="pd-tab-items-resize-handle" role="separator" aria-label="Resize tabs" onMouseDown={handleMouseDown}>
<div
ref={resizeHandleRef}
className="pd-tab-items-resize-handle"
role="separator"
aria-label="Resize tabs"
onMouseDown={handleMouseDown}
>
{/* Visual indicator line */}
<div className="resize-handle-indicator" />
</div>
Expand Down
19 changes: 11 additions & 8 deletions webapp/_webapp/src/libs/inline-suggestion.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,9 +52,7 @@ async function completeCitationKeys(state: EditorState, triggerText: string): Pr
const textBefore = state.doc.sliceString(textBeforeStart, cursorPos - triggerText.length);

// Split by sentence-ending punctuation and get the last sentence
const sentences = textBefore
.split(/(?<=[.!?])\s+/)
.filter((s) => s.trim().length > 0);
const sentences = textBefore.split(/(?<=[.!?])\s+/).filter((s) => s.trim().length > 0);
// Fall back to the bounded window if no sentence boundary found
const lastSentence = sentences.length > 0 ? sentences[sentences.length - 1] : textBefore.trim();
if (!lastSentence) {
Expand Down Expand Up @@ -636,8 +634,7 @@ export function createAutocompleteSuppressor(
this.handleKeydown = (e: KeyboardEvent) => {
if (e.key !== "Tab") return;

const suggestion =
this.view.state.field<SuggestionState>(suggestionState);
const suggestion = this.view.state.field<SuggestionState>(suggestionState);
if (!suggestion?.suggestion) return;

e.preventDefault();
Expand All @@ -663,8 +660,7 @@ export function createAutocompleteSuppressor(
}

update(update: ViewUpdate) {
const suggestion =
update.state.field<SuggestionState>(suggestionState);
const suggestion = update.state.field<SuggestionState>(suggestionState);
if (suggestion?.suggestion || isTriggerAtCursor(update.state)) {
suppress();
} else {
Expand Down Expand Up @@ -703,5 +699,12 @@ export function createSuggestionExtension(overleafCm: OverleafCodeMirror, config
const keymapBindingExtension = createExtensionKeymapBinding(overleafCm, suggestionAcceptanceEffect, suggestionState);
const autocompleteSuppressor = createAutocompleteSuppressor(overleafCm, suggestionState, suggestionAcceptanceEffect);

return [suggestionState, pluginConfigState, suggestionFetchPlugin, suggestionRenderingPlugin, keymapBindingExtension, autocompleteSuppressor];
return [
suggestionState,
pluginConfigState,
suggestionFetchPlugin,
suggestionRenderingPlugin,
keymapBindingExtension,
autocompleteSuppressor,
];
}
2 changes: 1 addition & 1 deletion webapp/_webapp/src/views/chat/body/index.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -134,7 +134,7 @@ export const ChatBody = ({ conversation }: ChatBodyProps) => {
}
}}
onKeyDown={async (e) => {
if (e.key === 'Enter' || e.key === ' ') {
if (e.key === "Enter" || e.key === " ") {
try {
const response = await getConversation({ conversationId: conversation?.id ?? "" });
if (!response.conversation) {
Expand Down
10 changes: 8 additions & 2 deletions webapp/_webapp/src/views/chat/footer/toolbar/selection.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,13 @@ export function Selection<T>({ items, initialValue, onSelect, onClose }: Selecti
const { user } = useAuthStore();
const scrollContainerRef = useRef<HTMLDivElement>(null);
const [{ selectedIdx, isKeyboardNavigation }, dispatchSelection] = useReducer(
(state: { selectedIdx: number; isKeyboardNavigation: boolean }, action: { type: "SET_IDX"; idx: number } | { type: "SET_KEYBOARD_NAV"; value: boolean } | { type: "SET_IDX_WITH_KEYBOARD_NAV"; idx: number }) => {
(
state: { selectedIdx: number; isKeyboardNavigation: boolean },
action:
| { type: "SET_IDX"; idx: number }
| { type: "SET_KEYBOARD_NAV"; value: boolean }
| { type: "SET_IDX_WITH_KEYBOARD_NAV"; idx: number },
) => {
switch (action.type) {
case "SET_IDX":
return { ...state, selectedIdx: action.idx };
Expand Down Expand Up @@ -182,7 +188,7 @@ export function Selection<T>({ items, initialValue, onSelect, onClose }: Selecti
onSelect?.(item);
}}
onKeyDown={(e) => {
if (e.key === 'Enter' || e.key === ' ') {
if (e.key === "Enter" || e.key === " ") {
if (item.disabled) return;
googleAnalytics.fireEvent(user?.id, `select_${normalizeName(item.title)}`, {});
onSelect?.(item);
Expand Down
2 changes: 1 addition & 1 deletion webapp/_webapp/src/views/chat/header/chat-button.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ export const ChatButton = ({
onClick?.();
}}
onKeyDown={(e) => {
if (e.key === 'Enter' || e.key === ' ') {
if (e.key === "Enter" || e.key === " ") {
if (disabled) return;
googleAnalytics.fireEvent(user?.id, "click_chat_button_" + normalizeName(text ?? alt ?? icon), {});
onClick?.();
Expand Down
4 changes: 3 additions & 1 deletion webapp/_webapp/src/views/devtools/index.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -310,7 +310,9 @@ export const DevTools = () => {

{/* Preparing delay */}
<div className="flex flex-row gap-2 items-center pt-2 border-t !border-orange-200">
<label htmlFor="preparing-delay-input" className="text-sm text-gray-700 whitespace-nowrap">Preparing delay (seconds):</label>
<label htmlFor="preparing-delay-input" className="text-sm text-gray-700 whitespace-nowrap">
Preparing delay (seconds):
</label>
<Input
id="preparing-delay-input"
size="sm"
Expand Down
8 changes: 7 additions & 1 deletion webapp/_webapp/src/views/embed-sidebar.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -264,7 +264,13 @@ export const EmbedSidebar = () => {
}}
>
{/* Resize handle on the left */}
<div ref={resizeHandleRef} className="pd-embed-resize-handle" role="separator" aria-label="Resize sidebar" onMouseDown={handleMouseDown}>
<div
ref={resizeHandleRef}
className="pd-embed-resize-handle"
role="separator"
aria-label="Resize sidebar"
onMouseDown={handleMouseDown}
>
{/* Visual indicator line */}
<div className="resize-handle-indicator" />
</div>
Expand Down
Loading