Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add section about 'Building configurable components' #488

Merged
merged 1 commit into from
Jan 9, 2025

Conversation

romaricpascal
Copy link
Member

@romaricpascal romaricpascal commented Nov 19, 2024

Important

Only to be merged as part of the 5.8.0 release

Adds the content drafted ahead of the release (internal link) to the 'Building your JavaScript components' page (preview).

I went through the code blocks paying attention to smart quotes and indentation, but it's possible I missed some so please keep an eye for those during the review.

Fixes #486

@romaricpascal romaricpascal added documentation Improvements or additions to documentation 🚀 release labels Nov 19, 2024
Copy link

netlify bot commented Nov 19, 2024

Deploy Preview for govuk-frontend-docs-preview ready!

Name Link
🔨 Latest commit b240387
🔍 Latest deploy log https://app.netlify.com/sites/govuk-frontend-docs-preview/deploys/677fbc32549cb50008bae26c
😎 Deploy Preview https://deploy-preview-488--govuk-frontend-docs-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@romaricpascal romaricpascal force-pushed the add-building-configurable-components branch from cf41d8d to 5ee8543 Compare November 19, 2024 17:26
@@ -253,6 +253,189 @@ createAll(ProjectComponent, undefined, {
})
```

## Building configurable components
The component you’re building may need some configuration to get the information it needs to:
Copy link
Member Author

Choose a reason for hiding this comment

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

nitpick The 'to:' ends up all alone on its own line even on wide screen. Admitedly that's something that's going to happen at other screen sizes, but should we fix that when the content container is at its widest?

Screenshot 2024-11-19 at 17 29 32

Copy link
Contributor

Choose a reason for hiding this comment

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

Don't think this should block updating. We can finesse if needed!

@romaricpascal romaricpascal force-pushed the add-building-configurable-components branch from 5ee8543 to b240387 Compare January 9, 2025 12:08
@romaricpascal romaricpascal merged commit 6a35c29 into main Jan 9, 2025
5 checks passed
@romaricpascal romaricpascal deleted the add-building-configurable-components branch January 9, 2025 12:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation 🚀 release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add 'Building configurable components' section to the Frontend Docs
2 participants