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. > > Regards, > Peter > > >
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. 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'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. 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. Thanks once again, --Michael > 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
