[prev in list] [next in list] [prev in thread] [next in thread]
List: kfm-devel
Subject: Re: Review Request: Proposal to add a 'lock toggle button' to filter bar to
From: "Stuart Citrin" <ctrn3e8 () gmail ! com>
Date: 2012-12-31 4:30:34
Message-ID: 20121231043034.1930.88474 () vidsolbach ! de
[Download RAW message or body]
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/107392/
-----------------------------------------------------------
(Updated Dec. 31, 2012, 4:30 a.m.)
Review request for Dolphin, Oxygen Transparent and KDE Usability.
Changes
-------
Usabliity and Oxygen people:
Please look at the two provided screen shots. "Filter bar with new text loc=
king button" uses a pin (really a thumbtack) icon while filterbarw/existing=
-lock-icon uses the existing system-lock-screen icon. Please comment on wh=
ether it is worth the effort to add a new 'pin' icon to oxygen for this pur=
pose or whether it would be better to use the existing system-lock-screen i=
con. Some pin icons are provided in file PinPngOxygenIcons.tar. Note, I a=
m no artist....I just found a public domain picture of a thumbtack, changed=
the color and Sinc interpolated the picture down to the icons. There is s=
ome aliasing(and maybe ringing) in the 16x16 icon...but it might be acccept=
able. =
(You may need to read the rest of the thread to figure out what all is goin=
g on...sorry)
Usability.... is it a big deal to have a button without a shortcut key....p=
lease comment. Again read the thread. Note, eliminating the shortcut key p=
rovides better code encapsculation and resistance to later bug 'regressions=
' and maintainance headaches.
(I couldn't find Nuno Pinheiro or Oxygen 'icon' in the people/groups...so I=
used Oxygen Transparent...if wrong please advise.)
Description
-------
I wrote a quick add-on for the Dolphin filter bar. It is a button which p=
revents the filter text from clearing when navigating to a different folder=
. =
Why this might be needed:
This is an attempt to resolve the following usability conflict: =
If one navigates to a different folder, the filter is automatically cleared=
since one does not want a situation where someone forgets a filter is appl=
ied and then doesn't understand why all the files within a folder are not a=
ppearing.(current behavior) =
versus: =
A user might want to keep a filter while navigating an arbitrary set of fol=
ders, but is unable to, since the filter is automatically cleared upon navi=
gation =
The implementation to try to resolve this conflict is as follows: =
1. A lock filter toggle button is added to the right side of the filter bar=
. =
2. If the lock filter button is clicked(on), then if the user navigates awa=
y from the current folder, the filter is not cleared. If the lock filter bu=
tton is released(off), then the text is cleared on navigation...which is th=
e current behavior. =
3. A configurable shortcut is added to toggle the lock filter button, with =
the default being Ctrl-Shift-I (since the default shortcut for hiding and s=
howing the filter bar is Ctrl-I.) =
4. If the user toggles the lock filter button to released (off), then the t=
ext in the filter also clears. The assumption here is that if the user no l=
onger wants the text to be locked, he probably also wants to clear the text.
5. When the filter bar is hidden, then the lock filter button is released.=
The next time the filter bar is shown, the lock filter button will be in t=
he released(off) position and the filter text will have been cleared. =
6. At application startup, the lock filter button is always off, and the fi=
lter bar text cleared. =
This also fixes a current bug in dolphin where one is unable to toggle the =
visibility of the filter bar using the tool bar filter button, or by using =
the assigned hot key.
Note... I think it is 'patch -p2' from the "dolphin" directory (above src d=
irectory ) to apply the patch
Diffs
-----
dolphin/src/dolphinviewcontainer.cpp 26a6803 =
dolphin/src/filterbar/filterbar.h 9546c63 =
dolphin/src/filterbar/filterbar.cpp f3076f0 =
Diff: http://git.reviewboard.kde.org/r/107392/diff/
Testing
-------
Went through all paths in dolphin. Tested: 1. Navigate to different folder=
. Text should clear only when text not locked
2.Ctl-I shortcut to toggle visiblility of filterbar
3.Text clears when filterbar hidden and then shown
4. Lock button locks and unlocks text
5. Shift-Ctl-I shortcut to toggle lock status of text. Lock button updates=
appropriately
6. Status (with non-perisistence from previous instance) of lock button whe=
n starting application..i.e. lock status is off on startup
7. Change keyboard shortcuts...works appropriately
8. F3 and split view..tabs..etc.. should still work appropriately
9. Monkey tested...cannot get it to crash
etc....
Screenshots
-----------
Filter bar with new text locking button
http://git.reviewboard.kde.org/r/107392/s/877/
filterbar-w/existing-lock-Icon
http://git.reviewboard.kde.org/r/107392/s/944/
Thanks,
Stuart Citrin
[Attachment #3 (text/html)]
<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 \
solid;"> <tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/107392/">http://git.reviewboard.kde.org/r/107392/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" \
style="background-image: \
url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); \
background-position: left top; background-repeat: repeat-x; border: 1px black \
solid;"> <tr>
<td>
<div>Review request for Dolphin, Oxygen Transparent and KDE Usability.</div>
<div>By Stuart Citrin.</div>
<p style="color: grey;"><i>Updated Dec. 31, 2012, 4:30 a.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Changes</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;">Usabliity and Oxygen people:
Please look at the two provided screen shots. "Filter bar with new text locking \
button" uses a pin (really a thumbtack) icon while filterbarw/existing-lock-icon \
uses the existing system-lock-screen icon. Please comment on whether it is worth the \
effort to add a new 'pin' icon to oxygen for this purpose or whether it would \
be better to use the existing system-lock-screen icon. Some pin icons are provided \
in file PinPngOxygenIcons.tar. Note, I am no artist....I just found a public domain \
picture of a thumbtack, changed the color and Sinc interpolated the picture down to \
the icons. There is some aliasing(and maybe ringing) in the 16x16 icon...but it \
might be accceptable.
(You may need to read the rest of the thread to figure out what all is going \
on...sorry)
Usability.... is it a big deal to have a button without a shortcut key....please \
comment. Again read the thread. Note, eliminating the shortcut key provides better \
code encapsculation and resistance to later bug 'regressions' and \
maintainance headaches.
(I couldn't find Nuno Pinheiro or Oxygen 'icon' in the people/groups...so \
I used Oxygen Transparent...if wrong please advise.) </pre>
</td>
</tr>
</table>
<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;">I wrote a quick add-on for the Dolphin filter bar. It is a button \
which prevents the filter text from clearing when navigating to a different folder.
Why this might be needed:
This is an attempt to resolve the following usability conflict:
If one navigates to a different folder, the filter is automatically cleared since one \
does not want a situation where someone forgets a filter is applied and then \
doesn't understand why all the files within a folder are not appearing.(current \
behavior)
versus:
A user might want to keep a filter while navigating an arbitrary set of folders, but \
is unable to, since the filter is automatically cleared upon navigation
The implementation to try to resolve this conflict is as follows:
1. A lock filter toggle button is added to the right side of the filter bar.
2. If the lock filter button is clicked(on), then if the user navigates away from the \
current folder, the filter is not cleared. If the lock filter button is \
released(off), then the text is cleared on navigation...which is the current \
behavior. 3. A configurable shortcut is added to toggle the lock filter button, with \
the default being Ctrl-Shift-I (since the default shortcut for hiding and showing the \
filter bar is Ctrl-I.) 4. If the user toggles the lock filter button to released \
(off), then the text in the filter also clears. The assumption here is that if the \
user no longer wants the text to be locked, he probably also wants to clear the text. \
5. When the filter bar is hidden, then the lock filter button is released. The next \
time the filter bar is shown, the lock filter button will be in the released(off) \
position and the filter text will have been cleared. 6. At application startup, the \
lock filter button is always off, and the filter bar text cleared.
This also fixes a current bug in dolphin where one is unable to toggle the visibility \
of the filter bar using the tool bar filter button, or by using the assigned hot key.
Note... I think it is 'patch -p2' from the "dolphin" directory \
(above src directory ) to apply the patch</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;">Went through all paths in dolphin. Tested: 1. Navigate to different \
folder. Text should clear only when text not locked 2.Ctl-I shortcut to toggle \
visiblility of filterbar 3.Text clears when filterbar hidden and then shown
4. Lock button locks and unlocks text
5. Shift-Ctl-I shortcut to toggle lock status of text. Lock button updates \
appropriately 6. Status (with non-perisistence from previous instance) of lock button \
when starting application..i.e. lock status is off on startup 7. Change keyboard \
shortcuts...works appropriately 8. F3 and split view..tabs..etc.. should still work \
appropriately 9. Monkey tested...cannot get it to crash
etc....</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>dolphin/src/dolphinviewcontainer.cpp <span style="color: \
grey">(26a6803)</span></li>
<li>dolphin/src/filterbar/filterbar.h <span style="color: \
grey">(9546c63)</span></li>
<li>dolphin/src/filterbar/filterbar.cpp <span style="color: \
grey">(f3076f0)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/107392/diff/" style="margin-left: \
3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Screenshots </h1>
<div>
<a href="http://git.reviewboard.kde.org/r/107392/s/877/"><img \
src="http://git.reviewboard.kde.org/media/uploaded/images/2012/12/07/FilterBarWithLockingButton_400x100.png" \
style="border: 1px black solid;" alt="Filter bar with new text locking button" /></a>
<a href="http://git.reviewboard.kde.org/r/107392/s/944/"><img \
src="http://git.reviewboard.kde.org/media/uploaded/images/2012/12/31/FilterBarWithSystemLockIcon_400x100.png" \
style="border: 1px black solid;" alt="filterbar-w/existing-lock-Icon" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic