Skip to content

Elementos - #946

Open
Manuel901 wants to merge 854 commits into
scakemyer:masterfrom
elgatito:master
Open

Manuel901 wants to merge 854 commits into
scakemyer:masterfrom
elgatito:master

Conversation

@Manuel901

@Manuel901 Manuel901 commented Jun 16, 2019

Copy link
Copy Markdown

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Translation

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

elgatito and others added 30 commits January 5, 2022 21:37
* Update strings.po

* Fix typo
* Fix for force view mode

The reaosn for the change:
- Since the default view mode value for all the view types in the Elementum settings are equal to "0",
Each navigation to any place in the addon force the view mode to be changed to "List" view (In my skin).
--- Probably "Container.SetViewMode(0)" in Kodi 19.2 force it to be changed to "List" as default since there is no view mode id of "0" in my skin (And I guess in others too)
- I also have another issue regarding to this:
One of the Lists in Elementum is used as a widget in my skin. 
So on each refreshing of the widget (Kodi startup/Reload skin etc...) actually runs Elemetum which activate the force view mode changing.
It cause that after each widget refreshing the first window in kodi that you navigate to it, its view mode will be changed according to what set in Elementum.

The fix:
It's so simple.
If the view mode is equal to "0" (Which is the defaul value), don't do any view mode changing.
Otherwise it will change it according to the Elementum settings.
In this case,  every change of the view modes inside Elementum by the skin context menu, the view mode is kept in Kodi/Skin/Addon cache or something (So at least I don't need this "Force" view mode changing). If for others these view mode changes are not kept in memory, they will need these options of "Force" changing and use it as always.

It was tested and works well.

Thanks.

* Fix for force view mode - Condition fix

* SetInfo doesn't have 'dbtype' key

ListItem setInfo function has 'infoLabels' dictionary argument.
But this dictionary doesn't have 'dbtype' key.
This is why there was error of: 
ERROR <general>: NEWADDON Unknown Video Info Key "dbtype"

The 'dbtype' only exist when getting the infoLabels from a ListItem.

For more information:

* ListItem setInfo: 
https://alwinesch.github.io/group__python__xbmcgui__listitem.html#ga0f1e91e1d5aa61d8dd0eac90e8edbf18

* ListItem info labels:
https://kodi.wiki/view/InfoLabels#ListItem

* SetInfo doesn't have 'dbtype' key

ListItem setInfo function has 'infoLabels' dictionary argument.
But this dictionary doesn't have 'dbtype' key.
This is why there was error of: 
' ERROR <general>: NEWADDON Unknown Video Info Key "dbtype" '

The 'dbtype' only exist when getting the infoLabels from a ListItem.

Fix: Added a function for removing this 'dbtype' field before using the ListItem's 'setInfo' function

For more information:

* ListItem setInfo: 
https://alwinesch.github.io/group__python__xbmcgui__listitem.html#ga0f1e91e1d5aa61d8dd0eac90e8edbf18

* ListItem info labels:
https://kodi.wiki/view/InfoLabels#ListItem

* SetInfo doesn't have 'dbtype' key

ListItem setInfo function has 'infoLabels' dictionary argument.
But this dictionary doesn't have 'dbtype' key.
This is why there was error of: 
' ERROR <general>: NEWADDON Unknown Video Info Key "dbtype" '

The 'dbtype' only exist when getting the infoLabels from a ListItem.

Fix: Added a function for removing this 'dbtype' field before using the ListItem's 'setInfo' function

For more information:

* ListItem setInfo: 
https://alwinesch.github.io/group__python__xbmcgui__listitem.html#ga0f1e91e1d5aa61d8dd0eac90e8edbf18

* ListItem info labels:
https://kodi.wiki/view/InfoLabels#ListItem
* Update DialogInsert.xml

There were problems with the appearance in various skins. Margins and some buttons were not well adapted to custom skins.

* Update DialogSelectLarge.xml

Fixing some visual bugs and skin polishing. There were also problems with displaying some buttons in custom skins.
* Visibility improvement

Just a small improvement in visibility on darker skins.

* Update DialogSelectLarge.xml
* Visibility improvement

Just a small improvement in visibility on darker skins.

* Update DialogInsert.xml
elgatito and others added 30 commits March 8, 2026 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.