kossebau added a comment.

  When going to adapt some code to follow the deprecation, I now look though at 
some inconsistency in the API:
  
  - KTitleWidget has a property "pixmap"
  - the `setPixmap` methods all set this pixmap, with overloads for convenience
  - `setIcon` is without a getter and a documented property, the relation to 
the pixmap property is unclear just by the API names
  
  With this in mind, I fear I consider this change a regression in API quality. 
I'd propose to revert this, or go the full mile and make "icon" a real 
property, deprecating the pixmap one. What do you think?

REPOSITORY
  R236 KWidgetsAddons

REVISION DETAIL
  https://phabricator.kde.org/D23927

To: vkrause, dfaure
Cc: kossebau, dhaumann, kde-frameworks-devel, LeGast00n, GB_2, michaelh, 
ngraham, bruns

Reply via email to