lxqt / xdg-desktop-portal-lxqt

A backend implementation for xdg-desktop-portal
GNU Lesser General Public License v2.1
22 stars 9 forks source link

Portal open/save paths broken at start #36

Closed stefonarch closed 6 months ago

stefonarch commented 6 months ago

It happened today after an update, no idea what component could be involved, it was mostly python. Broken both in firefox and thunderbird, while element (xwayland) doesn't use it anymore.

immagine

immagine

Filename is also empty if trying to save something.

tsujan commented 6 months ago

It happened to me a few weeks ago, when I tried the portal for the fist time since its creation. The problem is in that %00, which I don't know where comes from. Here the problem disappeared after I removed some redundant keys about "dir" and "down" (those with trash icons beside them) from about:config, but it showed up again after a short time.

I also remember that there was no problem in Chromium (I uninstalled the portal after checking it).

tsujan commented 6 months ago

Portal open/save paths broken

It isn't broken. The problem is that a wrong path is given to it at startup; otherwise, it can be used by changing directory.

@stefonarch, if you see a similar thing, please change the title of this report.

tsujan commented 6 months ago

OK, a patch is needed. KDE has added decodeFileName(), which should handle this situation. Will make a PR after some tests — by you ;)