Skip to content
This repository was archived by the owner on Feb 9, 2026. It is now read-only.

Implement clickable usernames in forum web page#531

Merged
VolkanSeki merged 4 commits into
mainfrom
make-usernames-clickable-in-forum-web-page
Nov 22, 2025
Merged

Implement clickable usernames in forum web page#531
VolkanSeki merged 4 commits into
mainfrom
make-usernames-clickable-in-forum-web-page

Conversation

@cck181851

@cck181851 cck181851 commented Nov 14, 2025

Copy link
Copy Markdown
Contributor

Corresponding issue

Add Clickable Username Functionality to Forum Components

Description

This PR adds clickable username functionality to forum threads, comments, and subcomments, allowing users to navigate to profile pages by clicking on usernames. This enhances user interaction and provides a seamless way to view other users' profiles directly from forum discussions.

Changes Made

  • Added useNavigate hook integration in CommentItem, SubcommentItem, and Thread page components
  • Implemented clickable username spans with proper hover effects
  • Added CSS styling for consistent clickable username appearance across all components
  • Ensured proper event handling to prevent parent click event interference
  • Maintained existing UI while adding navigation functionality

Features

  • Clickable usernames in thread author section, comments and subcomments

API Endpoints

No API endpoint introduced

Dependencies

No new dependencies required

Checklist

  • My code follows style guidelines
  • I have performed self-review
  • I have commented complex code
  • My changes generate no new warnings
  • Any dependent changes have been merged

@cck181851 cck181851 self-assigned this Nov 14, 2025
@cck181851 cck181851 added in progress This issue is being worked on new This is a new issue workload: moderate 1-3 hours per week priority: moderate Needs to be completed in a week need review Work related to this issue is completed, needs a review to close it. type: implementation This issue is about implementation. labels Nov 14, 2025
@cck181851 cck181851 added this to the MVP milestone Nov 14, 2025
@cck181851 cck181851 linked an issue Nov 14, 2025 that may be closed by this pull request
4 tasks
@cck181851

cck181851 commented Nov 14, 2025

Copy link
Copy Markdown
Contributor Author

@doran2728 Can you review the changes and merge with main if they are working as expected?

@cck181851
cck181851 requested a review from doran2728 November 14, 2025 15:38
@cck181851 cck181851 closed this Nov 15, 2025
@cck181851

Copy link
Copy Markdown
Contributor Author

I will fix an issue and reopen this PR afterward.

@cck181851 cck181851 reopened this Nov 18, 2025
@cck181851

cck181851 commented Nov 18, 2025

Copy link
Copy Markdown
Contributor Author

@doran2728 The PR is reopened. Could you review the changes and merge with main if the feature is working as expected? Note that I've resolved a conflict caused by another PR (#524), so both features should now be working correctly.

@VolkanSeki
VolkanSeki merged commit 4ad05dd into main Nov 22, 2025
2 checks passed
@VolkanSeki

VolkanSeki commented Nov 22, 2025

Copy link
Copy Markdown
Contributor

Good job

@cck181851

Copy link
Copy Markdown
Contributor Author

@volkanBora123 I think you have closed the wrong PR

@cck181851
cck181851 deleted the make-usernames-clickable-in-forum-web-page branch November 23, 2025 00:01
@cck181851 cck181851 added completed This issue is completed and removed in progress This issue is being worked on need review Work related to this issue is completed, needs a review to close it. labels Dec 11, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

completed This issue is completed new This is a new issue priority: moderate Needs to be completed in a week type: implementation This issue is about implementation. workload: moderate 1-3 hours per week

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Made the thread authors' profile pictures and commenters' names clickable

2 participants