Nextcloud-vue: ActionInput: add Multisect

Created on 20 Dec 2019  路  8Comments  路  Source: nextcloud/nextcloud-vue

It would be nice if we could provide a drop-down list for the ActionInput component (see https://github.com/nextcloud/notes/issues/413#issuecomment-565825366). Therefore, I propose to add the existing Multiselect component to ActionInput. It could be used instead of the default HTML input field if the new optional prop options is set. All other props would be passed, so further customization would be possible.

Any doubts on this? I would try to provide a PR otherwise.

1. to develop enhancement actions

Most helpful comment

I don鈥檛 see an issue with opening a dropdown in the actions menu, especially when it鈥檚 about suggestions for an element in there. It鈥檚 totally necessary. :)

All 8 comments

@korelstar I don't think we should allow opening a drop-down menu on top of the action menu, It can be very confusing.

@ma12-co But that's what's already done with the date picker when you have type=date in ActionInput. Where is the difference?

@korelstar that shouldn't happen either In my opinion.. I know it's technically possible right now but are you aware of a datepicker opened from an action menu in Nextcloud atm?

cc @skjnldsv @jancborchardt

This was implemented by @skjnldsv - maybe he can tell ...

We had this behaviour for the sharing section of the files sidebar, so this was implemented to avoid regression, but this is a fair discussion.
If anyone have a better idea, it's fine :)
I'll let Jan decide what is best here! :wink:

@jancborchardt What do you think about this? Can we enhance the ActionInput component by providing suggestions using the Multiselect component?

I don鈥檛 see an issue with opening a dropdown in the actions menu, especially when it鈥檚 about suggestions for an element in there. It鈥檚 totally necessary. :)

Anyone can take over this issue then :)

Was this page helpful?
0 / 5 - 0 ratings

Related issues

jancborchardt picture jancborchardt  路  3Comments

ma12-co picture ma12-co  路  7Comments

korelstar picture korelstar  路  7Comments

ma12-co picture ma12-co  路  4Comments

korelstar picture korelstar  路  7Comments