Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MenuEdit: Do not use autoconnection mechanism for slots used in QMenu #132

Merged
merged 2 commits into from
Nov 15, 2023

Commits on Nov 15, 2023

  1. MainWindow: Do not use autoconnection mechanism for slots used in QMenu

    Currently whenever UI gets loaded and we try to connect a slot with a
    signal emitted in QAction, for example in appending frame, inserting it,
    etc. we get warnings like these:
    
    qt.core.qmetaobject.connectslotsbyname: QMetaObject::connectSlotsByName: No matching signal for on_actionAdd_Frame_triggered()
    
    This is because MOC tries to automatically connect functions which have
    "on_objectname_triggered" signature, and it can't find a matching signal
    for it by default. So the solution for that is to simply not use the
    autoconnection mechanism by changing names of those slots, removing
    "on" prefix from them.
    tetektoza committed Nov 15, 2023
    Configuration menu
    Copy the full SHA
    46d4390 View commit details
    Browse the repository at this point in the history
  2. PaletteWidget: Do not use autoconnection mechanism for slots used in

    QMenu
    
    Currently whenever UI gets loaded and we try to connect a slot with a
    signal emitted in QAction, for example in appending frame, inserting it,
    etc. we get warnings like these:
    
    qt.core.qmetaobject.connectslotsbyname: QMetaObject::connectSlotsByName: No matching signal for on_actionAdd_Frame_triggered()
    
    This is because MOC tries to automatically connect functions which have
    "on_objectname_triggered" signature, and it can't find a matching signal
    for it by default. So the solution for that is to simply not use the
    autoconnection mechanism by changing names of those slots, removing
    "on" prefix from them.
    tetektoza committed Nov 15, 2023
    Configuration menu
    Copy the full SHA
    30337b2 View commit details
    Browse the repository at this point in the history