Skip to content

Conversation

LemonDrew
Copy link

No description provided.

@LemonDrew LemonDrew requested review from okkhoy, ITZXITZX, CJianzhi, hyc17003 and FooChao and removed request for okkhoy September 17, 2025 12:30
Copy link

@FooChao FooChao left a comment

Choose a reason for hiding this comment

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

Just a small minor bug in the editor where changing langauge does not actually change the syntax highlighting and suggestion.

For now it is ok but in future we should make it such that each language has its own state so that when change language it will disappear and reappear after they change it back.

@FooChao
Copy link

FooChao commented Sep 17, 2025

image image

additionally i notice a unnecessary scrollbar on the editor when the code is short and no need to scroll. but i feel that it may be the default library built in bug

FooChao
FooChao previously approved these changes Sep 19, 2025
Copy link

@FooChao FooChao left a comment

Choose a reason for hiding this comment

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

lgtm

@LemonDrew LemonDrew dismissed FooChao’s stale review September 19, 2025 10:46

The merge-base changed after approval.

@LemonDrew
Copy link
Author

image image
additionally i notice a unnecessary scrollbar on the editor when the code is short and no need to scroll. but i feel that it may be the default library built in bug

Bug has been fixed

Copy link

@CJianzhi CJianzhi left a comment

Choose a reason for hiding this comment

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

LGTM! Collab page looks clean and easy to use. However, is it possible to include a short description for future PRs so it is easier for reviewers to identify the areas to look out for? @LemonDrew

Verified that scroll bar has been disabled when code does not exceed current screen height

Verified that language selector works as intended with correct syntax highlighting

Verified that collab page is only accessible by authenticated users

Copy link

@hyc17003 hyc17003 left a comment

Choose a reason for hiding this comment

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

lgtm! looks and works as expected, code looks p clean as well, nice job
Agree with @CJianzhi 's comment on PR descriptions

Copy link

@ITZXITZX ITZXITZX left a comment

Choose a reason for hiding this comment

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

LGTM

  • Text Editor is now working, with highlighting, and scrolling enabled
  • Able to key into chat box, overflow spills right

import { ChevronRightIcon } from "lucide-react";

export default function ChatComponent() {
const chatMessages = ["HELLO", "HOW ARE YOU?"];

Choose a reason for hiding this comment

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

Noticed that there is some boiler plate chat messages here. Do you intend to render them for this PR, as its currently not rendered?

@ITZXITZX ITZXITZX merged commit 38073c0 into master Sep 19, 2025
1 check passed
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.

5 participants