On 08/19/2014 12:43 PM, Christy Perez wrote:
On 08/11/2014 12:41 PM, Aline Manera wrote:
> On 08/08/2014 10:31 AM, CrÃstian Viana wrote:
>> The small icons on each template indicate whether they are local or
>> remote but that may not be obvious to everyone.
>>
>> Show a description when the user hovers the mouse over the template icons
>> indicating what they mean: local or remote template. Also, the same
>> descriptive text will be shown in the icon's place if the browser is not
>> able to load images.
> I don't think a mouse hover with local/remote ISO will be useful.
> Instead of that I thought in displaying the ISO path which can be more
> informative.
> But when talking to Yu Xin, we identified this information (ISO path)
> should be always displayed as it is the main template information
> involved in the user's decision.
> So we fall back to a big change in the templates tab view. The idea is
> display the templates in a list view (the same we did for networks and
> storage pools).
> That way we can show more information to the user without the need to
> check the "Edit" dialog for it.
> It is our first idea. But more suggestions are welcome.
>
A small add-on to this (and the networks and storage pools) might be to
have an option to expand all the items at once. If I'm looking for a
particular template, it would save me time to see the details of them
all at once.
Seems what you are proposing is more like a filter, isn't it?
For example, display all the fedora templates or ubuntu.
>> Signed-off-by: CrÃstian Viana
<vianac(a)linux.vnet.ibm.com>
>> ---
>> ui/js/src/kimchi.template_main.js | 2 ++
>> ui/pages/i18n.json.tmpl | 2 ++
>> ui/pages/tabs/templates.html.tmpl | 2 +-
>> 3 files changed, 5 insertions(+), 1 deletion(-)
>>
>> diff --git a/ui/js/src/kimchi.template_main.js
>> b/ui/js/src/kimchi.template_main.js
>> index 3c8421d..ae3f290 100644
>> --- a/ui/js/src/kimchi.template_main.js
>> +++ b/ui/js/src/kimchi.template_main.js
>> @@ -25,8 +25,10 @@ kimchi.doListTemplates = function() {
>> var isLocal = /^\//.test(value['cdrom']);
>> if(isLocal){
>> value.location =
>> "images/theme-default/icon-local.png";
>> + value.iconDescription = i18n['KCHTMPL6004M'];
>> }else{
>> value.location =
>> "images/theme-default/icon-remote.png";
>> + value.iconDescription = i18n['KCHTMPL6005M'];
>> }
>> listHtml += kimchi.substitute(templateHtml, value);
>> });
>> diff --git a/ui/pages/i18n.json.tmpl b/ui/pages/i18n.json.tmpl
>> index f1478a7..8e6fcd6 100644
>> --- a/ui/pages/i18n.json.tmpl
>> +++ b/ui/pages/i18n.json.tmpl
>> @@ -61,6 +61,8 @@
>> "KCHTMPL6002M": "$_("It will take long time. Do you
want to
>> continue?")",
>> "KCHTMPL6003M": "$_("This will permanently delete the
template.
>> Would you like to continue?")",
>> + "KCHTMPL6004M": "$_("Local template")",
>> + "KCHTMPL6005M": "$_("Remote template")",
>> "KCHHOST6001E": "$_("Unable to shut down system as
there are
>> some virtual machines running!")",
>> diff --git a/ui/pages/tabs/templates.html.tmpl
>> b/ui/pages/tabs/templates.html.tmpl
>> index 7cf7fcd..4fe1d73 100644
>> --- a/ui/pages/tabs/templates.html.tmpl
>> +++ b/ui/pages/tabs/templates.html.tmpl
>> @@ -50,7 +50,7 @@
>> <div class="template-icon
template-icon-position">
>> <img alt="" src="{icon}">
>> - <img alt="" src="{location}"
>> class="template-type-icon-position">
>> + <img alt="{iconDescription}"
>> title="{iconDescription}" src="{location}"
>> class="template-type-icon-position">
>> </div>
>> <div class="template-general template-title
>> template-title-position">
>> <h2 class="title"
title="{name}">{name}</h2>
> _______________________________________________
> Kimchi-devel mailing list
> Kimchi-devel(a)ovirt.org
>
http://lists.ovirt.org/mailman/listinfo/kimchi-devel
_______________________________________________
Kimchi-devel mailing list
Kimchi-devel(a)ovirt.org
http://lists.ovirt.org/mailman/listinfo/kimchi-devel