stoli / Metropolis

Continuation of Metropolis skin for XBMC
Other
23 stars 22 forks source link

Use Skin Shortcuts addon for custom home items #256

Closed jingai closed 9 years ago

jingai commented 9 years ago

http://forum.xbmc.org/showthread.php?tid=178294

MacGyverr's had a stab at this: https://github.com/MacGyverr/Metropolis/tree/helix-custommenu-rewrite

jingai commented 9 years ago

A lot of this is in one massive commit:

https://github.com/MacGyverr/Metropolis/commit/8d288d76b80166a02b4f42be2d20cce236598ce7#diff-6ed3b9d8d2723487e6162eb621a63dc4R134

The changes pertinent to skin.shortcuts seem to be in: addon.xml Home.xml Includes.xml SkinSettings.xml Includes_Home?.xml Includes_HomeDeck.xml script-skinshortcuts*

MacGyverr commented 9 years ago

Done.

https://github.com/MacGyverr/Metropolis/commit/4b4dd260d33b6876129a2bc8425e7317da88df02

Try it without the Home.xml launched rebuild code, I think that the one in SkinSettings is enough.

jingai commented 9 years ago

Noting to update the features at http://kodi.wiki/view/Add-on:Metropolis when this is complete.

MacGyverr commented 9 years ago

Just finished the playlist widget and further HomeDeck refinements. https://github.com/MacGyverr/Metropolis/commit/421abb9105dbcb5701975c6efc23e2e72326c503 Example screenshots at the bottom in the comments.

jingai commented 9 years ago

Thanks MacGyver.

This particular issue is high on my priority list, but we have family coming into town for a week so it'll be a little bit before I can look over all you've done.

MacGyverr commented 9 years ago

I was going to include addons widgets like the other skins, but it seems kind of redundant as any addons can be added as a submenu to any menu (or as its own menu shortcut). I'm not sure I can see why anyone would want to have a widget that displays a list of ALL the addons? If I'm being short-sighted let me know and I'll code it up.
At this point the HomeDeck include file is almost a complete rewrite. Also, the in the comparison chart for Helix you have "Custom Home Items" showing no, and while the official version doesn't have complete customization options, it is by no means a No.
Anyway, have fun with the family. :)

jingai commented 9 years ago

You and I both know people (often adamantly) want some really strange things.. lol.. but hold off on that for now. I'll be giving this all a really thorough look when I get the chance. The core of Metropolis is close to where I want it to be now, so this will be addressed soon.

But, as for the feature on the wiki, I listed that as a No for now because I think what people really expect when they see that box ticked is the kind of customization addressed by this issue. Really.. it was debatable either way so I just held off on the "yes" until it's completely done (a la the mods you've made here).

jingai commented 9 years ago

Just noting this here:

http://forum.kodi.tv/showthread.php?tid=213669&pid=1890929#pid1890929

jingai commented 9 years ago

Getting closer on this.. few things remain:

MacGyverr commented 9 years ago

"media overlay for music overlaps the widgets" My version solves this for the home menus, I just need to add the settings menus to the list of conditionals of where not to display. Cut and paste as needed. :)

jingai commented 9 years ago

Which version are you referring to? The menurefactor branch was showing it overlapping because the condition in Includes_MediaOverlay.xml hadn't been updated.

MacGyverr commented 9 years ago

I stopped updating that branch because I was afraid it was getting too far off of the official for you to be able to merge properly. Do you have an official Menu-refactor branch that I can fork off of? Do you have an offline one that is now equal to it? I was updating my personal version with little fixes until you had an official branch with the menu-refactor, that way I could have an idea for code fixes that might come up with the switch to Jarvis.

jingai commented 9 years ago

Aye, that's why I've been watching your commits on your personal version :)

I will be pushing it all into master soon. I do have a branch at jingai/skin.metropolis currently, but I'm force-pushing it currently, so I don't recommend branching from there.. in fact, I'll go ahead and merge this in so you can help out if you want to. I'll make separate Issues for each thing that still needs to be completed.

It's almost done really. Last remaining issues/questions:

jingai commented 9 years ago

It's all sitting at https://github.com/stoli/Metropolis/tree/refactor now. I'm going to make separate issues for the things that need to be done before I can release.

I won't force-push to that branch, so you're safe to branch from there if you want.

jingai commented 9 years ago

All Issues created. Closing this now.