Great enhancements, but I suggest to make the initialization simpler and easier to understand:

   window.show(argc,argv);
+  choice.do_callback();         // make default take effect
   return Fl::run();

Replace choice.do_callback(); with window->icon(0); or

  window->icon((Fl_RGB_Image*)0);          // reset window icon

(if the cast is necessary). OTOH "no icon" would be the default anyway, so why bother and set "no icon" explicitly?

You could also mention that the icon used in Alt+Tab is changed as well.

Another info (for the docs) would be that some window managers don't set a window (i.e. title bar) icon, notably Cinnamon (which I am using), although the taskbar and Alt+Tab icon is set.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.