WindowIdentifier::from_native() may only be called once for Wayland GTK windows about ashpd

[bilelmoussaoui]

Yes that's expected because the handle is unexported when the WindowIdentifier is dropped.

I wonder what would be the best way forward here, but addikg a comment at least seems fine to me

[Cogitri]

Alright, I’ll submit a PR later today.

Another question: To work around this I currently just hold onto the WindowIndentifier in a OnceCell that’s initialised when the window is created. Is that fine or should I always re-request the WindowIdentifier?

[bilelmoussaoui]

I would say you can keep only once instance around by wrapping it in a OnceCell. I don't know if we can do something about it on the crate side?

[Cogitri]

Hm, I don’t really think we can other than hacky ways like having a global HashMap<WeakRef, WindowIdentifier> to remember the WindowIdentifier

[bilelmoussaoui]

Ah yeah indeed. That would be very painful to do. I guess going with a comment would be good enough for now.

Feel free to open another issue regarding the usage of the Default implementation of WindowIdentifier in some of the portals

[A6GibKm]

Cant we do a static Once that holds the identifier?

[bilelmoussaoui]

You can, from the application side