Skip to content

Conversation

@MohammadRehaanDev
Copy link
Contributor

@MohammadRehaanDev MohammadRehaanDev commented Oct 29, 2025

Screenshot 2025-10-29 at 9 30 57 PM ## ✨ What does this PR do?

This PR addresses and fixes Issue #72 by making UI improvements to the Docusaurus Welcome Page (index.jsx).

🛠️ Key Changes

  • Improved Heading Hierarchy: Changed the main title (<h1>) to a more semantically correct heading level, as it functions as a secondary title on the page.
  • Conciseness: Edited the accompanying text for better clarity and brevity, making the welcome experience less overwhelming for new users.

🔗 Related Issue

This PR closes: #72


🎃 Hacktoberfest Contribution

This pull request is submitted as part of Hacktoberfest 2025.

Summary by CodeRabbit

  • Style

    • Updated hero text presentation and color for a more cohesive look.
  • Refactor

    • Simplified the hero layout by consolidating separate heading and subtitle elements into a single paragraph for cleaner markup and consistent display.

@vercel
Copy link

vercel bot commented Oct 29, 2025

@MohammadRehaanDev 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.

@github-actions
Copy link

Thanks for your contribution! Join our Slack: https://join.slack.com/t/opsimate/shared_invite/zt-39bq3x6et-NrVCZzH7xuBGIXmOjJM7gA Please make sure to include an image with your PR — it really helps us review and understand the changes better. Only in rare cases will we accept a PR without one. Also, take a moment to review your code to ensure it’s clear, readable, and easy to follow. PRs from contributors who haven’t joined our Slack community or starred the repository won’t be reviewed — we’d love to see you join and be part of the project!

@coderabbitai
Copy link

coderabbitai bot commented Oct 29, 2025

Walkthrough

Consolidated the hero section text by collapsing separate subtitle/description elements into a single paragraph and modified the hero subtitle CSS selector and color.

Changes

Cohort / File(s) Change Summary
Hero content markup
opsimate-docs/src/pages/index.jsx, opsimate-docs/docs/intro.md
Removed separate subtitle/description elements and replaced them with a single paragraph element containing the hero description; the main h1 remains.
Hero subtitle stylesheet
opsimate-docs/src/pages/index.module.css
Changed selector from a class-style .heroSubtitle to css.heroSubtitle and updated color from #e3f2fd to #aeb4e7.

Assessment against linked issues

Objective Addressed Explanation
Subtitle should be smaller than the main title [#72]
Subtitle color should be slightly darker gray [#72]
Consider centering the title and subtitle [#72] No centering (e.g., text-center) added in the hero markup or CSS; unclear if global/layout styles center it.
Reduce size of other titles like "The OpsiMate Vision" [#72] Only the hero markup/CSS were changed; other page titles were not modified.

Assessment against linked issues: Out-of-scope changes

Code Change Explanation
Changed selector from .heroSubtitle to css.heroSubtitle (opsimate-docs/src/pages/index.module.css) This selector format is invalid for a class and is not an objective in the linked issue; it may break styling and is unrelated to the stated UI objectives.

Suggested labels

hacktoberfest


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

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 2

📜 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 1903e3c.

⛔ Files ignored due to path filters (1)
  • package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (2)
  • opsimate-docs/docs/intro.md (1 hunks)
  • opsimate-docs/src/pages/index.jsx (1 hunks)

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

📜 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 1903e3c and c59772e.

📒 Files selected for processing (3)
  • opsimate-docs/docs/intro.md (1 hunks)
  • opsimate-docs/src/pages/index.jsx (1 hunks)
  • opsimate-docs/src/pages/index.module.css (1 hunks)
🚧 Files skipped from review as they are similar to previous changes (2)
  • opsimate-docs/docs/intro.md
  • opsimate-docs/src/pages/index.jsx

@idanlodzki
Copy link
Contributor

@MohammadRehaanDev check the comments

@MohammadRehaanDev
Copy link
Contributor Author

MohammadRehaanDev commented Oct 30, 2025

Hi @idanlodzki,

The final fix for the subtitle color and CSS refactoring has been pushed in commit c59772e.

I have:

  • Refactored the styling to use CSS Modules consistently.
  • Attached a screenshot of the working change to the PR description.
  • Confirmed I have starred the repository and joined the Slack community (as requested by the bot).

@idanlodzki idanlodzki linked an issue Oct 30, 2025 that may be closed by this pull request
@idanlodzki idanlodzki added hacktoberfest hacktoberfest issue hacktoberfest-accepted hacktoberfest-accepted issue labels Oct 30, 2025
@idanlodzki idanlodzki merged commit d89cbba into OpsiMate:main Oct 30, 2025
1 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hacktoberfest hacktoberfest issue hacktoberfest-accepted hacktoberfest-accepted issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[UI] Welcome Page Improvements

2 participants