Where is addToolbarPopupMenuOption?

pfaffman · March 20, 2024, 4:49pm

I was using addToolbarPopupMenuOptionsCallback and got a notice it had been renamed to addToolbarPopupMenuOption , but when I did I call addToolbarPopupMenuOption I get a error that it does not exist.


  addToolbarPopupMenuOptionsCallback(opts) {
    deprecated(
      "`addToolbarPopupMenuOptionsCallback` has been renamed to `addToolbarPopupMenuOption`",
      {
        id: "discourse.add-toolbar-popup-menu-options-callback",
        since: "3.2",
        dropFrom: "3.3",
      }
    );

In GitHub - literatecomputing/discourse-qrcode-theme-component: Add QRcode to composer

david · March 20, 2024, 5:03pm

Looks like we have a typo in the deprecation message. Fix here: DEV: Fix toolbar API deprecation message by davidtaylorhq · Pull Request #26266 · discourse/discourse · GitHub

This is the one you want:

github.com/discourse/discourse

app/assets/javascripts/discourse/app/lib/plugin-api.gjs

e3cfb1967


      
          /**
           * Add a new button in the composer's toolbar options popup menu.
           *
           * @callback action
           * @param {Object} toolbarEvent - A toolbar event object.
           * @param {function} toolbarEvent.applySurround - Surrounds the selected text with the given text.
           * @param {function} toolbarEvent.addText - Append the given text to the selected text in the composer.
           *
           * @callback condition
           * @param {Object} composer - The composer service object.
           * @returns {boolean} - Whether the button should be displayed.
           *
           * @param {Object} opts - An Object.
           * @param {string} opts.icon - The name of the FontAwesome icon to display for the button.
           * @param {string} opts.label - The I18n translation key for the button's label.
           * @param {action} opts.action - The action to perform when the button is clicked.
           * @param {condition} opts.condition - A condition that must be met for the button to be displayed.
           *
           * @example
           * api.addComposerToolbarPopupMenuOption({

This file has been truncated. show original

Don · March 20, 2024, 5:06pm

I used addComposerToolbarPopupMenuOption when I changed it in Composer Footnote Button.

pfaffman · March 20, 2024, 5:47pm

Aha! So, it’s not just me. That’s great news, as I’ve spent most of the day trying to figure out things that are, in fact, my fault (but might still stand to have some better documentation). I’ll work on asking cogent questions about those things next.

This is resolved. Thanks so much!

Topic		Replies	Views
Depreciation notice in Spoiler Alert Support spoiler-alert	2	285	October 18, 2023
How to add my own drop down(popupMenu) like the composer options? Development composer	4	749	July 27, 2023
Add the ability to create your own dropdown menu and add custom items in the composer Feature composer , plugin-api	0	433	July 27, 2023
api.registerValueTransformer: transformer "post-menu-buttons" is unknown and will be ignored Support plugin-api	1	49	January 4, 2025
Add a button to the composer tools and change the order of buttons in Tools Support	2	571	December 12, 2021

Where is addToolbarPopupMenuOption?

Related topics