Vreman, Peter - Acision wrote:
>>> Attached is a patch that makes the sidebar dynamic based on the current
>>>       
>> page.
>>     
>>> Currently it contains only the Add actions that were already in the
>>>       
>> previous menu.
>>     
>>> There are some things in the style.css that are not working. E.g. the
>>>       
>> highlight of the current entry. And also some image urls are invalid. But
>> my html/css knowledge is too limited to get this fixed. Maybe somebody
>> else can have a look at it.
>>     
>> I like this.   It's a simple change but makes good sense to be looking
>> at the list page before you click add, and also saves room on the nav bar.
>>     
>
> Another thing I like to add with this is to make the Edit really edit and 
> move the 'Edit+Rename' and 'Edit+Copy' also to this menu. I think it is much 
> cleaner. Also operations can be added
>
> The menu for systems can then look like:
>
> Systems
>   -----
>   Add
>   Copy    (or is Add similar a better name?)
>   Rename
>   Delete
>   -----
>   Change netboot
>   Change profile
>   -----
>   Poweron
>   Poweroff
>   Reboot
>
> Thoughts?
>
>   
While the Add button adds an initially blank object, the others depend 
on an object to be selected to fill in the fields.

Rather than include those in the sidebar, how about including those 
links on each "object list" page?

Imagine a list of several systems, with each row looking like:

| system name | add icon | edit icon | copy icon | rename icon | delete 
icon | power on icon | power off icon | netboot toggle

We could make the delete button skip loading the current "edit" provided 
it was protected with a javascript "are you sure" kind of approach to 
prevent single clicks from causing a delete.

Each icon could have a javascript target to take the user to the proper 
edit page, or in the case of delete, make the call directly. They 
wouldn't necessarily have to be icons, they could also be links.

>
>   
>> Quick question --  Which image URLs do you mean are invalid?   I think
>> cobbler CSS has some leftover cruft in it that can be removed, we might
>> not be using them.    The Cobbler web CSS was adapted from the original
>> cobbler.et.redhat.com web site and not everything applies.  I too am not
>> a CSS wizard so I have not bothered addressing all of these things.
>>     
>
> I meant the URLs to the list-style-images and buttons.
>   

Yeah, I don't think we need these.

>   
>> I've applied this change and if someone else would like to take a shot
>> and making the CSS shinier, they are welcome to do so.
>>
>> Someone on IRC also brought up the issue of Cobbler's logo taking up too
>> much space, though I kind of like it.  I think we can leave that is is.
>>     
>
> I also think it takes too much space with a 1024x768 laptop screen.
> Maybe it can be moved above the menu in the sidebar.
>   

This sounds like a good idea.
>
>   
>> One possible CSS extension is on the various pages to group the various
>> edit fields by tabs, such as the virt details on a different tab than
>> the network details, to minimize scrolling.  I'm not sure that's ideal
>> or not but that was a thought.
>>     
>
> I was also thinking about this once more system properties are being added. 
> But I'm also not a CSS-guru.
>   

I've done this before and is something I could address for 1.6.

There are a lot of good hiding DIV/tab kind of examples on the web.


> Regards,
> Peter
>
> This e-mail and any attachment is for authorised use by the intended 
> recipient(s) only. It may contain proprietary material, confidential 
> information and/or be subject to legal privilege. It should not be copied, 
> disclosed to, retained or used by, any other party. If you are not an 
> intended recipient then please promptly delete this e-mail and any attachment 
> and all copies and inform the sender. Thank you.
>
>
> _______________________________________________
> cobbler mailing list
> [email protected]
> https://fedorahosted.org/mailman/listinfo/cobbler
>   

_______________________________________________
cobbler mailing list
[email protected]
https://fedorahosted.org/mailman/listinfo/cobbler

Reply via email to