'Re: [KDE Usability] Review Request 123658: If double-click is on empty area, then invoke back.'

[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-usability
Subject:    Re: [KDE Usability] Review Request 123658: If double-click is on empty area, then invoke back.
From:       "Bjoern Balazs" <b () lazs ! de>
Date:       2015-05-06 11:43:22
Message-ID: 20150506114322.16998.99110 () mimi ! kde ! org
[Download RAW message or body]

--===============6126764327835807865==
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

> On Mai 6, 2015, 10:45 vorm., Kai Uwe Broulik wrote:
> > -1
> > 
> > Please also add usability group for such reviews. I have never seen an \
> > application that does this nor would I consider that intuitive behavior. Also, \
> > many mice have dedicated back/forward buttons you can use.
> 
> Martin Klapetek wrote:
> I'm afraid I have to agree.
> 
> What's your rationale behind adding this feature?
> 
> Ashish Bansal wrote:
> According to me, this improves the flow of the application as well as user \
> experience. I can just double click on the empty area inside that folder and I'm in \
> its parent directory. I don't have to move my mouse to upper-left corner or press \
> keyboard key to go back. That's it :) 
> Although this feature is not available in many applications, I only have used \
> "Clover file manager" earlier in ms-windows which used to have this feature. \
> Yesterday while checking bug list, I found one feature request regarding this \
> feature and then it came to my mind. I tried to find that bug but I am unable to \
> find it currently.

I do not know why we shouldn't allow this behaviour. It is not the only way to \
navigate, so discoverability or what users are used to is not that important. I \
actually find the behaviour quite intuitive (as soon as you learned it exists) - so I \
wouldn't see a Usability argument to say no to this patch.

- Bjoern

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123658/#review79951
-----------------------------------------------------------

On Mai 6, 2015, 10:53 vorm., Ashish Bansal wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123658/
> -----------------------------------------------------------
> 
> (Updated Mai 6, 2015, 10:53 vorm.)
> 
> 
> Review request for Dolphin, KDE Usability and Emmanuel Pescosta.
> 
> 
> Repository: dolphin
> 
> 
> Description
> -------
> 
> If mouse double click is done on the empty area, then invoke the go back to \
> previous directory in the history. 
> 
> Diffs
> -----
> 
> src/kitemviews/kitemlistcontroller.h b8a93ed 
> src/kitemviews/kitemlistcontroller.cpp 1c86ff0 
> src/views/dolphinview.cpp d302358 
> 
> Diff: https://git.reviewboard.kde.org/r/123658/diff/
> 
> 
> Testing
> -------
> 
> Doesn't works sometimes.
> 
> 
> Thanks,
> 
> Ashish Bansal
> 
> 

--===============6126764327835807865==
MIME-Version: 1.0
Content-Type: text/html; charset="utf-8"
Content-Transfer-Encoding: 7bit

<html>
 <body>
  <div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
   <table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 \
solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">  \
<tr>  <td>
      This is an automatically generated e-mail. To reply, visit:
      <a href="https://git.reviewboard.kde.org/r/123658/">https://git.reviewboard.kde.org/r/123658/</a>
  </td>
    </tr>
   </table>
   <br />

<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: \
10px;">  <p style="margin-top: 0;">On Mai 6th, 2015, 10:45 vorm. UTC, <b>Kai Uwe \
Broulik</b> wrote:</p>  <blockquote style="margin-left: 1em; border-left: 2px solid \
#d0d0d0; padding-left: 10px;">  <pre style="white-space: pre-wrap; white-space: \
-moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: \
break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: \
inherit;white-space: inherit;">-1</p> <p style="padding: 0;text-rendering: \
inherit;margin: 0;line-height: inherit;white-space: inherit;">Please also add \
usability group for such reviews. I have never seen an application that does this nor \
would I consider that intuitive behavior. Also, many mice have dedicated back/forward \
buttons you can use.</p></pre>  </blockquote>

 <p>On Mai 6th, 2015, 11:03 vorm. UTC, <b>Martin Klapetek</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: \
10px;">  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: \
-pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: \
0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I'm \
afraid I have to agree.</p> <p style="padding: 0;text-rendering: inherit;margin: \
0;line-height: inherit;white-space: inherit;">What's your rationale behind adding \
this feature?</p></pre>  </blockquote>

 <p>On Mai 6th, 2015, 11:14 vorm. UTC, <b>Ashish Bansal</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: \
10px;">  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: \
-pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: \
0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: \
inherit;">According to me, this improves the flow of the application as well as user \
experience. I can just double click on the empty area inside that folder and I'm in \
its parent directory. I don't have to move my mouse to upper-left corner or press \
keyboard key to go back. That's it :)</p> <p style="padding: 0;text-rendering: \
inherit;margin: 0;line-height: inherit;white-space: inherit;">Although this feature \
is not available in many applications, I only have used "Clover file manager" earlier \
in ms-windows which used to have this feature. Yesterday while checking bug list, I \
found one feature request regarding this feature and then it came to my mind. I tried \
to find that bug but I am unable to find it currently.</p></pre>  </blockquote>

</blockquote>

<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: \
-pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: \
0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I do \
not know why we shouldn't allow this behaviour. It is not the only way to navigate, \
so discoverability or what users are used to is not that important. I actually find \
the behaviour quite intuitive (as soon as you learned it exists) - so I wouldn't see \
a Usability argument to say no to this patch.</p></pre> <br />

<p>- Bjoern</p>

<br />
<p>On Mai 6th, 2015, 10:53 vorm. UTC, Ashish Bansal wrote:</p>

<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: \
1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; \
-webkit-border-radius: 6px;">  <tr>
  <td>

<div>Review request for Dolphin, KDE Usability and Emmanuel Pescosta.</div>
<div>By Ashish Bansal.</div>

<p style="color: grey;"><i>Updated Mai 6, 2015, 10:53 vorm.</i></p>

<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt;">Repository: </b>
dolphin
</div>

<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
 <table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" \
style="border: 1px solid #b8b5a0">  <tr>
  <td>
   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: \
-moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: \
break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: \
inherit;white-space: inherit;">If mouse double click is done on the empty area, then \
invoke the go back to previous directory in the history.</p></pre>  </td>
 </tr>
</table>

<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: \
1px solid #b8b5a0">  <tr>
  <td>
   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: \
-moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: \
break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: \
inherit;white-space: inherit;">Doesn't works sometimes.</p></pre>  </td>
 </tr>
</table>

<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">

 <li>src/kitemviews/kitemlistcontroller.h <span style="color: \
grey">(b8a93ed)</span></li>

 <li>src/kitemviews/kitemlistcontroller.cpp <span style="color: \
grey">(1c86ff0)</span></li>

 <li>src/views/dolphinview.cpp <span style="color: grey">(d302358)</span></li>

</ul>

<p><a href="https://git.reviewboard.kde.org/r/123658/diff/" style="margin-left: \
3em;">View Diff</a></p>

  </td>
 </tr>
</table>

  </div>
 </body>
</html>

--===============6126764327835807865==--

[Attachment #3 (text/plain)]

_______________________________________________
kde-usability mailing list
kde-usability@kde.org
https://mail.kde.org/mailman/listinfo/kde-usability

[prev in list] [next in list] [prev in thread] [next in thread] 