[Konsole-devel] Review Request: Disable the feature of "shortening common dirnames" for %d by deafult

[Jekyll Wu]

> On Feb. 14, 2012, 3 p.m., Kurt Hindenburg wrote:
> > Since the default is to not use %d, the user would have to select it if they want.  I'm more inclined to live the code as-is.
> > 
> > Perhaps give some documentation about changing the defaults in the rc file would be better than nothing.
> 
> Jekyll Wu wrote:
>     Well, I think that "%D" being default is just another hack/workaround (like this one) to reduce the impact of the reported problem. 
>     
>     According to the git history, "%d" has been the default until the recent commit in the bug report. I think choosing "%d" as default is a good idea, because "%D" can be easily expanded to quite long. So even we now provide "%D" as default, I doubt users will use that default happily. They will probably try "%d", and find the reported problem quickly. So I think that workaround is not that useful.
>     
>     The ideal solution is to provide gui elements for controlling this feature. But I doubt the effort will pay off for this feature, or the priority is high enough. So before we finally provide a real solution, I propose to use another workaround than the current workaround. That is :
>     
>         * use "%d" as default
>         * disable that feature by default by settings the list as empty in the code
>         * add good documentation on how to enable that feature by editing konsolerc manually
>     
>     Yes, the proposed workaround means some (power) users who enjoy this feature might find this feature is gone in surprise. But I do believe they are the minority and are capable of finding how to get it back through reading documentation.
>     
>
> 
> Kurt Hindenburg wrote:
>     I'm not 100% that your propose is a good idea.  If we implement it, on updating konsole, can we check the users profiles and if they are using %d manually add current list to the CommonDirNames?  If not, all the users using %d will be confused on what happened.
>     
>     On a side note, I'm not against adding GUI for configurations options.  Currently, the config dialogs are not very well organized.  It would be better now that we're splitting the konsole and profile settings.  Both the 'manager profile' and 'configure profile' dialogs need a revamp.

"can we check the users profiles and if they are using %d manually add current list to the CommonDirNames?"

That should be possible. Writing another migration script based upon the kconf-update framework should not be a hard work.


- Jekyll


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103949/#review10631
-----------------------------------------------------------


On Feb. 12, 2012, 8:59 a.m., Jekyll Wu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103949/
> -----------------------------------------------------------
> 
> (Updated Feb. 12, 2012, 8:59 a.m.)
> 
> 
> Review request for Konsole.
> 
> 
> Description
> -------
> 
> The attached patch is only a hack, not a real solution.
> 
> I would like to discuss something:
> 
> 1). In principle, shall we provide GUI for each effective option? Is it OK to have some hidden options which are only configurable through manual editing, when such options are rarely used or hard to understand for ordinary users? Of course, we must provide clear document for those hidden options in that case.
> 
> 2). For this specific option, it currently always reads konsolerc, which is not quite good for konsolepart.
> 
> 
> This addresses bug 190281.
>     http://bugs.kde.org/show_bug.cgi?id=190281
> 
> 
> Diffs
> -----
> 
>   src/ProcessInfo.cpp 92e3eca 
> 
> Diff: http://git.reviewboard.kde.org/r/103949/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Jekyll Wu
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20120219/2c43ef80/attachment.html>