Set the Window Icon

[slime73]

Original report by Sean Donno (Bitbucket: [Sean Donnellan](https://bitbucket.org/Sean Donnellan), ).

---

This is a feature request for a way to set the icon on the window.

I propose love.graphics.setIcon( image )

Like SDL_WM_SetIcon in SDL.

Reason for it is makes identifying your LOVE game in Task Switcher (Alt+Tab) easier and nicer.

[slime73]

Original comment by Robin Wellner (Bitbucket: gvx, GitHub: gvx).

---

Alternatively or additionally, perhaps a setting in conf.lua?

[slime73]

Original comment by Bart van Strien (Bitbucket: bartbes, GitHub: bartbes).

---

@robin: in any case additionally.

Yeah, seems like a nice feature, if it's SDL it can't be too hard (can it?)

[slime73]

Original comment by Bill Meltsner (Bitbucket: bmelts, GitHub: bmelts).

---

It's pretty simple: get the raw pixel data of an Image, create an SDL_Surface with it, and pass that to SDL_WM_SetIcon.

I'll take this one.

[slime73]

Original comment by Sean Donno (Bitbucket: [Sean Donnellan](https://bitbucket.org/Sean Donnellan), ).

---

I managed to get some time today to have a look at trying to do it myself.

I was however not able to get it fully working, ran into some problems with the typing.
Incorrect parameter type: expected userdata.

I am sure that Creating a surface, setting an icon and then freeing the surface again is fine as the SetIcon will copy the image data, meaning it is safe to free as it is not being used.
For X11, the SetIcon function will XCreatePixmap and so forth thus duplication image data and the data is copied acorss. Similarly Windows does a similar thing CreateIconFromResourceEx, so that data stored b in the surface is not accessed by the WM after the SetIcon function call.

Sorry, for not being able to produce a fully working patch, but at the least I hope this can help as a base.

[slime73]

Original comment by Bill Meltsner (Bitbucket: bmelts, GitHub: bmelts).

---

Implemented in <>.

[slime73]

Original comment by Robin Wellner (Bitbucket: gvx, GitHub: gvx).

---

Not a setting in conf.lua yet, is that on purpose?