Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

fix: formProps/actionsRender params not working in QueryTable issue #116

Merged
merged 2 commits into from
Jul 22, 2021
Merged

fix: formProps/actionsRender params not working in QueryTable issue #116

merged 2 commits into from
Jul 22, 2021

Conversation

chenkang2208
Copy link

No description provided.

@xz8la8
Copy link
Member

xz8la8 commented Jul 21, 2021

还是传 actionsRender 进来吧，这样改怕对已有业务有影响

@chenkang2208
Copy link
Author

还是传 actionsRender 进来吧，这样改怕对已有业务有影响

传actionsRender，按钮是出现在表格的右侧，如果QueryTable里面不能支持修改QueryForm的actionsRender，这样QueryTable的灵活性就不如纯用QueryForm了，业务上需要修改Form的按钮 那就只能拆成QueryForm + Table来实现了

@xz8la8
Copy link
Member

xz8la8 commented Jul 21, 2021

还是传 actionsRender 进来吧，这样改怕对已有业务有影响

传actionsRender，按钮是出现在表格的右侧，如果QueryTable里面不能支持修改QueryForm的actionsRender，这样QueryTable的灵活性就不如纯用QueryForm了，业务上需要修改Form的按钮 那就只能拆成QueryForm + Table来实现了

不能直接把formProps优先级提这个高，这么改可能会影响已有业务，你再看下是否还有其他改法哈

@chenkang2208
Copy link
Author

还是传 actionsRender 进来吧，这样改怕对已有业务有影响

传actionsRender，按钮是出现在表格的右侧，如果QueryTable里面不能支持修改QueryForm的actionsRender，这样QueryTable的灵活性就不如纯用QueryForm了，业务上需要修改Form的按钮 那就只能拆成QueryForm + Table来实现了

不能直接把formProps优先级提这个高，这么改可能会影响已有业务，你再看下是否还有其他改法哈

其它改法就是放弃QueryTable了 :( � ，目前QueryTable的查询表单固定是两个按钮，想多加一个按钮进来用QueryTable是不行了

@xz8la8
Copy link
Member

xz8la8 commented Jul 21, 2021

还是传 actionsRender 进来吧，这样改怕对已有业务有影响

传actionsRender，按钮是出现在表格的右侧，如果QueryTable里面不能支持修改QueryForm的actionsRender，这样QueryTable的灵活性就不如纯用QueryForm了，业务上需要修改Form的按钮 那就只能拆成QueryForm + Table来实现了

不能直接把formProps优先级提这个高，这么改可能会影响已有业务，你再看下是否还有其他改法哈

其它改法就是放弃QueryTable了 :( � ，目前QueryTable的查询表单固定是两个按钮，想多加一个按钮进来用QueryTable是不行了

明天再讨论下吧，今天脑子已罢工。。。

@chenkang2208
Copy link
Author

还是传 actionsRender 进来吧，这样改怕对已有业务有影响

传actionsRender，按钮是出现在表格的右侧，如果QueryTable里面不能支持修改QueryForm的actionsRender，这样QueryTable的灵活性就不如纯用QueryForm了，业务上需要修改Form的按钮 那就只能拆成QueryForm + Table来实现了

不能直接把formProps优先级提这个高，这么改可能会影响已有业务，你再看下是否还有其他改法哈

其它改法就是放弃QueryTable了 :( � ，目前QueryTable的查询表单固定是两个按钮，想多加一个按钮进来用QueryTable是不行了

明天再讨论下吧，今天脑子已罢工。。。

如果怕影响面大的话，可以只把formProps里面的actionsRender优先级提高，其它属性还是不变

@xz8la8
Copy link
Member

xz8la8 commented Jul 21, 2021

还是传 actionsRender 进来吧，这样改怕对已有业务有影响

传actionsRender，按钮是出现在表格的右侧，如果QueryTable里面不能支持修改QueryForm的actionsRender，这样QueryTable的灵活性就不如纯用QueryForm了，业务上需要修改Form的按钮 那就只能拆成QueryForm + Table来实现了

不能直接把formProps优先级提这个高，这么改可能会影响已有业务，你再看下是否还有其他改法哈

其它改法就是放弃QueryTable了 :( � ，目前QueryTable的查询表单固定是两个按钮，想多加一个按钮进来用QueryTable是不行了

明天再讨论下吧，今天脑子已罢工。。。

如果怕影响面大的话，可以只把formProps里面的actionsRender优先级提高，其它属性还是不变

我也是这个意思

@xz8la8 xz8la8 merged commit a33250c into umijs:master Jul 22, 2021
@chenkang2208 chenkang2208 deleted the hotfix_querytable_formprops branch July 22, 2021 05:38
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@chenkang2208 @xz8la8 @kang-chen-20230331