ActionItem.triggered() has a caller parameter that is not needed.

Bug #1202668 reported by Tim Peeters
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu UI Toolkit
Invalid
Low
Tim Peeters

Bug Description

We used to have the caller parameter because the button instance that opened a popover was created inside the toolbar and not available in the code where Action's onTriggered was defined. This is no longer the case with ActionItems and ToolbarButtons, so the caller parameter should be removed to avoid confusion.

Tim Peeters (tpeeters)
Changed in ubuntu-ui-toolkit:
assignee: nobody → Tim Peeters (tpeeters)
Tim Peeters (tpeeters)
Changed in ubuntu-ui-toolkit:
importance: Undecided → High
Tim Peeters (tpeeters)
Changed in ubuntu-ui-toolkit:
status: New → Confirmed
summary: - ToolbarButton.triggered() has a caller parameter that is not needed.
+ ActionItem.triggered() has a caller parameter that is not needed.
Tim Peeters (tpeeters)
Changed in ubuntu-ui-toolkit:
importance: High → Low
Revision history for this message
Tim Peeters (tpeeters) wrote :

trigger(value) is some times used in the actions.

Changed in ubuntu-ui-toolkit:
status: Confirmed → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.