Skip to content

Conversation

@rsgb
Copy link
Contributor

@rsgb rsgb commented Jan 29, 2026

Summary

  • Fixed font-family to use Inter (matching published comments)
  • Fixed font-size to 16px on desktop (matching published comments)
  • Reduced textarea rows from 2 to 1 to align height with submit button
  • Added responsive styles for mobile (12px font, 11px padding to match 40px button)

Notes

On mobile, font-size is reduced to 12px to match the 40px button height. This prioritizes height alignment over font consistency on mobile screens.

Fixes #4432

PR Checklist

  • - Unit and/or e2e tests for the changes that have been added (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Refactoring (no functional changes)
  • Documentation
  • Other:

What is the new behavior?

Describe the new behaviour
If useful, provide screenshot or capture to highlight main changes

Does this PR introduce a DB Schema Change or Migration?

  • Yes
  • No

Git Issues

Closes #

What happens next?

Thank you for the contribution! We will review it ASAP.

If you need more immediate feedback you can reach out to us on Discord in the Community Platform development channel.

- Set font-family to Inter and font-size to 16px to match published comments
- Change rows from 2 to 1 to reduce initial height
- Add mobile media query for responsive height alignment

Fixes ONEARMY#4432

Co-authored-by: Cursor <[email protected]>
@rsgb
Copy link
Contributor Author

rsgb commented Jan 29, 2026

This is my first open source contribution! Happy to make any changes if needed.

@cypress
Copy link

cypress bot commented Jan 29, 2026

onearmy-community-platform    Run #8452

Run Properties:  status check failed Failed #8452  •  git commit a677fd960d: Merge branch 'master' into fix/comment-box-styling-4432
Project onearmy-community-platform
Branch Review pull/4590
Run status status check failed Failed #8452
Run duration 08m 00s
Commit git commit a677fd960d: Merge branch 'master' into fix/comment-box-styling-4432
Committer Mário Nunes
View all properties for this run ↗︎

Test results
Tests that failed  Failures 1
Tests that were flaky  Flaky 6
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 93
View all changes introduced in this branch ↗︎

Tests for review

Failed  src/integration/news/write.spec.ts • 1 failed test • ci-chrome

View Output Video

Test Artifacts
[News.Write] > [Create a news item] > [By Authenticated] Test Replay Screenshots Video
Flakiness  profile.spec.ts • 1 flaky test • ci-chrome

View Output Video

Test Artifacts
[Profile] > [By User] > [User directed to own profile] Test Replay Screenshots Video
Flakiness  settings.spec.ts • 2 flaky tests • ci-chrome

View Output Video

Test Artifacts
Notifications Test Replay Screenshots Video
[Settings] > Can create member Test Replay Screenshots Video
Flakiness  research/write.spec.ts • 1 flaky test • ci-chrome

View Output Video

Test Artifacts
[Research] > [Create research article] > [By Authenticated] Test Replay Screenshots Video
Flakiness  research/discussions.spec.ts • 1 flaky test • ci-chrome

View Output Video

Test Artifacts
[Research.Discussions] > allows authenticated users to contribute to discussions Test Replay Screenshots Video
Flakiness  common.spec.ts • 1 flaky test • ci-chrome

View Output Video

Test Artifacts
[Common] > [User Menu] > [By Authenticated] Test Replay Screenshots Video

@mariojsnunes mariojsnunes added the Review allow-preview ✅ Has received manual check for malicious code and can be safely built for preview label Jan 30, 2026
@mariojsnunes mariojsnunes merged commit ffa5688 into ONEARMY:master Jan 30, 2026
9 of 11 checks passed
@github-project-automation github-project-automation bot moved this from 🆕 Backlog to ✅ Done in Core Team - BAU list Jan 30, 2026
@mariojsnunes
Copy link
Contributor

Heey @rsgb, thanks for this!

Looks good on the preview branch, merging.
(Note that for this kind of UI changes it is nice to have a screenshot)

@mariojsnunes
Copy link
Contributor

@all-contributors add @rsgb for code

@allcontributors
Copy link
Contributor

@mariojsnunes

I've put up a pull request to add @rsgb! 🎉

@rsgb
Copy link
Contributor Author

rsgb commented Jan 30, 2026

Hey @mariojsnunes , thanks for the quick review! Also noted: will add screenshots next time, when changing UI!

@onearmy-bot
Copy link
Collaborator

🎉 This PR is included in version 2.105.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released Review allow-preview ✅ Has received manual check for malicious code and can be safely built for preview

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[bug] Fix the frontend of comment box

3 participants