On Thu, Dec 30, 2010 at 6:43 PM, stephen paul <[email protected]> wrote: > Yea I'm in n rush to submit as you only get one chance at a first impression. > I'll see what I can to do with those suggestions... > > - Side Note - > > I assume all skin updates, like an up date to Night, are on hold till > http://trac.xbmc.org/ticket/10876 is fixed, correct? > > --- On Thu, 12/30/10, Cory Fields <[email protected]> wrote: > >> From: Cory Fields <[email protected]> >> Subject: Re: [Xbmc-addons] [GIT Pull] skin.carmichael >> To: "stephen paul" <[email protected]> >> Cc: [email protected] >> Date: Thursday, December 30, 2010, 1:36 PM >> On Wed, Dec 29, 2010 at 10:09 PM, >> stephen paul <[email protected]> >> wrote: >> > skin.carmichael >> > >> > git://github.com/bartowski/thecarmichael.git >> > >> > ce9c1307b0eb44b >> > >> > I believe I'm happy to consider this 'finished' and >> ready to start the inclusion process... >> > >> > >> ------------------------------------------------------------------------------ >> > Learn how Oracle Real Application Clusters (RAC) One >> Node allows customers >> > to consolidate database storage, standardize their >> database environment, and, >> > should the need arise, upgrade to a full multi-node >> Oracle RAC database >> > without downtime or disruption >> > http://p.sf.net/sfu/oracle-sfdevnl >> > _______________________________________________ >> > Xbmc-addons mailing list >> > [email protected] >> > https://lists.sourceforge.net/lists/listinfo/xbmc-addons >> > >> >> This is a first... I can't find anything to complain about. >> I am ok >> with committing this as-is. >> >> I do believe there are some usability issues, but it's >> certainly not >> my place to hold it up. Take the points below purely as >> user input. If >> you want me to push as-is, say the word. >> >> 1. If a section has no sources (for ex. pictures, for me), >> you see an >> empty panel on the home screen. I could see how this would >> be >> confusing. Maybe hide the panel and say "no sources" or >> something >> instead? Same goes for weather... empty panel, user might >> think >> something is supposed to be there. >> 2. (nit) 3rd item in middle panel has a slight 3d effect >> 3. HD Movies lists recent/random non-hd movies as well. I >> realize it's >> probably running the same script as movies, but not (imo) >> the expected >> behavior. >> 4. The middle panel just looks a bit unpolished in general >> imo. Maybe >> if it stood out a bit less? Also, I think an option to >> disable it >> entirely would be appreciated by many (including myself) if >> it doesn't >> completely upset the flow of the skin. >> 5. The main (top) menu would be more functional if it >> didn't wrap. >> Less chance of jumping around. Also, some kind of effect, >> maybe some >> blur, texture, or shadowing might class it up a bit. >> >> Thanks for another great contribution, >> Cory >> >
Still investigating that one, yea. Adding a new skin is no problem though. Again, those are only suggestions. I'm certainly not here to police the style or function of a skin, only the technical merit. I'll push whenever you say 'go'. Thanks for the consideration though, I must commend you on how easy you make this on me :) Cory ------------------------------------------------------------------------------ Learn how Oracle Real Application Clusters (RAC) One Node allows customers to consolidate database storage, standardize their database environment, and, should the need arise, upgrade to a full multi-node Oracle RAC database without downtime or disruption http://p.sf.net/sfu/oracle-sfdevnl _______________________________________________ Xbmc-addons mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/xbmc-addons
