Skip to content

implemented readonly property for form components#202

Open
jlamanskygitt wants to merge 3 commits into
DiscipleTools:masterfrom
jlamanskygitt:master
Open

implemented readonly property for form components#202
jlamanskygitt wants to merge 3 commits into
DiscipleTools:masterfrom
jlamanskygitt:master

Conversation

@jlamanskygitt
Copy link
Copy Markdown
Contributor

Implemented readonly & created stories for all form fields except:
dt-button
dt-copy-text
dt-file-upload
dt-label

Some weren't changed much visually, including:
dt-church-health-circle
dt-toggle
dt-multi-select-button-group

Resolves #171

@netlify
Copy link
Copy Markdown

netlify Bot commented May 22, 2026

Deploy Preview for jade-chebakia-17493f ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit b82fc25
🔍 Latest deploy log https://app.netlify.com/projects/jade-chebakia-17493f/deploys/6a10789f2a7e850008cbd4f4
😎 Deploy Preview https://deploy-preview-202--jade-chebakia-17493f.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 project configuration.

@cairocoder01
Copy link
Copy Markdown
Collaborator

@jlamanskygitt This is looking great! Nice work so far. A few things from my testing:

  • For the location-meta field, it would be great to have a link to be able to view the location, like the mock-up below. It can open the same map modal that is opened for the editable version
  • The issue had a note about components that are marked both readonly and disabled, that they wouldn't show the edit pencil and wouldn't be able to go into edit mode. It looks like the functionality for that works. Can we add stories for that too?
  • Can the readonly version of multi-select-button-group show the same as the readonly of the multi-select so that it doesn't show the button UI?
  • dt-text-multi shows the plus icon instead of the edit icon for readonly
  • For dt-text-multi, there is a type attribute that can be used to set the type of all the text inputs. If that is "email," we should show the values as mailto: links. If it is "tel," we should show the readonly values as tel: links. See mockup below.
  • For connections and users-connection, there is an option to pass a link with each value that could link to the connected record. Can we add that to at least one of the values in the story and make sure the link is clickable.
  • Can you add another story under All Components that shows all of them in readonly mode, just like there's one that shows all of them in disabled mode?
image

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.

Readonly mode for all form components

2 participants