Skip to content

Pro 8247 sections empty state#5072

Merged
ValJed merged 5 commits intomainfrom
pro-8247-sections-empty-state
Sep 16, 2025
Merged

Pro 8247 sections empty state#5072
ValJed merged 5 commits intomainfrom
pro-8247-sections-empty-state

Conversation

@ValJed
Copy link
Copy Markdown
Contributor

@ValJed ValJed commented Sep 11, 2025

https://linear.app/apostrophecms/issue/PRO-8247/as-a-user-i-want-directions-for-how-to-add-my-first-section-so-that-i

Summary

  • Rework empty state style for pieces.
  • Adds pieces options to update empty state title message and icon.

What are the specific steps to test this change?

Cypress running: 🟠

What kind of change does this PR introduce?

  • Bug fix
  • New feature
  • Refactor
  • Documentation
  • Build-related changes
  • Other

Make sure the PR fulfills these requirements:

  • It includes a) the existing issue ID being resolved, b) a convincing reason for adding this feature, or c) a clear description of the bug it resolves
  • The changelog is updated
  • Related documentation has been updated
  • Related tests have been updated

If adding a new feature without an already open issue, it's best to open a feature request issue first and wait for approval before working on it.

Other information:

@linear
Copy link
Copy Markdown

linear Bot commented Sep 11, 2025

boutell
boutell previously approved these changes Sep 12, 2025
Copy link
Copy Markdown
Member

@stuartromanek stuartromanek left a comment

Choose a reason for hiding this comment

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

do not merge, styles need work, will update

stuartromanek
stuartromanek previously approved these changes Sep 15, 2025
}

.apos-modal__body-main {
flex-grow: 1;
Copy link
Copy Markdown
Contributor Author

@ValJed ValJed Sep 16, 2025

Choose a reason for hiding this comment

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

@stuartromanek I just changed this line, replaced the height: 100%.
Also running cypress tests.

@ValJed ValJed force-pushed the pro-8247-sections-empty-state branch from 0c528b4 to f447d6d Compare September 16, 2025 12:18
@ValJed ValJed merged commit 4c52d4b into main Sep 16, 2025
6 checks passed
@ValJed ValJed deleted the pro-8247-sections-empty-state branch September 16, 2025 12:55
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.

3 participants