Add Edit Dialog pattern
See original GitHub issueDo not use EditDialog
within a full page form/record layout. This not a replacement for Inline Edit.
It is meant for standalone “modify this word” parts of a page.
This is not an Inline Edit pattern, but does have a similar UX pattern. If you were using an InlineEdit pattern outside a form, then this Edit Dialog variant of input is the replacement component. An Inline Edit UX pattern within a form is A COMPLETELY DIFFERENT THING and is outside the scope of this Github issue. You should go look at #1395
DSR’s Inline Edit Form Input is a different pattern than https://www.lightningdesignsystem.com/components/form-element/#Inline-Edit
- Current
InlineEdit
component should be deprecated. [DONE] - There is no close or cancel on blur with the new examples. ~Inline Edit appears to be upgrade path to Fuel UX’s Placard pattern, but this should be discouraged.~ [DONE]
- ~Deprecation warning of current
InlineEdit
should point to a newPopover
example on the doc site that shows an edit icon triggering a Popover with a baseInput
within it.~ [DONE] - ~An example of a Inline Edit Form that swaps a read-only
Input
(that has a right-aligned edit icon on it) for a Base variantInput
should be added toInput
. Example should look like https://www.lightningdesignsystem.com/components/form-element/#View-Mode and https://www.lightningdesignsystem.com/components/form-element/#Edit-Mode (this is a different issue and outside scope of this issue, that should be considered a new Form component with a docked footer)~ #1395
~We’ll need to do this in consideration of https://github.com/salesforce/design-system-react/issues/273 which involves moving components/form/InlineEdit
to components/InlineEdit
, but in agreement with this issue should not be moved. It should be “removed” from the doc site and deprecated.~ [DONE]
This is NOT a breaking change.
Issue Analytics
- State:
- Created 5 years ago
- Comments:9 (7 by maintainers)
@shaundr Just contacted me and said that the Trailhead team was interested in this component.
I think this is the path forward for InlineEdit. @kwilloughby @aweibel