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

Information

pidgin/pidgin
default

Reviewers

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 ID
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.
93712eded6f64783f8ae69344718dde4b2006232
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)
     
     
     
     
    Show all issues

    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)
     
     
    Show all issues

    Remove the whole <child>?

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