D12327: Show Detailed Tree View by default

2019-03-26 Thread Noah Davis
ndavis added a comment. +1 Navigation is so much faster in tree view. I think navigation speed is what matters most of the time when people are selecting files or locations. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D12327 To: ngraham, #frameworks, #vdg, apol

D12327: Show Detailed Tree View by default

2019-03-26 Thread Nathaniel Graham
ngraham added a reviewer: apol. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D12327 To: ngraham, #frameworks, #vdg, apol Cc: kde-frameworks-devel, elvisangelaccio, abetts, #frameworks, michaelh, ngraham, bruns

D12327: Show Detailed Tree View by default

2019-03-26 Thread Nathaniel Graham
ngraham removed a dependency: D12326: In Short View, display icons on top and increase icon size. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D12327 To: ngraham, #frameworks, #vdg Cc: kde-frameworks-devel, elvisangelaccio, abetts, #frameworks, michaelh, ngraham, bruns

D12327: Show Detailed Tree View by default

2019-03-26 Thread Nathaniel Graham
ngraham edited the summary of this revision. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D12327 To: ngraham, #frameworks, #vdg Cc: kde-frameworks-devel, elvisangelaccio, abetts, #frameworks, michaelh, ngraham, bruns

D12327: Show Detailed Tree View by default

2019-03-19 Thread Nathaniel Graham
ngraham updated this revision to Diff 54303. ngraham added a comment. Rebase (I still think this makes sense) REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D12327?vs=32511&id=54303 BRANCH arcpatch-D12327 REVISION DETAIL https://phabricator.kde.org/D12327

D12327: Show Detailed Tree View by default

2018-08-24 Thread Henrik Fehlauer
rkflx resigned from this revision. Herald added a subscriber: kde-frameworks-devel. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D12327 To: ngraham, #frameworks, #vdg Cc: kde-frameworks-devel, elvisangelaccio, abetts, #frameworks, michaelh, ngraham, bruns

D12327: Show Detailed Tree View by default

2018-04-29 Thread Nathaniel Graham
ngraham added a comment. In D12327#255621 , @elvisangelaccio wrote: > FWIW I also disagree with D12077 . I don't see why we should show only two buttons (for short and detailed tree view), when we have four v

D12327: Show Detailed Tree View by default

2018-04-29 Thread Elvis Angelaccio
elvisangelaccio added a comment. In D12327#250799 , @rkflx wrote: > In D12327#250797 , @elvisangelaccio wrote: > > > But Detailed View and Detailed Tree View are two different things, no? (I'm not e

D12327: Show Detailed Tree View by default

2018-04-21 Thread Nathaniel Graham
ngraham added a comment. In D12327#250944 , @rkflx wrote: > In D12327#250911 , @ngraham wrote: > > > In fact, if we integrated the Up button into the `KURLNavigator` widget, it would solve some issu

D12327: Show Detailed Tree View by default

2018-04-21 Thread Henrik Fehlauer
rkflx added a comment. In D12327#250911 , @ngraham wrote: > In fact, if we integrated the Up button into the `KURLNavigator` widget, it would solve some issues automatically (e.g. lack of up button in Gwenview). I could see some people complaini

D12327: Show Detailed Tree View by default

2018-04-21 Thread Nathaniel Graham
ngraham added a comment. In D12327#250898 , @rkflx wrote: > In D12327#250864 , @ngraham wrote: > > > - Bandwagon: Windows and macOS's default details style views allow expansion by default too >

D12327: Show Detailed Tree View by default

2018-04-21 Thread Henrik Fehlauer
rkflx added a comment. In D12327#250864 , @ngraham wrote: > - Bandwagon: Windows and macOS's default details style views allow expansion by default too I'm all for going tree-style, but honestly I don't see expansion in Windows? Are we t

D12327: Show Detailed Tree View by default

2018-04-21 Thread Nathaniel Graham
ngraham added a comment. Or, a compromise: If we wanted to go all the way and have Dolphin-style options (Icons View, Compact view, Detailed view), I could support that for consistency's sake as long as we have three buttons like Dolphin does, rather than a combobox or a drop-down menu. That

D12327: Show Detailed Tree View by default

2018-04-21 Thread Nathaniel Graham
ngraham added a comment. Reasons to change the default to Detailed View or Detailed Tree View: - Ergonomics: the current default requires side-scrolling for long lists, which is almost never ideal - Usefulness: by showing the details columns by default (which also makes sorting more di

D12327: Show Detailed Tree View by default

2018-04-21 Thread Henrik Fehlauer
rkflx added a comment. One more idea: We could go all the way to a Windows style file dialog, i.e. use a single button to switch views, which has //all// options in it: - Short View > Next to Filename (needs new name) - Short View > Above to Filename (needs new name) - Detailed View

D12327: Show Detailed Tree View by default

2018-04-21 Thread Henrik Fehlauer
rkflx added a comment. In D12327#250797 , @elvisangelaccio wrote: > But Detailed View and Detailed Tree View are two different things, no? (I'm not even sure Windows supports the latter) That's correct, Windows does not support the latte

D12327: Show Detailed Tree View by default

2018-04-21 Thread Elvis Angelaccio
elvisangelaccio added a comment. In D12327#250786 , @rkflx wrote: > In D12327#250761 , @elvisangelaccio wrote: > > > I'm not sure I agree with this change > > > At least it's not a -1 yet, so m

D12327: Show Detailed Tree View by default

2018-04-21 Thread Henrik Fehlauer
rkflx added a comment. In D12327#250761 , @elvisangelaccio wrote: > I'm not sure I agree with this change At least it's not a -1 yet, so maybe we can convince you once everything is in place…!? > afaik Windows does not show the tree

D12327: Show Detailed Tree View by default

2018-04-21 Thread Elvis Angelaccio
elvisangelaccio added a comment. I'm not sure I agree with this change, and afaik Windows does not show the tree view by default in its file dialogs (unless you are referring to `FolderBrowserDialog` from .Net) REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D12327 To:

D12327: Show Detailed Tree View by default

2018-04-19 Thread Nathaniel Graham
ngraham retitled this revision from "[WIP/open dependencies] Show Detailed Tree View by default" to "Show Detailed Tree View by default". ngraham edited the summary of this revision. ngraham edited the test plan for this revision. ngraham added reviewers: Frameworks, VDG, rkflx. ngraham added a de