D11245: infopanel: Add choice of date display formats
Mark Gaiser
noreply at phabricator.kde.org
Wed Mar 14 22:28:48 GMT 2018
markg added a comment.
In D11245#226020 <https://phabricator.kde.org/D11245#226020>, @ngraham wrote:
> In D11245#225997 <https://phabricator.kde.org/D11245#225997>, @markg wrote:
>
> > In D11245#224414 <https://phabricator.kde.org/D11245#224414>, @ngraham wrote:
> >
> > > I think having it in the context menu makes enough sense; the toggle for the preview is already there, so there's already precedent for having the Information Panel's UI be controlled via context menu. +1.
> >
> >
> > @broulik and I seem to think that it should be in the settings, not under the RMB.
> > To be fair, i'm trusting your judgement so i'm fine either way, but i'm "leaning" towards in the settings as well.
>
>
> If we go down that path, I would argue that it makes sense to relocate the `[ ] preview` checkbox that's currently in the context menu to that window as well for consistency's sake. I'd be fine with it as long as we do that to maintain consistency.
That would be my preference.
REPOSITORY
R318 Dolphin
REVISION DETAIL
https://phabricator.kde.org/D11245
To: michaelh, #dolphin, markg
Cc: broulik, markg, ngraham, spoorun, navarromorales, isidorov, firef, andrebarros, alexeymin, genaxxx, emmanuelp
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20180314/9d7d1d6d/attachment.htm>
More information about the kfm-devel
mailing list