Skip to content

README tool count matches enabled tools - #26

Merged
nadimtuhin merged 1 commit into
mainfrom
fix/readme-tool-count
Oct 14, 2026
Merged

nadimtuhin merged 1 commit into
mainfrom
fix/readme-tool-count

Conversation

@nadimtuhin

Copy link
Copy Markdown
Owner

README said 38 tools; App.tsx has 33 enabled tools (34 isEnabled minus the Credits page). Drops the 5 disabled tools from the table and says offline works while the tab stays open (no service worker).

@vercel

vercel Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
devutils Ready Ready Preview Oct 14, 2026 4:40pm UTC

@nadimtuhin
nadimtuhin merged commit 7861052 into main Oct 14, 2026
2 of 3 checks passed
@nadimtuhin
nadimtuhin deleted the fix/readme-tool-count branch October 14, 2026 16:40
@kilo-code-bot

kilo-code-bot Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

Code Review Summary

Status: 1 Suggestion | Recommendation: Consider before merge

Overview

Severity Count
CRITICAL 0
WARNING 0
SUGGESTION 1
Issue Details (click to expand)

SUGGESTION

File Line Issue
README.md 145-146 Screenshots still show Color Converter and Markdown Preview, which are disabled tools removed from the table
Files Reviewed (1 file)
  • README.md - 1 suggestion

Analysis

The README changes are accurate:

  • Tool count (lines 94, 100): "33 tools" matches src/App.tsx — 34 tools with isEnabled: true minus the Credits page = 33.
  • Table (lines 107, 110): The 5 disabled tools (isEnabled: false in App.tsx: HTML Minify/Beautify, CSS Processor, JavaScript Formatter, Color Converter, Markdown Preview) are correctly removed.
  • Offline clarification (line 99): Accurately states there is no service worker and fresh visits need the network.
  • Footer (line 154): "Works without a network once loaded" remains consistent with the new line 99 wording.

Suggestion: The Screenshots section (lines 145-146) still includes screenshots of Color Converter and Markdown Preview — both are disabled tools (isEnabled: false in src/App.tsx:730,722) that were just removed from the tools table. Consider replacing these with screenshots of currently enabled tools for consistency with the updated table.


Reviewed by free · Input: 143.3K · Output: 24.8K · Cached: 131.1K

This branch was successfully deployed

1 active deployment
Preview — fa492f2a Deployed Oct 14, 2026 by vercel[bot]
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