Skip to content

feat add processing conversation on desktop#4607

Open
krushnarout wants to merge 2 commits intomainfrom
feat/desktop-ui-changes
Open

feat add processing conversation on desktop#4607
krushnarout wants to merge 2 commits intomainfrom
feat/desktop-ui-changes

Conversation

@krushnarout
Copy link
Member

Screenshot 2026-02-04 at 11 54 24 PM

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a widget to display conversations that are currently being processed on the desktop version of the application. The implementation is clean and integrates well with the existing conversation list. I have one suggestion to improve the new widget by making it display dynamic data instead of being a static placeholder, which also resolves an unused parameter issue.

Comment on lines 61 to 69
// Timestamp placeholder
Container(
width: 60,
height: 14,
decoration: BoxDecoration(
color: ResponsiveHelper.backgroundQuaternary,
borderRadius: BorderRadius.circular(4),
),
),
Copy link
Contributor

Choose a reason for hiding this comment

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

high

This placeholder for the timestamp can be replaced with the actual start time from the conversation object to provide more context about the processing item. This also resolves the issue of the conversation parameter being unused in this widget.

Note: You'll need to add the following import at the top of the file:

import 'package:intl/intl.dart';
Suggested change
// Timestamp placeholder
Container(
width: 60,
height: 14,
decoration: BoxDecoration(
color: ResponsiveHelper.backgroundQuaternary,
borderRadius: BorderRadius.circular(4),
),
),
// Timestamp
Text(
DateFormat.jm(Localizations.localeOf(context).languageCode)
.format(conversation.startedAt ?? conversation.createdAt),
style: const TextStyle(
color: ResponsiveHelper.textTertiary,
fontSize: 13,
fontWeight: FontWeight.w500,
),
),

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.

1 participant