That's great - yep it's worth it to get everything compliant - thanks for all your help.

Steve


On 6 September 2010 19:32, Cory Fields <TheUni@xbmc.org> wrote:
On Mon, Sep 6, 2010 at 1:59 PM, Sharpe <sharpe2@gmail.com> wrote:
> Oh dear silly me - hopefully fixed it with 2899.
>
> Cheers
>
> On 6 September 2010 18:31, Cory Fields <TheUni@xbmc.org> wrote:
>>
>> On Mon, Sep 6, 2010 at 1:25 PM, Sharpe <sharpe2@gmail.com> wrote:
>> > Oh that's great - thanks very much Cory everything seems to be working
>> > fine
>> > - I've updated the SVN with the changes.
>> >
>> > Hopefully we're ok to go ahead with the pull request now.
>> >
>> > Link to grab from :
>> >
>> >>
>> >>
>> >> https://xboxmediacenter.svn.sourceforge.net/svnroot/xboxmediacenter/Back-Row
>> >
>> >> Version: 2.8.9
>> >> SVN Revision 2898
>> >> Add-on Name :   skin.Back-Row
>> >
>> >
>> > 2010/9/6 Cory Fields <TheUni@xbmc.org>
>> >>
>> >> On Sun, Sep 5, 2010 at 9:42 PM, Cory Fields <TheUni@xbmc.org> wrote:
>> >> > On Sun, Sep 5, 2010 at 2:54 AM, Sharpe <sharpe2@gmail.com> wrote:
>> >> >> When I try and apply the patch I'm getting an error I can't seem to
>> >> >> get
>> >> >> past
>> >> >> - sorry.
>> >> >>
>> >> >> Would you mind supplying the full files and I'll update.
>> >> >>
>> >> >> Thanks
>> >> >>
>> >> >>
>> >> >> On 5 September 2010 02:18, Cory Fields <TheUni@xbmc.org> wrote:
>> >> >>>
>> >> >>> On Sat, Sep 4, 2010 at 2:46 PM, Sharpe <ratchford@gmail.com> wrote:
>> >> >>> > Hi Cory
>> >> >>> >
>> >> >>> > Sorry about this but I'm not really up speed regarding patching
>> >> >>> > files it
>> >> >>> > will take me a while to figure it out.
>> >> >>> >
>> >> >>> > However am I not in the same boat as Jezz_X regarding the patch
>> >> >>> > file? In
>> >> >>> > that I'm using SVN rather than GIT - will this patch work with my
>> >> >>> > SVN
>> >> >>> > setup?
>> >> >>> >
>> >> >>> > Also the "<visible>!Skin.HasSetting(Time&Date)</visible>" line is
>> >> >>> > what I
>> >> >>> > use
>> >> >>> > to allow users to diasable the time and dat labels on menus - it
>> >> >>> > seems
>> >> >>> > to
>> >> >>> > work fine - what is wrong with it so I can fix it?
>> >> >>> >
>> >> >>> > Thanks for your help
>> >> >>> >
>> >> >>> > Steve
>> >> >>> >
>> >> >>> >
>> >> >>> > Cory Fields wrote:
>> >> >>> >
>> >> >>> > On Fri, Sep 3, 2010 at 8:26 PM, Sharpe <sharpe2@gmail.com> wrote:
>> >> >>> >
>> >> >>> >
>> >> >>> > Link to grab from :
>> >> >>> >
>> >> >>> >
>> >> >>> >
>> >> >>> > https://xboxmediacenter.svn.sourceforge.net/svnroot/xboxmediacenter/Back-Row
>> >> >>> > Version: 2.8.9
>> >> >>> > SVN Revision 2897
>> >> >>> > Add-on Name :   skin.Back-Row
>> >> >>> >
>> >> >>> > Version 2.8.9
>> >> >>> > - Changed watched overlays default setting
>> >> >>> > - Added episodes label to TV show video library list view
>> >> >>> > - Utilised T!'s and Night's cool CD spinning animation for Music
>> >> >>> > OSD
>> >> >>> > - Added more CDArt to Music OSD
>> >> >>> >
>> >> >>> > Version 2.8.8
>> >> >>> > - VideoFullScreen.xml linux font fix (part II)
>> >> >>> > - Some changes to Filemode views (more fanart)
>> >> >>> > - VideoFullScreen.xml font fix
>> >> >>> >
>> >> >>> > Sorry about about this - I'm very conscious that it has been less
>> >> >>> > than 2
>> >> >>> > weeks since my last pull request but I fixed a fairly significant
>> >> >>> > linux
>> >> >>> > case
>> >> >>> > sensitive font bug recently and thought it best to get that out
>> >> >>> > mainly.
>> >> >>> >
>> >> >>> > If you need me to hold off and request this later than that's
>> >> >>> > cool -
>> >> >>> > just
>> >> >>> > let me know.
>> >> >>> >
>> >> >>> > Cheers
>> >> >>> >
>> >> >>> > Steve
>> >> >>> >
>> >> >>> >
>> >> >>> >
>> >> >>> >
>> >> >>> > ------------------------------------------------------------------------------
>> >> >>> > This SF.net Dev2Dev email is sponsored by:
>> >> >>> >
>> >> >>> > Show off your parallel programming skills.
>> >> >>> > Enter the Intel(R) Threading Challenge 2010.
>> >> >>> > http://p.sf.net/sfu/intel-thread-sfd
>> >> >>> > _______________________________________________
>> >> >>> > Xbmc-addons mailing list
>> >> >>> > Xbmc-addons@lists.sourceforge.net
>> >> >>> > https://lists.sourceforge.net/lists/listinfo/xbmc-addons
>> >> >>> >
>> >> >>> >
>> >> >>> >
>> >> >>> >
>> >> >>> > Following the others, here is a patch to ensure xml validation on
>> >> >>> > your
>> >> >>> > skin. Diff is against our repo, sorry. You'll probably have to
>> >> >>> > patch
>> >> >>> > -p1 it.
>> >> >>> >
>> >> >>> > Also, there are several of these:
>> >> >>> > <visible>!Skin.HasSetting(Time&Date)</visible> in
>> >> >>> > PAL16x9/includes.xml. This is not valid xml, and I'm relatively
>> >> >>> > certain it's not a valid HasSetting() either.
>> >> >>> >
>> >> >>> > Please fix these things, then I'll pull.
>> >> >>> >
>> >> >>> > Cory
>> >> >>> >
>> >> >>> >
>> >> >>> > ________________________________
>> >> >>> >
>> >> >>> >
>> >> >>> >
>> >> >>> > ------------------------------------------------------------------------------
>> >> >>> > This SF.net Dev2Dev email is sponsored by:
>> >> >>> >
>> >> >>> > Show off your parallel programming skills.
>> >> >>> > Enter the Intel(R) Threading Challenge 2010.
>> >> >>> > http://p.sf.net/sfu/intel-thread-sfd
>> >> >>> >
>> >> >>> > ________________________________
>> >> >>> > _______________________________________________
>> >> >>> > Xbmc-addons mailing list
>> >> >>> > Xbmc-addons@lists.sourceforge.net
>> >> >>> > https://lists.sourceforge.net/lists/listinfo/xbmc-addons
>> >> >>> >
>> >> >>>
>> >> >>> As Ansii said in the pm3.hd thread, that patch format should work
>> >> >>> just
>> >> >>> fine, else the tool is broken. If you're having lots of trouble let
>> >> >>> me
>> >> >>> know and I'll just attach the full files.
>> >> >>>
>> >> >>> Cory
>> >> >>>
>> >> >>>
>> >> >>>
>> >> >>>
>> >> >>> ------------------------------------------------------------------------------
>> >> >>> This SF.net Dev2Dev email is sponsored by:
>> >> >>>
>> >> >>> Show off your parallel programming skills.
>> >> >>> Enter the Intel(R) Threading Challenge 2010.
>> >> >>> http://p.sf.net/sfu/intel-thread-sfd
>> >> >>> _______________________________________________
>> >> >>> Xbmc-addons mailing list
>> >> >>> Xbmc-addons@lists.sourceforge.net
>> >> >>> https://lists.sourceforge.net/lists/listinfo/xbmc-addons
>> >> >>
>> >> >>
>> >> >
>> >> > On second look, there are a few more files that need some love. Will
>> >> > discuss and send the full files in the next day.
>> >> >
>> >> > Cory
>> >> >
>> >>
>> >> OK, all fixed up. FYI main problems were:
>> >>
>> >> -invalid xml comments
>> >> -& rather than &amp;
>> >> -Time&Date replaced with TimeDate
>> >>  For reference: <jmarshall> No reason at all why they're using
>> >> Time&Date though - could just as easily be TimeDate.  The & there is
>> >> meaningless.
>> >>  Please test to be sure this still works as intended.
>> >> -Extra </window> at the end of PAL16x9/Customisation.xml
>> >>
>> >> There is a complete lack of love for unified diffs on this list, so
>> >> full files attached. I'll include the diff inline for viewing, though
>> >> gmail will probably mangle it.
>> >>
>> >> Cory
>> >>
>> >> --
>> >>
>> >> diff --git a/skin.back-row/720p/Home.xml b/skin.back-row/720p/Home.xml
>> >> index 1ee6478..db3940d 100644
>> >> --- a/skin.back-row/720p/Home.xml
>> >> +++ b/skin.back-row/720p/Home.xml
>> >> @@ -1487,7 +1487,7 @@
>> >>  </control>
>> >>
>> >>
>> >> -<!---------Home Screen------->
>> >> +<!--Home Screen-->
>> >>
>> >>  <control type="group">
>> >>
>> >> @@ -4425,4 +4425,4 @@
>> >>
>> >>  </controls>
>> >>
>> >> -</window>
>> >> \ No newline at end of file
>> >> +</window>
>> >> diff --git a/skin.back-row/PAL/CommonViewTypes.xml
>> >> b/skin.back-row/PAL/CommonViewTypes.xml
>> >> index 6abf465..eaf4421 100644
>> >> --- a/skin.back-row/PAL/CommonViewTypes.xml
>> >> +++ b/skin.back-row/PAL/CommonViewTypes.xml
>> >> @@ -9408,7 +9408,7 @@
>> >>
>> >>  </include>
>> >>
>> >> -<!------ALT DVD CASE WRAP------>
>> >> +<!--ALT DVD CASE WRAP-->
>> >>
>> >>  <include name="DVD Case Wrap2">
>> >>
>> >> @@ -9732,7 +9732,7 @@
>> >>
>> >>  </include>
>> >>
>> >> -<!------ALT DVD CASE WRAP------>
>> >> +<!--ALT DVD CASE WRAP-->
>> >>
>> >>  <include name="DVD Case Wrap 2">
>> >>
>> >> @@ -10238,7 +10238,7 @@
>> >>  </control>
>> >>  </include>
>> >>
>> >> -<!------ALT Games CASE WRAP------>
>> >> +<!--ALT Games CASE WRAP-->
>> >>
>> >>  <include name="Games Case Wrap2">
>> >>
>> >> @@ -12099,7 +12099,7 @@
>> >>  <animation effect="fade" time="200">WindowClose</animation>
>> >>  </control>
>> >>
>> >> -<-------->
>> >> +<!---->
>> >>
>> >>  <control type="label">
>> >>  <posx>495</posx>
>> >> @@ -12152,7 +12152,7 @@
>> >>  <animation effect="fade" time="200">WindowClose</animation>
>> >>  </control>
>> >>
>> >> -<---------------->
>> >> +<!---->
>> >>
>> >>
>> >>  <control type="label">
>> >> @@ -12624,7 +12624,7 @@
>> >>  <visible>!Skin.HasSetting(TV Poster)</visible>
>> >>  </control>
>> >>
>> >> -<------------->
>> >> +<!---->
>> >>
>> >>  <control type="label">
>> >>  <posx>495</posx>
>> >> @@ -12808,7 +12808,7 @@
>> >>  <visible>Skin.HasSetting(TV Poster)</visible>
>> >>  </control>
>> >>
>> >> -<-------------------->
>> >> +<!---->
>> >>
>> >>
>> >>
>> >> diff --git a/skin.back-row/PAL/Home.xml b/skin.back-row/PAL/Home.xml
>> >> index a86f17c..0440d4c 100644
>> >> --- a/skin.back-row/PAL/Home.xml
>> >> +++ b/skin.back-row/PAL/Home.xml
>> >> @@ -1400,7 +1400,7 @@
>> >>
>> >>
>> >>
>> >> -<!---------Home Screen------->
>> >> +<!--Home Screen-->
>> >>
>> >>
>> >>  <control type="group">
>> >> @@ -2849,4 +2849,4 @@
>> >>
>> >>  </controls>
>> >>
>> >> -</window>
>> >> \ No newline at end of file
>> >> +</window>
>> >> diff --git a/skin.back-row/PAL16x9/Common Video Library Views.xml
>> >> b/skin.back-row/PAL16x9/Common Video Library Views.xml
>> >> index e1b9854..20398bc 100644
>> >> --- a/skin.back-row/PAL16x9/Common Video Library Views.xml
>> >> +++ b/skin.back-row/PAL16x9/Common Video Library Views.xml
>> >> @@ -1,6 +1,6 @@
>> >>  <includes>
>> >>
>> >> -<!---------------------------------------------------->
>> >> +<!---->
>> >>
>> >>  <include name="Library Root View">
>> >>
>> >> @@ -389,7 +389,7 @@
>> >>  </include>
>> >>
>> >>
>> >> -<!---------------------------------------------------->
>> >> +<!---->
>> >>
>> >>
>> >>
>> >> @@ -725,7 +725,7 @@
>> >>  </include>
>> >>
>> >>
>> >> -<!---------------------------------------------------->
>> >> +<!---->
>> >>
>> >>
>> >>
>> >> @@ -2809,7 +2809,7 @@
>> >>  </include>
>> >>
>> >>
>> >> -<!---------------------------------------------------->
>> >> +<!---->
>> >>
>> >>
>> >>
>> >> @@ -3400,7 +3400,7 @@
>> >>  </include>
>> >>
>> >>
>> >> -<!---------------------------------------------------->
>> >> +<!---->
>> >>
>> >>
>> >>
>> >> @@ -3982,7 +3982,7 @@
>> >>  </include>
>> >>
>> >>
>> >> -<!-------------------------------------->
>> >> +<!---->
>> >>
>> >>
>> >>
>> >> @@ -4729,7 +4729,7 @@
>> >>
>> >>
>> >>
>> >> -<!---------------------------------------------------->
>> >> +<!---->
>> >>
>> >>
>> >>
>> >> @@ -5048,7 +5048,7 @@
>> >>
>> >>
>> >>
>> >> -<!---------------------------------------------------->
>> >> +<!---->
>> >>
>> >>
>> >>
>> >> @@ -8735,7 +8735,7 @@
>> >>
>> >>
>> >>
>> >> -<---------------------->
>> >> +<!---->
>> >>
>> >>  <control type="largeimage" id="77020">
>> >>  <posx>535</posx>
>> >> @@ -8952,7 +8952,7 @@
>> >>  </control>
>> >>
>> >>
>> >> -<--------->
>> >> +<!---->
>> >>
>> >>
>> >>
>> >> @@ -9707,7 +9707,7 @@
>> >>  <visible>!Skin.HasSetting(TV Poster)</visible>
>> >>  </control>
>> >>
>> >> -<-------->
>> >> +<!---->
>> >>
>> >>
>> >>  <control type="image">
>> >> @@ -9900,7 +9900,7 @@
>> >>
>> >>
>> >>
>> >> -<--------->
>> >> +<!---->
>> >>
>> >>  </control>
>> >>
>> >> diff --git a/skin.back-row/PAL16x9/CommonViewTypes.xml
>> >> b/skin.back-row/PAL16x9/CommonViewTypes.xml
>> >> index f60f460..222af6e 100644
>> >> --- a/skin.back-row/PAL16x9/CommonViewTypes.xml
>> >> +++ b/skin.back-row/PAL16x9/CommonViewTypes.xml
>> >> @@ -10763,7 +10763,7 @@
>> >>  </control>
>> >>  </include>
>> >>
>> >> -<!------ALT Games CASE WRAP------>
>> >> +<!--ALT Games CASE WRAP -->
>> >>
>> >>  <include name="Games Case Wrap2">
>> >>
>> >> diff --git a/skin.back-row/PAL16x9/Customisation.xml
>> >> b/skin.back-row/PAL16x9/Customisation.xml
>> >> index cddc75b..1f54bec 100644
>> >> --- a/skin.back-row/PAL16x9/Customisation.xml
>> >> +++ b/skin.back-row/PAL16x9/Customisation.xml
>> >> @@ -3040,7 +3040,7 @@
>> >>  <visible allowhiddenfocus="true">Control.HasFocus(4) |
>> >> ControlGroup(13).HasFocus()</visible>
>> >>
>> >>  <control>
>> >> -<description>Display Time & Date Switch</description>
>> >> +<description>Display Time &amp; Date Switch</description>
>> >>  <type>radiobutton</type>
>> >>  <textoffsetx>10</textoffsetx>
>> >>  <id>600</id>
>> >> @@ -3048,11 +3048,11 @@
>> >>  <posy>115</posy>
>> >>  <include>20061</include>
>> >>  <font>fontKB</font>
>> >> -<selected>!Skin.HasSetting(Time&Date)</selected>
>> >> +<selected>!Skin.HasSetting(TimeDate)</selected>
>> >>  <align>left</align>
>> >>  <texturefocus>input-focus.png</texturefocus>
>> >>  <texturenofocus></texturenofocus>
>> >> -<onclick>Skin.ToggleSetting(Time&Date)</onclick>
>> >> +<onclick>Skin.ToggleSetting(TimeDate)</onclick>
>> >>  <width>560</width>
>> >>  <height>30</height>
>> >>  <onleft>4</onleft>
>> >> @@ -4863,4 +4863,4 @@
>> >>  </control>
>> >>
>> >>  </controls>
>> >> -</window>
>> >> \ No newline at end of file
>> >> +</window>
>> >> diff --git a/skin.back-row/PAL16x9/DialogSubMenu.xml
>> >> b/skin.back-row/PAL16x9/DialogSubMenu.xml
>> >> index 264a4c2..68510f8 100644
>> >> --- a/skin.back-row/PAL16x9/DialogSubMenu.xml
>> >> +++ b/skin.back-row/PAL16x9/DialogSubMenu.xml
>> >> @@ -34,7 +34,7 @@
>> >>  </control>
>> >>
>> >>
>> >> -<!------New Sub menu controls------>
>> >> +<!--New Sub menu controls-->
>> >>
>> >>  <control type="list" id="51">
>> >>  <viewtype label="535">list</viewtype>
>> >> @@ -291,4 +291,4 @@
>> >>  </control>
>> >>
>> >>  </controls>
>> >> -</window>
>> >> \ No newline at end of file
>> >> +</window>
>> >> diff --git a/skin.back-row/PAL16x9/Home.xml
>> >> b/skin.back-row/PAL16x9/Home.xml
>> >> index 40219ba..d7e1300 100644
>> >> --- a/skin.back-row/PAL16x9/Home.xml
>> >> +++ b/skin.back-row/PAL16x9/Home.xml
>> >> @@ -1430,7 +1430,7 @@
>> >>  </control>
>> >>
>> >>
>> >> -<!---------Home Screen------->
>> >> +<!--Home Screen-->
>> >>
>> >>
>> >>  <control type="group">
>> >> @@ -2882,7 +2882,3 @@
>> >>  </controls>
>> >>
>> >>  </window>
>> >> -
>> >> -</controls>
>> >> -
>> >> -</window>
>> >> \ No newline at end of file
>> >> diff --git a/skin.back-row/PAL16x9/includes.xml
>> >> b/skin.back-row/PAL16x9/includes.xml
>> >> index 89529b7..6ff46da 100644
>> >> --- a/skin.back-row/PAL16x9/includes.xml
>> >> +++ b/skin.back-row/PAL16x9/includes.xml
>> >> @@ -54,7 +54,7 @@
>> >>  <include name="20068"><label>31050</label></include>
>> >>
>> >>
>> >> -<!-- Settings Labels
>> >> ---------------------------------------------------
>> >>  -->
>> >> +<!-- Settings Labels -->
>> >>  <include name="20120"><label>31023</label></include>
>> >>  <include name="20121"><label>31024</label></include>
>> >>  <include name="20122"><label>31025</label></include>
>> >> @@ -69,13 +69,13 @@
>> >>  <include name="20131"><label>31034</label></include>
>> >>  <include name="20132"><label>31037</label></include>
>> >>
>> >> -<!-- Menu Background - Color Presets
>> >> -----------------------------------
>> >>  -->
>> >> +<!-- Menu Background - Color Presets -->
>> >>
>> >>  <!-- FF435570 : Purple/Grey -->
>> >>  <!-- FF001e39 : Blue -->
>> >> -<!-- FF293d47 : Dark Blue/Grey #1
>> >> -<!-- FF23343c : Dark Blue/Grey #2
>> >> -<!-- FF1a272d : Dark Blue/Grey #3
>> >> +<!-- FF293d47 : Dark Blue/Grey #1 -->
>> >> +<!-- FF23343c : Dark Blue/Grey #2 -->
>> >> +<!-- FF1a272d : Dark Blue/Grey #3 -->
>> >>  <!-- FFb5e817 : Light Green -->
>> >>  <!-- FF506b3e : Dark Green -->
>> >>  <!-- FF610505 : Red  -->
>> >> @@ -89,7 +89,7 @@
>> >>  </include>
>> >>
>> >>
>> >> -<!-- Text color Presets
>> >> -------------------------------------------------
>> >> -->
>> >> +<!-- Text color Presets -->
>> >>  <include name="textcolorwhite">
>> >>  <textcolor>white</textcolor>
>> >>  <shadowcolor>black</shadowcolor></include>
>> >> @@ -125,7 +125,7 @@
>> >>  <selectedcolor>d0ffffff</selectedcolor>
>> >>  </include>
>> >>
>> >> -<!-- Home - Backgrounds / Navigation
>> >> -----------------------------------
>> >>  -->
>> >> +<!-- Home - Backgrounds / Navigation -->
>> >>  <!-- Background MultiImages -->
>> >>  <include name="home-backgrounds">
>> >>  <type>multiimage</type>
>> >> @@ -156,7 +156,7 @@
>> >>  </include>
>> >>
>> >>
>> >> -<!-- Home - Sub-Buttons
>> >> -------------------------------------------------
>> >> -->
>> >> +<!-- Home - Sub-Buttons -->
>> >>
>> >>  <!-- DVD Launch Button -->
>> >>  <include name="home-dvd-button">
>> >> @@ -169,7 +169,7 @@
>> >>  <animation effect="fade" time="200">VisibleChange</animation>
>> >>  </include>
>> >>
>> >> -<!-- Home - Media Info's
>> >> ------------------------------------------------
>> >> -->
>> >> +<!-- Home - Media Info's -->
>> >>  <!-- Music Info -->
>> >>  <include name="home-music-info">
>> >>  <visible>Skin.HasSetting(home-music-info) + Player.HasAudio</visible>
>> >> @@ -181,12 +181,12 @@
>> >>  <animation effect="slide" start="720,0" end="0,0" acceleration="1"
>> >> time="200">WindowOpen</animation>
>> >>  </include>
>> >>
>> >> -<!-- Home - ScanLines -------------------------------------------- -->
>> >> +<!-- Home - ScanLines -->
>> >>  <include name="scanlines">
>> >>  <visible>Skin.HasSetting(scanlines)</visible>
>> >>  </include>
>> >>
>> >> -<!-- Home / Menu - System InfoBar
>> >> ---------------------------------------
>> >> -->
>> >> +<!-- Home / Menu - System InfoBar -->
>> >>  <!-- InfoBar SystemInfo Label -->
>> >>  <include name="system-info-label">
>> >>  <type>label</type>
>> >> @@ -215,13 +215,13 @@
>> >>  </control>
>> >>  </include>
>> >>
>> >> -<!-- Menu Background - Visible
>> >> ------------------------------------------
>> >> -->
>> >> +<!-- Menu Background - Visible -->
>> >>  <!-- BG-Panel Overlay Fade/Idle Time -->
>> >>  <include name="menu-overlay">
>> >>  <visible>!Skin.HasSetting(menu-overlay)</visible>
>> >>  </include>
>> >>
>> >> -<!-- MusicOverlay Label - (MusicOverlay.xml / VideoOverlay.xml)
>> >> ---------
>> >> -->
>> >> +<!-- MusicOverlay Label - (MusicOverlay.xml / VideoOverlay.xml) -->
>> >>  <!-- MusicOverlay Label -->
>> >>
>> >>  <include name="musicoverlay-label">
>> >> @@ -241,7 +241,7 @@
>> >>  <include>textcolorwhite-50</include>
>> >>  </include>
>> >>
>> >> -<!-- Slide Panels - "SlideBar" - Visible / Animation Effects
>> >> ------------
>> >> -->
>> >> +<!-- Slide Panels - "SlideBar" - Visible / Animation Effects -->
>> >>  <!-- SeekBar -->
>> >>  <include name="seekbar">
>> >>  <visible>Player.Seeking | Player.DisplayAfterSeek | Player.Paused |
>> >> Player.Forwarding | Player.Rewinding + System.IdleTime(5)</visible>
>> >> @@ -264,7 +264,7 @@
>> >>  <animation effect="slide" end="720,0" acceleration="1.1"
>> >> time="250">WindowClose</animation>
>> >>  </include>
>> >>
>> >> -<!-- Slide Panels - "ContextMenu's / FileBrowser" - Animation Effects
>> >> ---
>> >> -->
>> >> +<!-- Slide Panels - "ContextMenu's / FileBrowser" - Animation Effects
>> >> -->
>> >>  <!-- ContextMenu's / FileBrowser -->
>> >>  <include name="slidepanel">
>> >>  <animation effect="slide" start="720,0" acceleration="-1"
>> >> condition="!System.IdleTime(5)" time="400">WindowOpen</animation>
>> >> @@ -280,7 +280,7 @@
>> >>  <visible>!Skin.HasSetting(menu-overlay)</visible>
>> >>  </include>
>> >>
>> >> -<!-- Codec / Media Info's Panels Visibility Effects - (Fullscreen)
>> >> ------
>> >> -->
>> >> +<!-- Codec / Media Info's Panels Visibility Effects - (Fullscreen) -->
>> >>  <include name="media-infopanel-effect">
>> >>  <animation effect="fade" time="400">VisibleChange</animation>
>> >>  </include>
>> >> @@ -292,18 +292,18 @@
>> >>  <animation effect="fade" time="50">Visible</animation>
>> >>  </include>
>> >>
>> >> -<!-- ButtonMenu  - Buttons / Icons - Animation Effects
>> >> ------------------
>> >> -->
>> >> +<!-- ButtonMenu  - Buttons / Icons - Animation Effects -->
>> >>  <include name="buttonmenu-button">
>> >>  <animation effect="fade" time="200"
>> >> reversible="false">Focus</animation>
>> >>  </include>
>> >>
>> >> -<!-- New Settings - Animation Effect -  Labels
>> >> -------------------------
>> >>  -->
>> >> +<!-- New Settings - Animation Effect -  Labels -->
>> >>  <include name="settings-labels">
>> >>  <visible>![Player.HasMedia + Skin.HasSetting(menu-visualisation) +
>> >> System.IdleTime(30)]</visible>
>> >>  <animation effect="fade" time="200">VisibleChange</animation>
>> >>  </include>
>> >>
>> >> -<!-- Dialogs - Animation Effects
>> >> ----------------------------------------
>> >> -->
>> >> +<!-- Dialogs - Animation Effects -->
>> >>  <include name="dialog-effect">
>> >>  <animation effect="fade" time="200">WindowOpen</animation>
>> >>  <animation effect="fade" time="200">WindowClose</animation>
>> >> @@ -313,12 +313,12 @@
>> >>  <animation effect="fade" time="200">WindowClose</animation>
>> >>  </include>
>> >>
>> >> -<!-- Radiobutton / Checkbox - Animation Effect
>> >> --------------------------
>> >> -->
>> >> +<!-- Radiobutton / Checkbox - Animation Effect -->
>> >>  <include name="radiobutton">
>> >>  <animation effect="fade" time="200">VisibleChange</animation>
>> >>  </include>
>> >>
>> >> -<!-- Menu Animations - Visible / Animation Effect
>> >> -----------------------
>> >> -->
>> >> +<!-- Menu Animations - Visible / Animation Effect -->
>> >>  <include name="slideright">
>> >>  <animation effect="slide" start="600,0" end="0,0" acceleration="-1.1"
>> >> condition="!Skin.HasSetting(animation)"
>> >> time="250">WindowOpen</animation>
>> >>  </include>
>> >> @@ -335,7 +335,7 @@
>> >>  </include>
>> >>
>> >>
>> >> -<!-- Home / Menu - Time and Date/CPU and RAM Labels
>> >> ---------------------
>> >> -->
>> >> +<!-- Home / Menu - Time and Date/CPU and RAM Labels -->
>> >>  <include name="time-and-date">
>> >>  <control>
>> >>  <description>Date</description>
>> >> @@ -415,7 +415,7 @@
>> >>  </control>
>> >>  </include>
>> >>
>> >> -<!-- Description and Number of Files Labels - Animation Effect
>> >> ----------
>> >> -->
>> >> +<!-- Description and Number of Files Labels - Animation Effect -->
>> >>  <include name="description">
>> >>  <type>label</type>
>> >>  <posx>20</posx>
>> >> @@ -444,7 +444,7 @@
>> >>  </control>
>> >>  </include>
>> >>
>> >> -<!-- Home Main-Top Shaders
>> >> ----------------------------------------------
>> >> -->
>> >> +<!-- Home Main-Top Shaders -->
>> >>  <include name="shader-home">
>> >>  <control>
>> >>  <type>image</type>
>> >> @@ -460,7 +460,7 @@
>> >>  </include>
>> >>
>> >>
>> >> -<!-- Common Menu - PAL4x3
>> >> -----------------------------------------------
>> >> -->
>> >> +<!-- Common Menu - PAL4x3 -->
>> >>  <include name="4x3menu-background">
>> >>  <control>
>> >>  <description>Background Right-Panel</description>
>> >> @@ -515,7 +515,7 @@
>> >>  </control>
>> >>  </include>
>> >>
>> >> -<!-- Common Menu Sliders - PAL4x3
>> >> ---------------------------------------
>> >> -->
>> >> +<!-- Common Menu Sliders - PAL4x3 -->
>> >>  <include name="4x3slider">
>> >>  <control>
>> >>  <description>Menu Main-Top</description>
>> >> @@ -571,7 +571,7 @@
>> >>  <include>menu-system-infobar</include>
>> >>  </include>
>> >>
>> >> -<!-- Common Menu - PAL16x9
>> >> ----------------------------------------------
>> >> -->
>> >> +<!-- Common Menu - PAL16x9 -->
>> >>  <include name="16x9menu-background">
>> >>  <control>
>> >>  <description>Background Right-Panel</description>
>> >> @@ -626,7 +626,7 @@
>> >>  </control>
>> >>  </include>
>> >>
>> >> -<!-- Common Menu Sliders - PAL16x9
>> >> --------------------------------------
>> >> -->
>> >> +<!-- Common Menu Sliders - PAL16x9 -->
>> >>  <include name="16x9slider">
>> >>  <control>
>> >>  <description>Menu Main-Top</description>
>> >> @@ -1063,7 +1063,7 @@
>> >>  <bordersize>1</bordersize>
>> >>  <visible>[!Control.IsVisible(59) | !Container(59).Row(0) |
>> >> !Container.Content(Movies)]</visible>
>> >>  <visible>[!Control.IsVisible(56) | !Container(56).Row(0) |
>> >> !Container.Content(MusicVideos)]</visible>
>> >> -<visible>!Skin.HasSetting(Time&Date)</visible>
>> >> +<visible>!Skin.HasSetting(TimeDate)</visible>
>> >>  <visible>!Window.IsActive(101) + !Window.IsActive(133) +
>> >> !Window.IsActive(112)</visible>
>> >>  </control>
>> >>
>> >> @@ -1079,7 +1079,7 @@
>> >>  <label>[COLOR white][UPPERCASE]$INFO[System.Date][/UPPERCASE]  |
>> >> [/COLOR][COLOR white]$INFO[System.Time][/COLOR]</label>
>> >>  <visible>[!Control.IsVisible(59) | !Container(59).Row(0) |
>> >> !Container.Content(Movies)]</visible>
>> >>  <visible>[!Control.IsVisible(56) | !Container(56).Row(0) |
>> >> !Container.Content(MusicVideos)]</visible>
>> >> -<visible>!Skin.HasSetting(Time&Date)</visible>
>> >> +<visible>!Skin.HasSetting(TimeDate)</visible>
>> >>  <visible>!Window.IsActive(101) + !Window.IsActive(133) +
>> >> !Window.IsActive(112)</visible>
>> >>  </control>
>> >>
>> >> @@ -1096,7 +1096,7 @@
>> >>  <texture>time.png</texture>
>> >>  <bordertexture>time1.png</bordertexture>
>> >>  <bordersize>1</bordersize>
>> >> -<visible>!Skin.HasSetting(Time&Date)</visible>
>> >> +<visible>!Skin.HasSetting(TimeDate)</visible>
>> >>  <visible>!Window.IsActive(101) + !Window.IsActive(133) +
>> >> !Window.IsActive(112)</visible>
>> >>  <animation effect="fade" time="200">WindowClose</animation>
>> >>  </control>
>> >> @@ -1110,7 +1110,7 @@
>> >>  <align>left</align>
>> >>  <aligny>center</aligny>
>> >>  <label>[COLOR white][UPPERCASE]$INFO[System.Date][/UPPERCASE]  |
>> >> [/COLOR][COLOR white]$INFO[System.Time][/COLOR]</label>
>> >> -<visible>!Skin.HasSetting(Time&Date)</visible>
>> >> +<visible>!Skin.HasSetting(TimeDate)</visible>
>> >>  <visible>!Window.IsActive(101) + !Window.IsActive(133) +
>> >> !Window.IsActive(112)</visible>
>> >>  <animation effect="fade" time="200">WindowClose</animation>
>> >>  </control>
>> >> @@ -1131,7 +1131,7 @@
>> >>  <bordersize>1</bordersize>
>> >>  <visible>[!Control.IsVisible(59) | !Container(59).Row(0) |
>> >> !Container.Content(Movies)]</visible>
>> >>  <visible>[!Control.IsVisible(56) | !Container(56).Row(0) |
>> >> !Container.Content(MusicVideos)]</visible>
>> >> -<visible>!Skin.HasSetting(Time&Date)</visible>
>> >> +<visible>!Skin.HasSetting(TimeDate)</visible>
>> >>  <visible>!Window.IsActive(101) + !Window.IsActive(133) +
>> >> !Window.IsActive(112)</visible>
>> >>  </control>
>> >>
>> >> @@ -1147,7 +1147,7 @@
>> >>  <label>[COLOR white][UPPERCASE]$INFO[System.Date][/UPPERCASE]  |
>> >> [/COLOR][COLOR white]$INFO[System.Time][/COLOR]</label>
>> >>  <visible>[!Control.IsVisible(59) | !Container(59).Row(0) |
>> >> !Container.Content(Movies)]</visible>
>> >>  <visible>[!Control.IsVisible(56) | !Container(56).Row(0) |
>> >> !Container.Content(MusicVideos)]</visible>
>> >> -<visible>!Skin.HasSetting(Time&Date)</visible>
>> >> +<visible>!Skin.HasSetting(TimeDate)</visible>
>> >>  <visible>!Window.IsActive(101) + !Window.IsActive(133) +
>> >> !Window.IsActive(112)</visible>
>> >>  </control>
>> >>
>> >> @@ -2002,7 +2002,7 @@
>> >>
>> >>  </include>
>> >>
>> >> -<!----Very cool CD animation borrowed from the excellent Night
>> >> skin---->
>> >> +<!--Very cool CD animation borrowed from the excellent Night skin -->
>> >>
>> >>  <include name="CD Spin">
>> >>  <animation type="WindowOpen">
>> >> @@ -2060,4 +2060,4 @@
>> >>  </include>
>> >>
>> >>
>> >> -</includes>
>> >> \ No newline at end of file
>> >> +</includes>
>> >> diff --git a/skin.back-row/language/English/strings.xml
>> >> b/skin.back-row/language/English/strings.xml
>> >> index 713c840..677c143 100644
>> >> --- a/skin.back-row/language/English/strings.xml
>> >> +++ b/skin.back-row/language/English/strings.xml
>> >> @@ -21,7 +21,7 @@
>> >>  <string id="31020">Reset Skin</string>
>> >>  <string id="31021">Disable Curtain Background</string>
>> >>  <string id="31022">Toggle Now Showing In Coverflow View</string>
>> >> -<string id="31023">Configure Look & Feel</string>
>> >> +<string id="31023">Configure Look &amp; Feel</string>
>> >>  <string id="31024">Configure Video Settings</string>
>> >>  <string id="31025">Configure Music Settings</string>
>> >>  <string id="31026">Configure Pictures Settings</string>
>> >> @@ -109,7 +109,7 @@
>> >>  <string id="31108">Recently Added Movies, Episodes &amp;
>> >> Music</string>
>> >>  <string id="31109">New Music</string>
>> >>  <string id="31110">Use Conditional Weather Backgrounds</string>
>> >> -<string id="31111">Movies & TV Shows Default To Root Level</string>
>> >> +<string id="31111">Movies &amp; TV Shows Default To Root
>> >> Level</string>
>> >>  <string id="31112">Enable Movie Title Labels In Fanart Wrap
>> >> View</string>
>> >>  <string id="31113">Submenu Custom Scripts</string>
>> >>  <string id="31114">Music Lyrics Script</string>
>> >> diff --git a/skin.back-row/language/Portuguese/strings.xml
>> >> b/skin.back-row/language/Portuguese/strings.xml
>> >> index 778ba54..a63e039 100644
>> >> --- a/skin.back-row/language/Portuguese/strings.xml
>> >> +++ b/skin.back-row/language/Portuguese/strings.xml
>> >> @@ -21,7 +21,7 @@
>> >>  <string id="31020">Reiniciar tema</string>
>> >>  <string id="31021">Desligar fundo em cortina</string>
>> >>  <string id="31022">Ligar 'a reproduzir' na vista de capas</string>
>> >> -<string id="31023">Configurar vista & funcionalidade</string>
>> >> +<string id="31023">Configurar vista &amp; funcionalidade</string>
>> >>  <string id="31024">Configurar opções de vídeo</string>
>> >>  <string id="31025">Configurar opções de música</string>
>> >>  <string id="31026">Configurar opções de fotografias</string>
>> >> @@ -109,7 +109,7 @@
>> >>  <string id="31108">Filmes e episódios adicionados recentemente &amp;
>> >> Música</string>
>> >>  <string id="31109">Nova música</string>
>> >>  <string id="31110">Usar fundos de acordo com condições
>> >> meteorológicas</string>
>> >> -<string id="31111">Filmes & Séries de TV ao nível da raiz, por
>> >> defeito</string>
>> >> +<string id="31111">Filmes &amp; Séries de TV ao nível da raiz, por
>> >> defeito</string>
>> >>  <string id="31112">Ligar etiquetas com título do filme na vista
>> >> Fanart</string>
>> >>  <string id="31113">Scripts personalizados do submenu</string>
>> >>  <string id="31114">Script de letras de música</string>
>> >> @@ -168,4 +168,4 @@
>> >>  <string id="31167">Itens</string>
>> >>  <string id="31168">Descarregar logótipos de TV com o scriptdo
>> >> 'ppic'</string>
>> >>  <string id="31169">Enable Clear Art in Video OSD</string>
>> >> -</strings>
>> >> \ No newline at end of file
>> >> +</strings>
>> >> diff --git a/skin.back-row/language/Swedish/strings.xml
>> >> b/skin.back-row/language/Swedish/strings.xml
>> >> index eef9b56..1f98891 100644
>> >> --- a/skin.back-row/language/Swedish/strings.xml
>> >> +++ b/skin.back-row/language/Swedish/strings.xml
>> >> @@ -114,7 +114,7 @@
>> >>   <string id="31108">Nyligen tillagda filmer, avsnitt &amp;
>> >> musik</string>
>> >>   <string id="31109">Ny musik</string>
>> >>   <string id="31110">Villkorliga väderbakgrunder</string>
>> >> -  <string id="31111">Movies & TV Shows Default To Root Level</string>
>> >> +  <string id="31111">Movies &amp; TV Shows Default To Root
>> >> Level</string>
>> >>   <string id="31112">Enable Movie Title Labels In Fanart Wrap
>> >> View</string>
>> >>  <string id="31113">Submenu Custom Scripts</string>
>> >>  <string id="31114">Music Lyrics Script</string>
>> >
>> >
>>
>> Something funny happened with the languages. I think you merged
>> Portuguese into Italian on accident? See here:
>>
>> http://xboxmediacenter.svn.sourceforge.net/viewvc/xboxmediacenter/Back-Row/language/Italian/strings.xml?r1=2898&r2=2897&pathrev=2898
>>
>> Cory
>
>

Great, now all files are compliant. Thanks for your help. Should help
us to catch xml errors (like the extraneous </window> and </controls>)
much easier, maybe eliminate a regression or two.

Pushed as 661a8e9.

Cory