bigretromike / nakamori

Nakamori is Kodi addon that use Shoko (known as Japanese Media Manager (JMM)) Server as back-end for metadata information.
https://shokunin.monogatari.pl/nakamori/
49 stars 5 forks source link

New context menu for calendar #230

Closed bigretromike closed 6 years ago

bigretromike commented 6 years ago

any suggestions ? @da3dsoul

bigretromike commented 6 years ago

cross this with some kind of trailer platform.... that would be sweet

da3dsoul commented 6 years ago

It would need to be a custom layout. I would make it a filter item in the main menu

bigretromike commented 6 years ago

yeah, I was thinking about WindowXML but currently its also not bad (as its in code anyway). plus is that that using current listview give us compatible for most (if not all) skins.

bigretromike commented 6 years ago

custom calendar v2 added; that is concept with bad graphic etc. was hell of challenge why it didn't work with keyboard but hell, did it and need to fix two things with it;

da3dsoul commented 6 years ago

Some feedback on the new Calendar:

bigretromike commented 6 years ago

There should be option for calendar with dates. Text over was for testing any idea how it should be hmm I can easily see them on 55 so .... Also there is option for that hehe ... But thanks for feedback we can edit it around your suggestions

On March 24, 2018 3:41:31 AM GMT+01:00, Thomas Baer notifications@github.com wrote:

Some feedback on the new Calendar:

  • Dates are inverted for my locale. We use 12/31/2018, not...whatever Europe uses.
  • The text can't go over the poster. Basically none of it is legible. White on white, in addition to being too small to read.
  • Make the posters 2-3x bigger. Scrolling is fine, but I can barely see them on my 50", so that says plenty.

-- You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub: https://github.com/bigretromike/nakamori/issues/230#issuecomment-375840940

-- Sent from my Android device with K-9 Mail. Please excuse my brevity.

da3dsoul commented 6 years ago

I actually didn't look at the settings.... I guess I'm too used to you just adding something and needing to add settings myself haha. The text is white and most of the posters are mostly white, so the text needs to be above or below the poster, but not on it. And do be aware that I'm blind. I can't read small text in general.

bigretromike commented 6 years ago

I wanted disable text but then it would be harder to know the titles

On March 24, 2018 4:17:31 PM GMT+01:00, Thomas Baer notifications@github.com wrote:

I actually didn't look at the settings.... I guess I'm too used to you just adding something and needing to add settings myself haha. The text is white and most of the posters are mostly white, so the text needs to be above or below the poster, but not on it

-- You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub: https://github.com/bigretromike/nakamori/issues/230#issuecomment-375896988

-- Sent from my Android device with K-9 Mail. Please excuse my brevity.

bigretromike commented 6 years ago

Prototype added

https://github.com/bigretromike/nakamori.script/commit/ecc5e6a65089686cf407d740585c68b91a90449e

bigretromike commented 6 years ago

Context menu have been added - not functional that much but is added and handle all needed action;