Skip to content

Implement units design changes#776

Merged
rachelekm merged 7 commits into
developfrom
fix/rkm/tweak-find-units-ui
Dec 22, 2025
Merged

Implement units design changes#776
rachelekm merged 7 commits into
developfrom
fix/rkm/tweak-find-units-ui

Conversation

@rachelekm

@rachelekm rachelekm commented Dec 22, 2025

Copy link
Copy Markdown
Contributor

Overview

This PR implements a design refactor of the neighborhood details units tab following a client meeting and some small UX clean up.

The refactor better promotes the affordability calculator, including:

  • Adding the affordability calculator as a 3rd step following "find units"
  • Removing the units modal entirely that appears on clicking find units platform link
  • Adding link to affordability calculator in subtitle of find units step

Also includes:

  • Adding commute disclaimer under "Your Trips"
  • Adding missing translations and cleaning up default translations before handoff

Checklist

  • fixup! commits have been squashed
  • CHANGELOG.md updated with summary of features or fixes, following
    Keep a Changelog guidelines
  • README.md updated if necessary to reflect the changes
  • Run ./scripts/format to lint, format, and fix the application source code.
  • CI passes after rebase

Demo

Screenshot 2025-12-22 at 10 35 49 AM Screenshot 2025-12-22 at 10 36 05 AM

Testing Instructions

  • scripts/server and login
  • Select a neighborhood to open details modal
  • Confirm commute disclaimer appears under "Your Trips"
  • Navigate to units tab
  • Confirm can click "Affordability Calculator" in the Step 2 Find Units section and it opens the affordability calculator page in a new tab
  • Click any of the "find units" housing platform buttons and confirm no regressions opening in new tab
  • Confirm modal no longer appears back in app tab after clicking
  • Confirm new Step 3 that matches designs, here

Resolves #731

Comment thread src/frontend/public/locales/zh/translation.json
Comment thread src/frontend/src/pages/Discover/Neighborhoods.tsx

@aaronxsu aaronxsu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looking great and working very well. This is good to go.

components={{
calcLink: (
<a
href="https://www.bostonhousing.org/en/Section-8-Leased-Housing/How-Rent-is-Set/Can-I-afford-this-apartment-Use-our-new-Rent-Esti.aspx"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should we add this link to constants?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good call 👍

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Added in 9ae7689

@rachelekm rachelekm force-pushed the fix/rkm/tweak-find-units-ui branch from c5b8d43 to 93425ee Compare December 22, 2025 22:48
@rachelekm rachelekm force-pushed the fix/rkm/tweak-find-units-ui branch from 93425ee to 544424f Compare December 22, 2025 22:53
@rachelekm rachelekm mentioned this pull request Dec 22, 2025
5 tasks
@rachelekm rachelekm merged commit c41f150 into develop Dec 22, 2025
2 checks passed
@rachelekm rachelekm deleted the fix/rkm/tweak-find-units-ui branch December 22, 2025 23:07
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.

Neighborhood details copy updates

2 participants