From 9de4e906c2056bc0278f0f74a33a53eb7f0c29ab Mon Sep 17 00:00:00 2001 From: Henriette Darge Date: Fri, 4 Oct 2024 08:25:43 +0200 Subject: [PATCH] Remove outdated section about vertical spacing in forms --- lookbook/docs/patterns/02-forms.md.erb | 8 -------- 1 file changed, 8 deletions(-) diff --git a/lookbook/docs/patterns/02-forms.md.erb b/lookbook/docs/patterns/02-forms.md.erb index 81a134ef375c..5008f1872a85 100644 --- a/lookbook/docs/patterns/02-forms.md.erb +++ b/lookbook/docs/patterns/02-forms.md.erb @@ -45,14 +45,6 @@ The options are: <%= embed Patterns::FormsPreview, :default, panels: %i[] %> -## Vertical spacing - -By default, form elements do not have proper vertical spacing in Primer. We recommend a 16px (`stack/gap/normal`) vertical separate between individual elements. There is an [open issue in Primer's GitHub](https://github.com/primer/view_components/issues/3042) to fix this. - -Until this is fixed at a component level, please do not manually apply form padding. - -An alternative approach is to wrap individual form elements using `Form group`. Please only use this sparingly and only when it is absolutely necessarily. - ## Technical notes ### Usage