Skip to content

Improve gray content#757

Merged
feruzm merged 2 commits into
developfrom
sty
Apr 14, 2026
Merged

Improve gray content#757
feruzm merged 2 commits into
developfrom
sty

Conversation

@feruzm
Copy link
Copy Markdown
Member

@feruzm feruzm commented Apr 14, 2026

Summary by CodeRabbit

Release Notes

  • New Features

    • Enhanced link styling with colored underlines across all themes (developer, minimal, modern-gradient).
    • Added improved visibility indicators for hidden and muted content.
  • Bug Fixes

    • Adjusted content visibility thresholds to more accurately reflect community moderation decisions.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 14, 2026

Warning

Rate limit exceeded

@feruzm has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 28 minutes and 4 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 28 minutes and 4 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 3bc9f625-b328-4c0f-af61-e5e69f2a06c3

📥 Commits

Reviewing files that changed from the base of the PR and between 00e6358 and 18a912c.

📒 Files selected for processing (6)
  • apps/web/src/app/(dynamicPages)/entry/[category]/[author]/[permlink]/_components/entry-page-warnings.tsx
  • apps/web/src/app/waves/_components/waves-list-item.tsx
  • apps/web/src/features/shared/discussion/discussion-item.tsx
  • apps/web/src/features/shared/entry-list-item/entry-list-item-muted-content.tsx
  • apps/web/src/utils/index.ts
  • apps/web/src/utils/is-hidden-post.ts
📝 Walkthrough

Walkthrough

The PR updates link styling across three theme files to display underlines with semi-transparent decoration colors, adjusts content visibility thresholds across multiple components from -7,000,000,000 to -10,000,000,000 net rshares, adds internationalization support for a hidden content message, and refactors muted content overlay handling to separate hidden and moderated states.

Changes

Cohort / File(s) Summary
Theme Link Styling
apps/self-hosted/src/styles/themes/developer.css, minimal.css, modern-gradient.css
Updated link decoration from none to underline with semi-transparent colored decoration values across all three themes (developer: rgba(137, 180, 250, 0.4), minimal: rgba(0, 102, 204, 0.3), modern-gradient: color-mix formula).
Hidden Content Threshold
apps/web/src/app/(dynamicPages)/entry/[category]/[author]/[permlink]/_components/entry-page-warnings.tsx, apps/web/src/app/waves/_components/waves-list-item.tsx, apps/web/src/features/shared/discussion/discussion-item.tsx
Raised the net rshares visibility threshold from -7,000,000,000 to -10,000,000,000 across three components that determine hidden content status, while maintaining the additional requirement for active votes.
Muted Content Overlay Refactoring
apps/web/src/features/shared/entry-list-item/entry-list-item-muted-content.tsx
Refactored muted/hidden content handling: removed image rendering when content is muted, separated showHidden state from showModMuted, replaced inline reveal logic with dedicated handler, added opacity styling, and updated message selection to use i18n keys for mod-muted vs hidden vs muted states.
Internationalization Addition
apps/web/src/features/i18n/locales/en-US.json
Added new i18n message key g.hidden-message with value "Downvoted by users, Reveal content" for hidden content indicators.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

  • #729: Modifies waves-list-item.tsx with layout and styling changes in the same component affected by threshold logic updates.
  • #733: Updates waves-list-item.tsx with image grid functionality alongside the threshold changes in this PR.

Suggested labels

patch

Poem

🐰 Links now wear their stripes with pride,
Hidden posts set a higher tide,
Themes aligned in color schemes,
Content sorted by new thresholds' dreams,
A patch that smooths the UI seams! ✨

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check ❓ Inconclusive The title 'Improve gray content' is vague and doesn't clearly convey what specific aspects of gray/hidden content handling are being improved. Use a more specific title that describes the main changes, such as 'Update link styling across themes and adjust hidden content threshold' or 'Refactor hidden content detection and theme link styles'.
✅ Passed checks (1 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch sty

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

coderabbitai[bot]

This comment was marked as resolved.

@feruzm feruzm merged commit fc39428 into develop Apr 14, 2026
1 check passed
@feruzm feruzm deleted the sty branch April 14, 2026 07:04
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