[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-panel-devel
Subject: Re: Review Request: Simplify the comic config ui
From: todd rme <toddrme2178 () gmail ! com>
Date: 2011-04-27 17:02:32
Message-ID: BANLkTin5tYaOHJGiJ8GYqaMNd4=ZdkUhCA () mail ! gmail ! com
[Download RAW message or body]
[Attachment #2 (multipart/alternative)]
On Wed, Apr 27, 2011 at 12:02 PM, Matthias Fuchs <mat69@gmx.net> wrote:
>
> On April 27th, 2011, 12:29 p.m., *Aaron J. Seigo* wrote:
>
> great steps towards improvements; one further set of improvements is to get rid of \
> the checkbox+spinner combos. instead, try sth like:
> Comics cache: [ No size limit ] <== where No Limit is a "special text value" for, \
> e.g., 0 which does the same as not checking the checkbox; a suffix of, e.g. " \
> strips per comic" would help further disambiguate ("2 strips per comic")
> Automatic update: [ Never ] <== Never is the special text for 0 (and equal to \
> unchecking the box), and other values would be "Every %1 days"; or perhaps even get \
> rid of "no automatic update" and just have "Automatic update: [ Daily / Every %1 \
> Days ]"
> Thanks a lot for your input!
> I also added the ideas you mentioned above and commited it now. Is way more clear \
> than it was before. :)
>
> - Matthias
>
> I couple quick points:
What does middle-clicking on a comic do if the box about "show at original
size" is not checked? It isn't clear. Does it do anything? If not, what
is the reason for having this option at all? Can't people who don't want to
see it at its original size just not middle click?
If you get rid of that, you could also get rid of the "Comic" and "Update"
headers. If you don't want to, "Update" has a colon while "Comic", "Cache",
and "Error Handling" don't. This may or may not be a typo.
You also have "Comic" twice, once as a header and once as a title for the
list of comics. This is probably redundant, I think it is clear that the
list is the list of comics.
It is unclear what will happen if the "show error message" check box is not
clicked. Does the tooltip make clear what not checking this box would do?
-Todd
[Attachment #5 (text/html)]
<div class="gmail_quote">On Wed, Apr 27, 2011 at 12:02 PM, Matthias Fuchs <span \
dir="ltr"><<a href="mailto:mat69@gmx.net">mat69@gmx.net</a>></span> \
wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex;">
<div><div style="font-family:Verdana, Arial, Helvetica, Sans-Serif"><div \
class="im"><blockquote style="margin-left:1em;border-left:2px solid \
#d0d0d0;padding-left:10px"> <p style="margin-top:0">On April 27th, 2011, 12:29 p.m., \
<b>Aaron J. Seigo</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">great \
steps towards improvements; one further set of improvements is to get rid of the \
checkbox+spinner combos. instead, try sth like:
Comics cache: [ No size limit ] <== where No Limit is a "special text \
value" for, e.g., 0 which does the same as not checking the checkbox; a suffix \
of, e.g. " strips per comic" would help further disambiguate ("2 \
strips per comic")
Automatic update: [ Never ] <== Never is the special text for 0 (and equal to \
unchecking the box), and other values would be "Every %1 days"; or perhaps \
even get rid of "no automatic update" and just have "Automatic update: \
[ Daily / Every %1 Days ]"</pre>
</blockquote>
</blockquote>
</div><pre style="white-space:pre-wrap;white-space:-moz-pre-wrap;white-space:-pre-wrap;white-space:-o-pre-wrap;word-wrap:break-word">Thanks \
a lot for your input! I also added the ideas you mentioned above and commited it now. \
Is way more clear than it was before. :)</pre> <br><font color="#888888">
<p>- Matthias</p></font><br></div></div></blockquote><div>I couple quick \
points:<br><br>What does middle-clicking on a comic do if the box about "show at \
original size" is not checked? It isn't clear. Does it do anything? If \
not, what is the reason for having this option at all? Can't people who \
don't want to see it at its original size just not middle click?<br>
<br>If you get rid of that, you could also get rid of the "Comic" and \
"Update" headers. If you don't want to, "Update" has a colon \
while "Comic", "Cache", and "Error Handling" don't. \
This may or may not be a typo.<br>
<br>You also have "Comic" twice, once as a header and once as a title for \
the list of comics. This is probably redundant, I think it is clear that the list is \
the list of comics.<br><br>It is unclear what will happen if the "show error \
message" check box is not clicked. Does the tooltip make clear what not \
checking this box would do?<br>
<br>-Todd<br></div></div>
_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic