From: Andrej N. G. <an...@re...> - 2012-09-17 18:22:35
|
Hello! PCMan has written on Sunday, 16 September, at 21:03: >On Sun, Sep 16, 2012 at 6:18 PM, Andrej N. Gritsenko <an...@re...> wrote: >> PCMan has written on Sunday, 16 September, at 16:04: >>>> 2. The search result should always be shown in "detailed list" mode >>>> regardless of the default view mode set in config. >> I don't like the idea. Some may want to see search results with icon >> view for example. It should be done via 'per-folder view mode' setting >> which was discussed a bit already, BTW. >Adding many items to the icon view induced more UI flickering sometimes >since this caused relayout of a lot of items. >In detailed list mode, this also cause insertion of items and changed >the layout, but the performance hit and flickering caused is less when >compared to icon view. >Besides, the most important issue is for two files in different >folders with the same time, it's not able to differentiate them in >icon view mode. >In detailed list view mode, we can add a column for showing their locations. >For usability and performance perspective, whether this is >customizable or not, the search result should use detailed list mode >"by default". >Of course we can make this customizable and let the user change it, >but detailed list should be the default. That's OK for me but we should hardcode that behavior before we can make it customizable which isn't nice you know. But I agree, we can do it as temporary workaround. I would like you to test how "internal" VFS "example" for menu:// works so we can eliminate hardcodes for search:// too and migrate all its engine into VFS instead - that's not hard to do but will simplify things a lot. I think the sequence of branches merge should be now: 1) 1.0.1 is released; 2) two days timeout; 3) '1.0.2' rebased+merged into 'master', tagged 1.0.2~alpha0; 4) 'fm-file' rebased+merged into 'master'; 5) 'search' rebased+merged into 'master'; 6) FmSearchJob migrated to VFS; 7) search GUI moved from pcmanfm into libfm, tagged 1.0.2~alpha1; 8) let users test everything. With best regards. Andriy. |