scripts icon indicating copy to clipboard operation
scripts copied to clipboard

urlselect.lua - closing with script active causes useless bar to show on restart

Open rdrms opened this issue 2 years ago • 3 comments

Hi, I really like this script, but whenever I leave the url selector open and end up exiting out of weechat, perhaps not cleanly through /exit but through logging out of the session or clicking the (X) on the terminal emulator, it doesn't restart properly (that is, with urlselect closed). The URLselect bar shows, but has no information. You cannot close it by hitting any of the key combinations that I have tried. The only way is when someone finally posts a url, you can restart the script by getting it to grab the URL, and then you can cancel it. I myself have resorted to going into a test room and logging on with webchat and posting a url there just so I can get the bar to go away.

Thanks for listening!

rdrms avatar Jan 13 '22 20:01 rdrms

You should probably point this out to the author of the script

https://github.com/tomoe-mami/weechat-scripts/issues

Nomarian avatar Jan 14 '22 17:01 Nomarian

I didn't see that repository, thank you for the hint.

On Fri., Jan. 14, 2022, 10:23 lan Nomar, @.***> wrote:

You should probably point this out to the author of the script

https://github.com/tomoe-mami/weechat-scripts/issues

— Reply to this email directly, view it on GitHub https://github.com/weechat/scripts/issues/490#issuecomment-1013313760, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAWM6PRKC7YLF24QSTMG6FDUWBL23ANCNFSM5L44LCMQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you authored the thread.Message ID: @.***>

rdrms avatar Jan 14 '22 18:01 rdrms

after looking at the script, perhaps you should run cmd_action_deactivate on a loop on all buffers before setup() ? that's the only thing that occurs to me at a quick glance.

Nomarian avatar Jan 14 '22 18:01 Nomarian