Skip to content

feat: add estimated reading time feature#214

Open
SamruddhiPatil0078 wants to merge 3 commits into
pushkarscripts:mainfrom
SamruddhiPatil0078:feature/estimated-reading-time
Open

feat: add estimated reading time feature#214
SamruddhiPatil0078 wants to merge 3 commits into
pushkarscripts:mainfrom
SamruddhiPatil0078:feature/estimated-reading-time

Conversation

@SamruddhiPatil0078
Copy link
Copy Markdown

Related Issue

Fixes #211


Description

Implemented an Estimated Reading Time feature for course content and notes.

The feature automatically calculates reading time based on the content's word count and displays it below the course/chapter title. This helps students estimate the time required to complete a topic before they start reading.


Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • UI/UX improvement
  • Refactor

Screenshots / Demo

Added reading time indicator below the course title.

Example:

  • ⏱️ 2 min read
image

How Has This Been Tested?

  • Tested locally using the development server.
  • Verified reading time calculation for content with different word counts.
  • Confirmed that the reading time badge renders correctly on course pages.
  • Checked responsive behavior on different screen sizes.

Checklist

  • My code follows the project structure and conventions
  • I have tested the changes locally
  • I have linked the related issue properly
  • I have added screenshots where applicable
  • No existing functionality was broken

AI Usage

  • I have not used AI tools for this contribution

  • I have used AI tools (ChatGPT, Copilot, Claude, etc.) and I have reviewed, verified, and understood all generated code/content before submitting this PR

If AI tools were used, briefly explain how:

Used ChatGPT for brainstorming the feature idea, drafting issue/PR descriptions, and reviewing implementation approaches. All code was reviewed, tested, and verified before submission.


Additional Notes

This feature improves the learning experience by helping students plan their study sessions more effectively. The implementation is lightweight and integrates seamlessly with the existing UI.

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 29, 2026

@SamruddhiPatil0078 is attempting to deploy a commit to the PushkarScripts Team on Vercel.

A member of the Team first needs to authorize it.

@SamruddhiPatil0078
Copy link
Copy Markdown
Author

Resolved merge conflicts with upstream/main. Both the BookmarkButton and the estimated reading time feature are preserved.

@pushkarscripts
Copy link
Copy Markdown
Owner

@SamruddhiPatil0078 Thanks for working on this.

One thing to note: contributors are expected to wait for approval/assignment before starting work on an issue. Since this is your first contribution and the feature makes sense, I'll let it pass this time. Please follow the workflow going forward.

As for the implementation, I think the placement can be improved a bit. It may fit better right after the Course Outline heading or alongside the module/chapter title rather than below it.
image

Try a couple of placement options and see what feels more natural with the existing layout. Once updated, feel free to request another review.

@pushkarscripts pushkarscripts added enhancement New feature or request gssoc-26 Part of GirlScript Summer of Code 2026 gssoc:approved Approved for GSSOC '26 level:advanced quality:exceptional type:feature labels Jun 4, 2026
@SamruddhiPatil0078
Copy link
Copy Markdown
Author

I have updated the layout so the Reading Time indicator now sits right alongside the module/chapter title as requested!
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request gssoc:approved Approved for GSSOC '26 gssoc-26 Part of GirlScript Summer of Code 2026 level:advanced quality:exceptional type:feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE] Add Estimated Reading Time for Notes and Articles

2 participants