Reviewing merge request #1673: Added QToolBar::visibilityChanged(bool) signal

250728 - It would be nice if we have signal "visibilityChanged()" in QToolBar.
http://qt.nokia.com/developer/task-tracker/index_html?method=entry&id=250728

Commits that would be merged:

Version 1
  • Version 1
  • badcaa7
  • b19260e
  • Added QToolBar::visibilityChanged(bool) signal.

Showing badcaa7-b19260e

Comments

hmm
what wrong with eventFilter-s?

Sure. You can do it by installing an event filter, but that’s not very convenient, is it? Besides, QDockWidget already offers visibilityChanged() signal so I'd like QToolBar to offer the same thing for consistency.

→ State changed from New to Merged

Add a new comment:

Login or create an account to post a comment

How to apply this merge request to your repository