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

Define architecture for inline editing display #308

Closed
10 of 12 tasks
brynwhyman opened this issue Aug 7, 2018 · 4 comments
Closed
10 of 12 tasks

Define architecture for inline editing display #308

brynwhyman opened this issue Aug 7, 2018 · 4 comments

Comments

@brynwhyman
Copy link

brynwhyman commented Aug 7, 2018

An API should be used to define the UI of inline editing of a block. This would allow the developer to define what fields of a block should be edited 'inline' and what should be accessed via the [future] (...) menu item to be edited with the existing GridFeildDetailForm.

Investigate how this will be implemented.

Acceptance Criteria

@robbieaverill
Copy link
Contributor

Related: #304

@ScopeyNZ
Copy link
Contributor

I think we should aim for a react alternative to GridFieldDetailForm. Currently we will still need it for blocks that aren't inline editable (eg. userform blocks). Related issue: silverstripe/silverstripe-admin#590

@ScopeyNZ
Copy link
Contributor

If we want to have the forms saved via a GraphQL mutation, there is an issue for this on GraphQL: silverstripe/silverstripe-graphql#17 . As long as we are using FormBuilderLoader and the API it provides we should be able to update this fairly painlessly (in theory). I suggest we use the existing REST implementation now - baked into FormBuilderLoader

@ScopeyNZ
Copy link
Contributor

ScopeyNZ commented Aug 16, 2018

In regards to work for the OSSers - perhaps it would be best to discuss this while we estimate issues - opting to leave ones that might be suited for them? I'm not really sure how to decide on best candidate issue for them. Some existing issues on their repos like silverstripe/silverstripe-graphql#17, silverstripe/silverstripe-admin#556 and silverstripe/silverstripe-admin#590 would be nice to have - but not required.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants