Skip to content

fix(frontend): show feature card text on mobile devices#1056

Open
kshah123919 wants to merge 4 commits into
imDarshanGK:mainfrom
kshah123919:fix/1047-feature-card-mobile-text
Open

fix(frontend): show feature card text on mobile devices#1056
kshah123919 wants to merge 4 commits into
imDarshanGK:mainfrom
kshah123919:fix/1047-feature-card-mobile-text

Conversation

@kshah123919

Copy link
Copy Markdown
Contributor

Description

Fixed a mobile responsiveness issue where feature card titles and descriptions were hidden on screens below 400px. Removed the CSS rule that hid the feature card text, ensuring that feature icons, titles, and descriptions remain visible and readable on mobile devices.

Related Issue

Fixes #1047

Type of change

  • Bug fix
  • New feature / enhancement
  • Documentation update
  • Test addition
  • Refactor

Checklist

  • I have read [CONTRIBUTING.md](../CONTRIBUTING.md)
  • My branch is up to date with main
  • I have run pytest -v and all tests pass
  • I have not introduced duplicate issues or features
  • My PR title follows the format: feat/fix/docs/test: short description
  • I have added tests for new features (Level 2 and 3 issues)
  • No hardcoded secrets or API keys in my code
  • This PR is linked to a GSSoC 2026 issue

Screenshots (if frontend change)

Before

Feature card titles and descriptions were hidden on mobile devices, leaving only the icons visible.

After

Feature card icons, titles, and descriptions are now visible and readable on mobile devices.

Screenshot_17-6-2026_11179_127 0 0 1

Test evidence

Verified manually using Chrome DevTools mobile device emulation.

Tested on multiple mobile device profiles, including iPhone and Samsung devices.

Results:

  • ✓ Feature icons are visible
  • ✓ Feature titles are visible
  • ✓ Feature descriptions are visible
  • ✓ Content remains readable on mobile screens
  • ✓ No text overlap or truncation observed

@kshah123919 kshah123919 requested a review from imDarshanGK as a code owner June 17, 2026 05:59
@kshah123919

Copy link
Copy Markdown
Contributor Author

Hi @imDarshanGK ,

I have submitted a PR for Issue #1047. The issue causing feature card titles and descriptions to be hidden on mobile devices has been fixed. I have also attached screenshots showing the before and after behavior for verification.

Could you please review the PR when you get a chance ?

Thank you for your time and feedback.

@imDarshanGK imDarshanGK left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

The fix addresses Issue #1047 by making the feature card text visible on mobile.

However, the mobile layout could be improved. Currently all feature cards stack vertically, which feels a bit crowded. Consider using a responsive grid (e.g., 2 cards per row on mobile where possible) to maintain readability and better use of screen space.

Please share updated mobile screenshots after the change.

@kshah123919

Copy link
Copy Markdown
Contributor Author

Thank you for reviewing the PR and for the feedback.

I'll improve the mobile layout by making the feature cards responsive (showing two cards per row where possible) to make better use of the available space and improve readability.

I'll update the screenshots and push the changes shortly.

@kshah123919

Copy link
Copy Markdown
Contributor Author

Thank you for the feedback @imDarshanGK ,

I've updated the mobile layout to use a more compact two-column arrangement and improved the card sizing and spacing for better use of screen space. I've also attached updated screenshots for verification.

Screenshot_20-6-2026_114211_

Please let me know if any further adjustments are needed.

Thank you!

@kshah123919 kshah123919 force-pushed the fix/1047-feature-card-mobile-text branch from 2e8748c to 00f7639 Compare June 20, 2026 07:02
@kshah123919 kshah123919 requested a review from imDarshanGK June 21, 2026 05:24

@imDarshanGK imDarshanGK left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

@kshah123919 I noticed this line was removed from .github/pull_request_template.md:

- [ ] I have updated docs/CHANGELOG.md for user-facing changes or fixes

Could you explain why this checklist item was removed? It seems unrelated to the scope of this issue.

@kshah123919

Copy link
Copy Markdown
Contributor Author

Thanks for pointing this out. This removal was accidental and unrelated to the scope of the issue. I didn't intend to modify the PR template, and I'll restore the checklist item and update the PR accordingly.

@kshah123919 kshah123919 force-pushed the fix/1047-feature-card-mobile-text branch from 8fc7588 to c2e1787 Compare June 22, 2026 04:47
@kshah123919 kshah123919 requested a review from imDarshanGK June 22, 2026 04:51
@kshah123919

Copy link
Copy Markdown
Contributor Author

Hi @imDarshanGK,

I have restored the accidentally removed checklist item and updated the PR. All checks are passing now.
Could you please review it when you get a chance ?

Thank you!

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] Feature Card Text Content Not Visible on Mobile Devices

2 participants