Remove the Pidgin icon themes which are covered by XDG icon themes now and remove the stock api as that's all that was using it yet.

Review Request #1088 — Created Oct. 21, 2021 and submitted

grim
pidgin/pidgin
default
pidgin
Remove the Pidgin icon themes which are covered by XDG icon themes now and remove the stock api as that's all that was using it yet.
  • Built everything, including the docs and pidgin-pot
  • Verified that the preferences window still functions properly
  • Verified that the user list in the conversation window had properly sized icons
  • Verified all of the status icons were not affected.
Summary
Remove the Pidgin icon themes which are covered by XDG icon themes now and remove the stock api as that's all that was using it yet.
Description From Last Updated

I left this for the size, so wondering what sets the size now?

QuLogicQuLogic

Remove the whole <child>?

QuLogicQuLogic
grim
grim
  1. 
      
  2. just wanted to add a side note here (mostly for myself) that purple_mkstemp can be removed after this lands.

  3. 
      
QuLogic
  1. 
      
  2. pidgin/gtkconv.c (Diff revision 1)
     
     
     
     

    I left this for the size, so wondering what sets the size now?

    1. nothing, but in my testing the default chosen size was fine.

  3. pidgin/resources/Prefs/prefs.ui (Diff revision 1)
     
     

    Remove the whole <child>?

  4. 
      
grim
QuLogic
  1. Ship It!
  2. 
      
grim
Review request changed

Status: Closed (submitted)

Loading...