https://bugs.kde.org/show_bug.cgi?id=443168

--- Comment #27 from Oded Arbel <o...@geek.co.il> ---
(In reply to Robin Bankhead from comment #26)
> Based on Andrew's initial post it seems clear that Gwenview knows nothing of
> .directory files, not even through Dolphin: It simply copies Dolphin's
> *global* sort setting.

That is incorrect: Gwenview follows Dolphin's per-directory sort setting, which
is stored in `.directory` files under the section name `[Dolphin]`. You can see
that by having two folders with images, then using Dolphin - set each to
different sorting (lets say, one by date and one by name), then launch Gwenview
with first the one folder as the argument to gwenview, then the other - you'd
see that Gwenview tracks the per-directory sort mode set by Dolphin.

You can also run gwenview under strace and see that it reads the `.directory`
file in the folder where you've opened it to (with either a folder argument or
a file argument). There's no additional communications between Dolphin and
Gwenview.

> Any solution to this particular bug needs to address how Gwenview behaves
> when there are no .directory files involved.

That is only workable if you also remove Gwenview's feature to read
`.directory` files. I'm personally OK with that, but I'm guessing that feature
exists for a reason.

> 2) Dolphin should invoke Gwenview with the switches to mirror its own
> sort-mode, or not, as a configuration option (which option is the default:
> TBD)

I believe that is a non-starter: Dolphin does not integrate with Gwenview -
doing any kind of deep integration with Gwenview (knowing about specific
arguments needed to launch specifically Gwenview) goes in direct contrast to
how a modular and generic system should work. If someone wants to implement
another image viewer that can compete with Gwenview on features - including
tracking the sorting order in file managers, now they can't do that. I don't
think KDE devs would go for that and I don't think they should.

> Alternatively, if it's considered undesirable for the two apps to be
> codependent in this way for the sake of this one feature, let Gwenview
> ignore Dolphin entirely.

I'm willing to put up the MR to remove Gwenview's `.directory` parsing feature
- but only if maintainers say they will accept this behavior change. As far as
I understand, this behavior is intentional and there's no point in me putting
the work in if no one is going to even look at the code.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to