Skip to content

Improve compact feature recommendation panel in onboarding - #500

Merged
harshitarora-in merged 3 commits into
developmentfrom
feat/onboarding-compact-features-panel
Aug 5, 2026
Merged

Improve compact feature recommendation panel in onboarding#500
harshitarora-in merged 3 commits into
developmentfrom
feat/onboarding-compact-features-panel

Conversation

@harshitarora-in

@harshitarora-in harshitarora-in commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Summary

This PR improves the onboarding feature recommendation panel to make it more compact and easier to scan, while keeping the existing selection behavior intact.

Changes included:

  • Improved the compact Features panel layout in onboarding for better usability in limited vertical space.
  • Added auto-fit behavior so feature descriptions collapse when panel height is not enough.
  • Kept descriptions expanded by default when there is enough space.
  • Prioritized collapsing selected/required feature descriptions first to preserve marketing copy on unselected cards when possible.
  • Added fallback behavior to collapse all descriptions if space is still insufficient.
  • Improved feature card accessibility (including contrast/focus visibility improvements).
  • Updated onboarding styling to support the new compact/auto-fit behavior.
  • Added/updated onboarding E2E test coverage for the compact features panel behavior.

Will affect visual aspect of the product

YES

Screenshots

Screenshot 2026-08-04 at 6 31 00 PM

Test instructions

  1. Open onboarding and go to the Starter Site step with the Features panel.
  2. At normal screen height:
    • Feature descriptions should be expanded by default.
  3. Reduce the browser window height:
    • Descriptions should auto-collapse so the panel fits.
  4. Reduce height further (very small viewport):
    • All descriptions should be collapsed.
  5. Click any feature card chevron:
    • The card should expand/collapse correctly.

Expected Result

  • The Features panel should not overflow into the bottom area.
  • Compact behavior should trigger automatically on smaller heights.

Check before Pull Request is ready:

Closes #.

@harshitarora-in
harshitarora-in requested a review from Copilot July 22, 2026 12:51
@pirate-bot pirate-bot added the pr-checklist-incomplete The Pull Request checklist is incomplete. (automatic label) label Jul 22, 2026
@pirate-bot

pirate-bot commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator

Plugin build for cc30a46 is ready 🛎️!

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Compacts and simplifies the onboarding feature recommendation panel.

Changes:

  • Tightens panel and feature-card spacing.
  • Consolidates Otter recommendations.
  • Shortens plugin descriptions.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File Description
onboarding/src/scss/_site-settings.scss Adds compact feature-step styling.
onboarding/src/scss/_features_list.scss Restyles feature cards and layout.
onboarding/src/Components/SiteSettings.js Applies feature-step-specific class.
onboarding/src/Components/FeaturesList.js Consolidates recommendations and shortens descriptions.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread onboarding/src/scss/_features_list.scss Outdated
Comment thread onboarding/src/scss/_features_list.scss Outdated
display: flex;
flex-direction: column;
width: 100%;
overflow-x: hidden;

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.

@Soare-Robert-Daniel Soare-Robert-Daniel left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@harshitarora-in can you check if the E2E also need to be updated?

@harshitarora-in

Copy link
Copy Markdown
Contributor Author

@copilot Can you review this PR and make sure it won't fail the E2E tests? If the tests need to be updated because of these changes, please update them.

Also, can you check whether this PR satisfies the following checklist items?

If you find anything that needs to be updated, please make the necessary changes and explain what you changed.

@Soare-Robert-Daniel Soare-Robert-Daniel added the pr-checklist-skip Allow this Pull Request to skip checklist. label Aug 4, 2026
@pirate-bot pirate-bot added pr-checklist-complete The Pull Request checklist is complete. (automatic label) and removed pr-checklist-incomplete The Pull Request checklist is incomplete. (automatic label) labels Aug 4, 2026
Descriptions now render expanded and collapse only when the card list
would run past the footer, so the panel no longer needs scrolling on
short viewports. Each card gets a chevron toggle that reuses the title
for its accessible name, and selected features give up their
description first since unselected ones still need the pitch.

Updates the e2e selectors to target the new markup via data-plugin and
relaxes the card count assertion, as the collection is now five entries.
@harshitarora-in
harshitarora-in force-pushed the feat/onboarding-compact-features-panel branch from 7b1c0b6 to cc30a46 Compare August 4, 2026 12:55
@harshitarora-in

Copy link
Copy Markdown
Contributor Author

@Soare-Robert-Daniel, with the old implementation, it was introducing multiple new strings. To avoid that, I changed the implementation a bit by adding an expandable card while keeping the existing string unchanged.

Screenshot 2026-08-04 at 6 28 28 PM

@harshitarora-in
harshitarora-in merged commit 71d796c into development Aug 5, 2026
8 checks passed
@harshitarora-in
harshitarora-in deleted the feat/onboarding-compact-features-panel branch August 5, 2026 12:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-checklist-complete The Pull Request checklist is complete. (automatic label) pr-checklist-skip Allow this Pull Request to skip checklist.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants