Allow set a function for `joinLastRow` parameter

[sdcdsvsdv423]

In some complicated cases I need to display a button on the same or on the next row depending on context. Now the joinLastRow parameter has boolean type and it's impossible to set value dynamically.

It would be nice if this parameter has ContextPathFunc<Context, boolean> type (the same as hide parameter).

[EdJoPaTo]

Interesting idea. Not sure if that's easy or messy to add. I will look into it when I find the time.

[EdJoPaTo]

The joinLastRow parameter is used on startup to create the keyboard structure. It is not used afterwards. Which means… refactoring that will be messy / not as simple.

https://github.com/EdJoPaTo/telegraf-inline-menu/blob/ff9823cac16dd646e412669ba3aa28a4d3203bf1/source/keyboard.ts#L37-L46

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

how about introducing a new function

 menu.joinLastRow(ctx => true)

[EdJoPaTo]

The issue isn't adding a new function. It could be integrated in the current one. The refactoring is needed to change the current internal structure as the current menu row layout is set up on startup and not when rendering it. So a new function would help for this case.

When anyone has the time to do the refactoring I'll be happy to guide the PR and merge it. But personally I currently don't have the time to work on it myself.

@EdJoPaTo thanks for clarification I got it now
there is a workaround he can do is to create two buttons and hide the one that is not needed 😄