Skip to content

Conversation

@AbhiCode-247
Copy link
Contributor

@AbhiCode-247 AbhiCode-247 commented Oct 28, 2025

Closes #80

This PR implements the UI enhancement for the community links as described in the issue.

Changes Made

  • Removed the default underline from the "Slack Community" and "GitHub repository" links.
  • Made the link text slightly bolder (font-weight: 500) for emphasis.
  • Added a smooth transition so the underline now appears only on hover.

Screenshots

Before (No Hover)
image

After (With Hover on Slack Community)
image

Summary by CodeRabbit

  • Style
    • Enhanced link styling in documentation admonitions with improved hover effects and smooth text-decoration transitions.

@vercel
Copy link

vercel bot commented Oct 28, 2025

@AbhiCode-247 is attempting to deploy a commit to the idan lodzki's projects Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai
Copy link

coderabbitai bot commented Oct 28, 2025

Walkthrough

CSS styling rules are added to modify link behavior within admonition content blocks. The changes remove visible underlines by default, increase font weight, add a smooth transition, and reveal underlines on hover with the current link color.

Changes

Cohort / File(s) Summary
Admonition content link styling
opsimate-docs/src/css/custom.css
Adds rules for .admonitionContent_pbrs p a to hide underlines by default (transparent decoration color), increase font-weight, and transition text-decoration-color to currentColor on hover

Assessment against linked issues

Objective Addressed Explanation
Remove underline from links by default [#80]
Display underline only on hover [#80]
Make link text slightly bolder [#80]
Add smooth ease-in-out transition [#80] Summary indicates a transition is added for text-decoration-color but does not specify if the timing function is explicitly set to ease-in-out

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c3c5373 and 5fdad3f.

📒 Files selected for processing (1)
  • opsimate-docs/src/css/custom.css (1 hunks)
🔇 Additional comments (2)
opsimate-docs/src/css/custom.css (2)

871-880: Implementation approach is effective and aligns with PR objectives.

The CSS uses a modern technique: rendering the underline with a transparent color by default, then transitioning it to visible on hover. This achieves all stated objectives:

  • ✓ Removes visible underline by default
  • ✓ Appears only on hover
  • ✓ Smooth transition (0.3s ease-in-out, consistent with other transitions in the file)
  • ✓ Increased font-weight (500) for emphasis without relying on underline alone

The use of currentColor for the underline color is semantically correct and ensures the underline inherits the link's text color.


871-880: Verify that broad admonition link styling is intentional.

The CSS selector .admonitionContent_pbrs p a successfully targets Docusaurus admonition elements. However, verification reveals it applies to all links in all admonitions throughout the documentation (intro.md, deploy.md, core-features.md, system-requirements.md), not just the Slack/GitHub community links in intro.md.

Confirm:

  • Is styling all admonition links intentional, or should the selector be narrower (e.g., targeting only the community admonition)?
  • If broad styling is intentional, this is working as expected and the implementation is sound.

The CSS itself is technically correct—no conflicts found, and the text-decoration animation approach is valid.


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

@idanlodzki idanlodzki merged commit 39a4ba5 into OpsiMate:main Oct 29, 2025
2 of 4 checks passed
@AbhiCode-247 AbhiCode-247 deleted the fix/issue-80-link-underline branch October 29, 2025 21:25
@AbhiCode-247
Copy link
Contributor Author

@idanlodzki could you please add the hacktoberfest and hacktoberfest-accepted labels to this 🙏🏻

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.

[UI]: Underline from Slack and GitHub links should be removed

2 participants