Project

General

Profile

Actions
Copy link

Bug #114

closed

Skin Changes Merge

Added by dropnuts over 13 years ago. Updated over 13 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
User Interface
Target version:
3.0
Start date:
Due date:
% Done:

0%

Resolution:
fixed
Affected Version:

Description

Not sure what merges are and aren't possible on the xbox, but could we merge some of the skinning updates from xbmc?

As pointed out by Hitcher here: [http://www.xbmc4xbox.org/forum/viewtopic.php?p=8871#p8871] these updates have allowed him to clean up/reduce a lot of skinning code. This would help with memory consumption on the xbox, as well as porting skins across.

Actions
Copy link
 #1

Updated by arnova over 13 years ago

Really need more specifics than just "skin updates". Which features/revisions are we talking about?

Actions
Copy link
 #2

Updated by dropnuts over 13 years ago

I was afraid you would ask that. I was hoping it was a case of "Our skinning code is here, there's is here, merge" Ha.

I will dig through the xbmc Trac and speak to Hitcher and see what's needed. Thanks.

Actions
Copy link
 #3

Updated by dropnuts over 13 years ago

Okay. I've found some.

fadelabels
"backgroundcolor
[http://trac.xbmc.org/changeset/31008":http://trac.xbmc.org/changeset/26444][http://trac.xbmc.org/changeset/26740][http://trac.xbmc.org/changeset/26741]

I think these are the main ones for now. Any others I find I'll create new tickets.

Actions
Copy link
 #4

Updated by dropnuts over 13 years ago

26740 and 26741 already merged.

26444 and 31008 still requested.

Actions
Copy link
 #5

Updated by arnova over 13 years ago

26444 has been merged in r30636

31008 is less trivial for Xbox, so I doubt it will (ever) be merged

Actions
Copy link
 #6

Updated by dropnuts over 13 years ago

26444 was the major one for porting some skins. Thanks for that.

31008 I don't really understand to be honest, but Hitcher suggested it.

Actions
Copy link
 #7

Updated by bulkzooi over 13 years ago

The mentioned topic has an alternative on implementing the not merged 31008. This also gives an idea about how this feature can save RAM-usage for XBOX. If still not merged, this ticket can be closed.

Actions
Copy link
 #8

Updated by arnova over 13 years ago

Sorry, but I can't really spot "the alternative" you're mentioning. What I need are patches...

Actions
Copy link
 #9

Updated by arnova over 13 years ago

  • Status changed from New to Closed
  • Resolution set to fixed

r30666 should do what you want concerning the background color.

Actions
Copy link

Also available in: Atom PDF