Ticket #8927 (closed bug: fixed)

Opened 23 months ago

Last modified 22 months ago

Menu: Can't change icons option after init

Reported by: scott.gonzalez Owned by:
Priority: minor Milestone: 1.10.0
Component: ui.menu Version: 1.9.2
Keywords: Cc:
Blocking: Blocked by:

Description


Change History

comment:1 Changed 23 months ago by scott.gonzalez

  • Status changed from new to open

comment:2 Changed 22 months ago by joern.zaefferer

The "menus" and "role" options can't be changed after init either. "position" should be fine as that applies the next time a submenu is opened, unless the expectation would be to reposition an open submenu when the position option is updated. Is it? I'd like confirmation on that (and new tickets, if necessary) before addressing this one, since there's no _setOption in menu, yet. Would handle them all at once.

comment:3 Changed 22 months ago by scott.gonzalez

I think we need to start making explicit exceptions for certain options being able to be changed after init.

menus: I don't think we should let users change options that deal with how the widget's structure is created. This is similar to accordion's header option.

role: I know some users re-use dialogs, so I could see someone wanting to change the role on a dialog (if we ever let that be customizable so you can have an alertdialog). I'm inclined to say that we don't allow changing this, and wait for good use cases to come in (if there are any).

position: We do re-position dialogs if they're open, but I think that's a pretty different situation from an open menu or open autocomplete. I'd say that changing the position while the menu is open shouldn't do anything until the next submenu is opened.

So, unless there are objections, I'd say icons is the only option to be handled.

comment:4 Changed 22 months ago by Jörn Zaefferer

  • Status changed from open to closed
  • Resolution set to fixed

Menu: Allow changing icons option after creation. Fixes #8927 - Menu: Can't change icons option after init

Changeset: 2c3d311f90281e95827708e2e8d0e52832a437de

Note: See TracTickets for help on using tickets.