Skip to content

Fixed the PostCard, and HeroPost card to make them consistent #113

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Swapnendu003
Copy link

Related Tickets & Documents

Fixes: #2653

Description

This PR enhances the visual consistency and user experience of blog post cards throughout the website.

  • Implemented consistent card heights regardless of content length.
  • Added dynamic content display based on title length.
  • Enhanced hover effects with orange borders and shadows.

Changes

  • Modified post-preview.tsx to ensure cards maintain consistent height with dynamic excerpt display.
  • Updated hero-post.tsx to match the styling patterns of post previews.
  • Improved hover interactions with consistent orange-themed transitions.

Type of Change

  • Chore (maintenance, refactoring, tooling updates)
  • Bug fix (non-breaking change that fixes an issue)
  • New feature (change that adds functionality)
  • Breaking Change (may require updates in existing code)
  • UI improvement (visual or design changes)
  • Performance improvement (optimization or efficiency enhancements)
  • Documentation update (changes to README, guides, etc.)
  • CI (updates to continuous integration workflows)
  • Revert (undo a previous commit or merge)

Testing

  • Tested with various post titles (short, medium, long) to ensure dynamic content display works correctly.
  • Verified consistent card heights across all screens.
  • Confirmed hover effects work smoothly in Firefox, Chrome, and Safari.

Demo

Before: Cards had inconsistent heights based on content
image
After: All cards maintain consistent layout with proper spacing regardless of content length
image

Environment and Dependencies

  • New Dependencies: None
  • Configuration Changes: None

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have added corresponding tests
  • My changes have been tested across relevant browsers/devices
  • For UI changes, I've included visual evidence of my changes

Signed-off-by: Swapnendu003 <swaps.b003@gmail.com>
Copy link

keploy bot commented Apr 23, 2025

Nice Pr 😄. We currently support Unit Test Generation only for Go projects.
Stay tuned – support for more languages is coming soon! 🚀

Achanandhi-M and others added 2 commits April 24, 2025 18:43
* Added google-adsense

Signed-off-by: Achanandhi-M <achanandhi.m@gmail.com>

* chore/modified-google-adsense-tag

Signed-off-by: Achanandhi-M <achanandhi.m@gmail.com>

---------

Signed-off-by: Achanandhi-M <achanandhi.m@gmail.com>
@nehagup
Copy link
Member

nehagup commented Apr 24, 2025

@amaan-bhati please review

@amaan-bhati
Copy link
Member

Looks good to me, can be merged, nothing conflicting here, just changes in the card ui, content displayed on the card and the size of the card.
Only one concern here : @Swapnendu003 I am not testing it locally rn since the code looks working, can you confirm if it is responsive for mobile screens as well, maybe add a sccreenshot for mobile view as well.

@amaan-bhati
Copy link
Member

Also @Swapnendu003 You need to run npm run build every time when you make changes before creating a pr. If nothing breaks, take a screenshot and add it here itself. There will be no breaks in the workflow as well.

@Swapnendu003
Copy link
Author

image

image

@amaan-bhati I have checked the build there is no build issues in this PR

@amaan-bhati
Copy link
Member

@Swapnendu003 Looks good to me!!

@Swapnendu003
Copy link
Author

Looks good to me, can be merged, nothing conflicting here, just changes in the card ui, content displayed on the card and the size of the card. Only one concern here : @Swapnendu003 I am not testing it locally rn since the code looks working, can you confirm if it is responsive for mobile screens as well, maybe add a sccreenshot for mobile view as well.

@amaan-bhati These are the mobile and tablet views
image
Screenshot 2025-05-07 201536

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.

[bug]: Fix PostCard & HeroPost Layout Consistency in blog website
4 participants