Skip to content

improvement(docs): soften video hover opacity#4339

Merged
waleedlatif1 merged 1 commit intostagingfrom
waleedlatif1/video-hover-opacity
Apr 29, 2026
Merged

improvement(docs): soften video hover opacity#4339
waleedlatif1 merged 1 commit intostagingfrom
waleedlatif1/video-hover-opacity

Conversation

@waleedlatif1
Copy link
Copy Markdown
Collaborator

Summary

  • Reduce hover opacity change on docs video components from 95% to 97% for a subtler effect

Type of Change

  • Improvement

Testing

Tested manually

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 29, 2026

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

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Apr 29, 2026 3:39pm

Request Review

@cursor
Copy link
Copy Markdown

cursor Bot commented Apr 29, 2026

PR Summary

Low Risk
CSS-only tweak to a docs UI hover effect; no functional or data-handling behavior changes.

Overview
Adjusts the docs Video component hover styling when enableLightbox is on by reducing the opacity change from hover:opacity-95 to hover:opacity-[0.97] for a subtler hover effect.

Reviewed by Cursor Bugbot for commit 29c8f88. Configure here.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Apr 29, 2026

Greptile Summary

This PR softens the hover opacity effect on docs video components by changing hover:opacity-95 to hover:opacity-[0.97], producing a more subtle visual feedback on hover. The arbitrary-value Tailwind syntax [0.97] is appropriate since 97 is not in Tailwind's default opacity scale.

Confidence Score: 5/5

Safe to merge — single cosmetic change with no logic impact.

The change is a one-line Tailwind class update with no functional, logic, or security implications. No custom rules are violated.

No files require special attention.

Important Files Changed

Filename Overview
apps/docs/components/ui/video.tsx Single-line tweak: hover opacity on the lightbox-enabled video element changed from opacity-95 (0.95) to an arbitrary value opacity-[0.97] (0.97) for a subtler hover effect.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Video Component] --> B{enableLightbox?}
    B -- Yes --> C[Apply cursor-pointer and transition-opacity]
    C --> D[On hover: opacity 0.97 subtle fade]
    B -- No --> E[No hover effect applied]
Loading

Reviews (1): Last reviewed commit: "improvement(docs): soften video hover op..." | Re-trigger Greptile

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